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

Question

想象一下对以下示例场景进行单元测试。假设我已经模拟了 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;
    }
}

Answer 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，并且该功能不一定存在，那么您可以排除该属性，客户端只需检查是否有任何订单，方式与您的 customerDTO 相同.getCustomerOrders.isEmpty()

Answer 2

您想要测试 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));
}

因为这样，当有 1 个错误和 4 个错误时，就有一项测试失败。您修复了一个错误并期望测试能够通过，但随后在下一行再次失败。将所有断言放入不同的测试中并使用描述名称。