测试从单元返回的对象的所有可能条件或状态是否正确?

时间:2011-05-25 19:06:49

标签: unit-testing

想象一下单元测试以下示例场景。假设我已经嘲笑了CustomerDAO,以便返回有效的客户和客户订单。测试这种情况非常简单。除非我开始测试客户是否有订单的布尔值。在一些现实世界的场景中,他们不会有订单。那么我需要在我的模拟DAO中添加一些条件来返回没有订单的客户,然后测试它吗?现在想象它要复杂得多,并且有几个DTO可以包含各种信息,具体取决于从数据库返回的实际结果。我是否需要测试所有这些不同的条件?

public class Manager {

    public CustomerDTO getCustomerInformation() {

        CustomerDAO customerDAO = new CustomerDAO();

        CustomerDTO customerDTO = new CustomerDTO();
        customerDTO.setCustomer(customerDAO.getCustomer(1));
        customerDTO.setCustomerOrders(customerDAO.getCustomerOrders(1));
        if (!customerDTO.getCustomerOrders.isEmpty()) {
            customerDTO.setHasCustomerOrders(true);
        }

        return customerDTO;
    }
}

2 个答案:

答案 0 :(得分:1)

简而言之,我认为是的,当DAO返回DTO上预期状态存在的各种事物时,你应该测试一下。否则,您如何确信DTO能够准确表示数据存储区中的数据?

您应该为每个测试创建一个带有测试所需数据的模拟DAO,或者为每个需要测试的状态创建一个模拟DAO。

我可能会有以下测试:

CustomerHasOrders_WhenDaoReturnsNoOrders_ReturnsFalse

CustomerHasOrders_WhenDaoReturnsOrders_ReturnsTrue

GetCustomer_WhenDaoReturnsCustomer_CustomerIsSame

GetCustomerOrders_WhenDaoReturnsOrders_OrdersAreTheSame

GetCustomerOrders_WhenDaoReturnsNoOrders_OrdersAreEmpty

然后测试如果对Dao的任何调用失败会发生什么......

在这个例子中,DTO中的标志似乎是多余的,因为它只是其他数据的不同表示。您可以将其实现为CustomerDTO上的扩展方法,或setCustomerOrders中的某些逻辑。如果DTO要通过电汇发送,并且功能不会在那里,那么您可以排除该属性,客户可以只是检查是否有任何订单,就像您{{1} }

答案 1 :(得分:1)

您想测试Manager是否将Customer及其相关数据转换为CustomerDTO。所以是的,你必须测试所有这些不同的场景。但是不要在一次测试中全部完成。每项测试都应该在成功或失败时完成。我的意思是:


// Don't do this
[Test]
public void Manager_Converts_Customer_to_CustomerDTO()
{
  // mock setup
  var dto = Manager.GetCustomer();

  Assert.That(dto, Is.Not.Null);
  Assert.That(dto.Firstname, Is.EqualTo("what has been setup in mock"));
  Assert.That(dto.Orders.Count, Is.EqualTo(expected_set_in_mock));
  Assert.That(dto.Orders[0].Product, Is.EqualTo(expected_set_in_mock_product));
}

因为当有一个错误和有4个错误时,你有一个测试失败。你修复了一个bug并期望测试通过但是它在下一行再次失败。将所有断言放在不同的测试中并描述名称。