Mockito 中模拟方法的单元测试最佳实践

Question

假设我们有在类 A 中测试的方法，该方法调用类 B 中的方法。为了测试它，我们为 B 创建了模拟，然后验证它是否被调用。 verify(...) 是否足以进行单元测试，或者我需要断言测试方法的实际结果？ 下面是澄清我的担忧的简化示例：

public class StringWriterATest {
    StringWriterB b = mock(StringWriterB.class);

    @Test
    public void stringWriterATest() {
        StringBuffer sb = new StringBuffer();
        StringWriterA a = new StringWriterA();
        a.stringWriterB=b;

        a.append(sb);

        ArgumentCaptor<StringBuffer> argument = ArgumentCaptor.forClass(StringBuffer.class);
        verify(b).append(argument.capture());

        assertEquals("StringWriterA", ((StringBuffer)argument.getValue()).toString());

        //do we really need this or above is enough for proper unit test of method a.append(sb); 
        //assertEquals("StringWriterA_StringWriterB", sb);
    }
}


public class StringWriterA {
    public StringWriterB stringWriterB;

    public void append(StringBuffer sb) {
        sb.append("StringWriterA");
        stringWriterB.append(sb);
    }
}

class StringWriterB {
    public void append(StringBuffer sb) {
        sb.append("StringWriterB");
    }
}

问候， 最大限度

Answer 1

永远不需要同时模拟返回值和验证对象。

考虑一下：

StringWriterA 是被测试的类。因此，您肯定会想要使用断言来验证此类的行为。为此，您需要模拟一个依赖项 StringWriterB。

您不想想要在 StringWriterA 测试中测试 StringWriterB，因此测试中 StringWriterB 交互的任何断言在错误的地方。

您必须假设 StringWriterB 的行为符合预期。您要么想要验证 StringWriterA 是否正确调用了 StringWriterB（使用 verify()），要么想要模拟它预期行为并模拟返回值。

如果您进行模拟，则验证是隐式的，因为如果未调用该方法，则不会返回模拟的返回值。

在您的情况下，StringWriterA.append() 不会返回任何值，因此甚至只能进行验证。 StringWriterB.append() 也可以工作，应该在它自己的 stringWriterBTest 中进行类似的验证测试。

注意： 明确地进行测试是件好事。由于测试方法永远不会在框架之外调用，因此永远不需要将它们键入，因此您可以使用比生产代码方法中更长的方法名称。一个很好的约定是：

Should[When]()

即

stringWriterAShoulderAppendConstantAndDelegateToStringWriterB()
stringWriterAShouldThrowNullPointerExceptionWhenNullArgument()

当您的构建（持续集成）中出现测试失败时，您不必追查出了什么问题，方法名称会在失败处出现，您可以读取它确切地知道必须纠正哪些行为。

Answer 2

在您的示例中，StringWriterB 不存储任何状态，并且 append 方法很容易是静态的。在这种情况下，调用纯粹是副作用，不需要测试。

但是，我怀疑您的真实代码要复杂得多。如果有另一个对象正在访问 StringWriterB，那么您可能需要模拟它，以防出现意外的调用。如果您希望将来扩展 B，您可能还需要添加 B 的验证 - 可能会存储来自追加调用的状态并具有访问器。

需要考虑的一件事是调用 StringWriterA.append() 的目的是什么。如果它的工作是附加字符串 StringWriterAStringWriterB 那么这就是您应该测试的内容，并且不需要模拟。 StringWriterA 如何完成该任务并不重要。但是，如果其工作的一部分还包括调用 StringWriterB.append() 方法，则可能需要模拟，除非您想在 A 的测试中测试 StringWriterB。

我的 WRT 模拟经验法则是使用真实的对象，直到我不直接测试的对象的接线变得太毛茸茸或太脆弱。如果我觉得我的测试的很大一部分实际上是在测试其他对象，那么模拟将是一个好主意。

Answer 3

首先尝试向其他人展示您编写的测试。在我看来，这很难读。 您无法真正从中看出您正在测试什么行为。可以在此处找到有关如何使其更具可读性的简要介绍 如何编写干净、可测试的代码代码。
使用参数捕获器也是一种味道。有关如何避免这种情况的一些示例，请参见tdd 博客< /a>.

为了回答您的问题，verify 用于验证类之间的交互。 它用于驱动代码的设计。结果（如果需要）应由when或given在处指定>测试开始。

有关如何使用模拟驱动设计的更多信息（when、given、verify...）以及模拟与存根有何不同，请参阅在这里找到：模拟不是存根。该示例使用 JMock 而不是 Mockito 来定义模拟，但它非常相似（它是关于概念，而不是您使用的实现和库的细节）。