为什么Mockito“when”在非模拟对象上工作？

Question

我最近看到了一些Mockito 1.9.5代码，其工作原理如下：

MyObject myObject = new MyObject();
...
Mockito.when(myObject.someMethod()).thenReturn("bogus");

由于myObject不是一个模拟对象，而是一个非模拟类的实例，所以我很惊讶，它编译并运行时没有通过单元测试。我希望我会遇到一个失败，比如“你让我在非模拟对象上设置一个期望值，而我希望只在模拟对象上设定期望值。”

为什么这段代码不会导致测试失败？

更新:添加更多的代码，这是实际复制我感到困惑的行为所必需的。这些例子充分说明了我的问题。下面的代码如我所料——当我运行这个测试时，测试失败，并显示一条消息

when()需要一个必须是“模拟上的方法调用”的参数。

public class AnotherObject{
    public String doSomething(){
        return "did something";
    };
}

public class MyObject{
    private AnotherObject anotherObject = new AnotherObject();

    public void setAnotherObject(AnotherObject anotherObject) {
        this.anotherObject = anotherObject;
    }

    public String someMethod(){
        return anotherObject.doSomething();
    }
}

@Test
public void WhyDoesWhenWorkOnNonMock() throws Exception {
    MyObject myObject = new MyObject();
    Mockito.when(myObject.someMethod()).thenReturn("bogus");
}

现在，如果我在这个设计好的测试中添加几行特定的代码，测试将不再失败，尽管我预计会出现与之前相同的失败和消息:

public class AnotherObject{
    public String doSomething(){
        return "did something";
    };
}

public class MyObject{
    private AnotherObject anotherObject = new AnotherObject();

    public void setAnotherObject(AnotherObject anotherObject) {
        this.anotherObject = anotherObject;
    }

    public String someMethod(){
        return anotherObject.doSomething();
    }
}

@Test
public void WhyDoesWhenWorkOnNonMock() throws Exception {
    MyObject myObject = new MyObject();
    AnotherObject mockAnotherObject = Mockito.mock(AnotherObject.class);
    myObject.setAnotherObject(mockAnotherObject);
    Mockito.when(myObject.someMethod()).thenReturn("bogus");
}

Answer 1

可能是不可思议且脆弱的巧合，除非myObject实际上被设置为间谍。

Mockito允许创建真实对象的“间谍”：

MyObject myObject = spy(new MyObject());
Mockito.when(myObject.someMethod()).thenReturn("something");

// myObject is actually a duplicate of myObject, where all the fields are copied
// and the methods overridden. By default, Mockito silently records interactions.

myObject.foo(1);
verify(myObject).foo(anyInt());

// You can stub in a similar way, though doReturn is preferred over thenReturn
// to avoid calling the actual method in question.

doReturn(42).when(myObject).bar();
assertEquals(42, myObject.bar());

除此之外，这段代码可能没有按照它看起来应该的方式工作。当的参数没有意义时，糖是为了隐藏模拟的交互是对模拟的最新方法调用。例如：

SomeObject thisIsAMock = mock(SomeObject.class);
OtherObject notAMock = new OtherObject();

thisIsAMock.methodOne();
Mockito.when(notAMock.someOtherMethod()).thenReturn("bar");
// Because notAMock isn't a mock, Mockito can't see it, so the stubbed interaction
// is the call to methodOne above. Now methodOne will try to return "bar",
// even if it isn't supposed to return a String at all!

像这样的不匹配很容易成为ClassCastException，InvalidUseOfMatchersException和其他奇怪错误的来源。也有可能你的真正的MyObject类将模拟作为参数，并且最后一次交互是与someMethod交互的模拟。

您的编辑证实了我的怀疑。就Java而言，它需要先计算参数到何时，然后才能调用何时，因此您的测试调用someMethod（real）。Mockito看不到这一点 - 它只能在您与其中一个模拟交互时采取行动 - 因此在您的第一个示例中，它看到与模拟的交互为零，因此失败。在第二个示例中，您的someMethod调用doSomething，Mockito可以看到，因此它返回默认值（null）并将其标记为最新的方法调用。然后调用when（null）发生，Mockito忽略参数（null）并引用最近调用的方法（doSomething），并存根以从该点开始返回“bogus”。

通过将这个断言添加到您的测试中，您可以看到这一点，即使您从未显式地删除它:

assertEquals("bogus", mockAnotherObject.doSomething());

作为额外的参考，我在Mockito matchers上写了一个单独的SO答案，其中的实现细节可能会有用。有关类似问题的扩展视图，请参见步骤5和6。