我会试一试:。
首先,我没有看到您编写测试的方式有任何不正确之处。
根据测试用例描述,我假设测试用例用于存储联系人列表的AddressBook类,并且您正在测试AddressBook类公开的方法addContact。
也就是说,您仍然可以通过在addContact方法中执行以下操作来使您的类更加健壮:
public void addContact(Contact contact) throws IllegalArgumentException
{
if(contact == null)
{
//throw an exception after logging that contact is null
throw new IllegalArgumentException("Passed in contact cannot be null!!")
}
this.contactsList.add(contact);
}
现在您的testAddOneContact测试代码必须测试两个不同的输入用例,这可以使用两个单独的测试用例如下完成
@Test
public void testAddOneContact() {
final Contact contact = this.context.mock(Contact.class);
this.addressBook.addContact(contact);
assertTrue("Object added to list", this.addressBook.getNumberOfContacts() == 1);
//assuming that Contact class implements the equals() method you can test that the contact
//added is indeed the one that you passed in
assertTrue(addressBook.get(0).equals(contact));
}
//the below test ensures that there is exception handling mechanism within your library code
@Test
@Expected(IllegalArgumentException.class)
public void testShouldThrowWhenContactIsNull()
{
this.addressBook.addContact(null);
}
顺便说一句 - 请注意实现一个好的测试类如何让您考虑要作为 API 公开的方法的设计,以及某些方法如何喜欢hashCode
和equals()
需要被覆盖。它也让你思考 - “我如何处理错误情况?”。这些深思熟虑的问题对于确保您交付的代码准确地解决它应该以有效且无错误的方式解决的问题至关重要。
希望这可以帮助