4

如果我必须测试使用可变实体的服务，我将构建我需要的最小对象（真实对象）并将其传递给我的服务。例子：

User joe = new User();
joe.setEmail("joe@example.com");

resetPasswordService.resetPassword(joe);

verif(emailServiceMock).sendEmail("joe@example.com", "Your password has been reset!");

显然用户有很多字段，但我没有设置它们，因为resetPasswordService不需要它们。这对重构非常友好，因为如果我重命名不是电子邮件的用户字段，则不会更改此测试。

当我尝试对Immutables对象执行相同操作时，就会出现问题。我将坚持使用相同的示例，并将 User 从实体转变为不可变的。

@Value.Immutable
public abstract class User {
    public abstract String getEmail();
    public abstract PostalAddress getPostalAddress();
    //more fields
}

User joe = new ImmutableUserBuilder().email("joe@example.com").build();

resetPasswordService.resetPassword(joe);

verif(emailServiceMock).sendEmail("joe@example.com", "Your password has been reset!");

java.lang.IllegalStateException：无法构建用户，一些必需的属性没有设置 [postalAddress, signupDate, city, ....]

当构建器尝试构建对象时，它会失败。所以我该怎么做？

  • 为用户使用模拟并让它返回模拟，即使每次模拟返回模拟时，仙女都会死去
  • 创建一个测试 DSL 并拥有某种工厂来构建包含所有我不需要的字段的整个用户树结构？看起来很重，而且对重构不太友好。这使得测试的要求不那么透明。
  • 使用户中@Nullable的所有字段并让构建器不验证对象？这会使我面临在生产中拥有不完整对象的风险，对吧？
  • 我错过了其他选择？

我知道用户应该是实体而不是不可变的值对象。我在这个例子中使用了 User，因为它很容易理解。

4

2 回答 2

11

简单的答案：只有在必要时才使用模拟。

含义：当您需要以“真实”类不支持的方式控制对象的行为时。或者当您必须在模拟上验证调用时。

所以：当你可以编写一个测试用例来做你想做的事情而不使用模拟时——那就去做吧。

模拟框架是工具。你不是因为你可以使用它们，而是因为它们为你解决了一个你无法（容易）解决的问题。

除此之外：正如所解释的，默认值应该是避免模拟。另一方面，编程总是关于平衡努力和“投资回报”。这就是我在上面轻松使用这个词的原因。当事实证明使用模拟导致写下 2、3 行易于理解的代码时......但使用“真实”类要复杂得多（或依赖于关于该类如何工作的某些隐含假设） - 那么使用模拟可能是更好的选择。

从这个意义上说，答案是：不要把答案和规则作为黄金标准。最后，这总是关于人类的判断。

于 2017-08-30T09:37:23.000 回答
3

您的测试当前依赖于密码重置功能的实现细节。

这是您要测试的行为：

  • 给定一个用户
  • 当该用户请求重置密码时
  • 然后发送一封电子邮件

假设您稍后决定更改密码重置功能，以便电子邮件包含他们的姓名：

亲爱的乔，

您已请求重设密码...

您的测试现在将失败，因为您的测试策略是基于实例永远不需要名称NullPointerException的假设。User一个完全无害的变化导致我们的测试在它应该通过的时候却失败了。

解决方案：使用实物。如果您发现在不同的测试中创建了大量用户，请将用户创建重构为自己的功能：

private User getUser()
{
    User joe = new User();
    joe.setEmail("joe@example.com");
    joe.setName("Joe");
    joe.setAge(20);
    joe.setHeight(180);
    return joe;
}
于 2017-08-30T10:02:17.980 回答