JUNIT 空指针异常

Question

这是我第一次使用 JUNIT，我真的无法让它工作。

我有一个 person 类，其中包含 firstName 和 lastName，还有一个测试类，我需要在其中测试方法。但是，每次我尝试测试一个时，如果我为特定方法编写了一个测试，它就会失败。

这是我的代码。

人员类

public class Person {
    private String firstName;
    private String lastName;    

    public Person (String a, String b) {
        firstName = a;
        lastName = b;
    }

    public String getfirstName() {
        return firstName;
    }

    public void setfirstName(String firstName) {
        this.firstName = firstName;
    }

    public String getlastName() {
        return lastName;
    }

    public void setlastName(String lastName) {
        this.lastName = lastName;
    }

    @Override
    public String toString() {
        return  firstName + " " + lastName;
    }
}

人物测试类

public class PersonTest {
    Person p1;

    public PersonTest() {
        Person p1 = new Person ("Thomas", "Brown");
    }

    @Before
    public void setUp() {}

    @After
    public void tearDown() {}

    @Test
    public void testGetfirstName() {
        assertEquals("Thomas", p1.getfirstName());
    }

    @Test
    public void testSetfirstName() {}

    @Test
    public void testGetlastName() {
        assertEquals("Brown", p1.getlastName());
    }

    @Test
    public void testSetlastName() {}

    @Test
    public void testToString() {
        assertEquals("Thomas Brown", p1.toString());
    }
}

谁能指出我正确的方向？

Answer 1

这是正确的做法：

@Before
public void setUp() {
    p1 = new Person ("Thomas", "Brown");
}

您的代码中有 2 个问题。

public PersonTest() {
    Person p1 = new Person ("Thomas", "Brown");
}

这将创建一个局部变量，并且您的字段 p1 保持为 null。

第二个是在你的setUp方法中你没有初始化p1。

您使用@Before 注释的方法将在每次测试之前运行，您应该将初始化放在那里。我还建议使用更具描述性的名称，因此您应该将 p1 更改为 target 或类似的名称。

编辑：对于您的 set... 方法，您可以执行以下操作：

public class PersonTest {
    private static final String TEST_FIRST_NAME = "some name";

    Person target;

    // ...
    @Test
    public void testSetFirstName() {
        target.setFirstName(TEST_FIRST_NAME);
        Assert.assertEquals(target.getFirstName(), TEST_FIRST_NAME);
    }
}

此时您可以假设 getFirstName 有效，因为您也对其进行了测试。

旁注：我认为只要使用 Eclipse 生成 getter 和 setter，就不必测试它们。这只是不必要的。

Answer 2

改变

Person p1 = new Person ("Thomas", "Brown"); //this is local variable

到

 p1 = new Person ("Thomas", "Brown");// and this will use the instance variable