Question

我有一个非常简单的方法，我试图进行单元测试：

public class MyAntTask extends org.apache.tools.ant.Task {
    public void execute() {
        fire();
    }

    public void fire() {
        // Do stuff
    }
}

我只想编写一个单元测试，确认调用execute()总是调用fire()，所以我写了这个：

@Test
public void executeCallsFire() {
    //GIVEN
    MyAntTask myTask = Mockito.mock(MyAntTask.class);

    // Configure the mock to throw an exception if the fire() method
    // is called.
    Mockito.doThrow(new RuntimeException("fired")).when(myTask).fire();

    // WHEN
    try {
        // Execute the execute() method.
        myTask.execute();

        // We should never get here; HOWEVER this is the fail() that's
        // being executed by JUnit and causing the test to fail.
        Assert.fail();
    }
    catch(Exception exc) {
        // THEN
        // The fire() method should have been called.
        if(!exc.getMessage().equals("fired"))
            Assert.fail();
    }
}

我猜（我不是专家）Mockito通常无法模拟返回void的方法，但这是一种解决方法。你基本上说“用Mock包裹我的对象，只要特定方法即将被执行，它将始终返回特定的RuntimeException”。因此，Mockito只是看到它即将执行并抛出异常，而不是fire()实际执行。执行验证？检查。

而不是传递，它在Assert.fail()的呼叫正下方的第一个myTask.execute()处失败。

对于我的生活，我无法弄清楚为什么。这是JUnit为失败提供的巨大堆栈跟踪的前10行左右：

java.lang.AssertionError
    at org.junit.Assert.fail(Assert.java:92)
    at org.junit.Assert.fail(Assert.java:100)
    at net.myproj.ant.tasks.MyAntTaskUnitTest.executeCallsFire(MyAntTaskUnitTest.java:32)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke(Method.java:616)

这里有任何想法，你是 StackOverflow的Mockito Gurus ？提前谢谢！

Answer 1

因为myTask是一个模拟器，所以根本不调用真实对象。要调用真实对象，请使用间谍。

您可以测试使用验证调用方法，因此不需要例外。

public void executeCallsFire() {
    MyAntTask myTask = Mockito.spy(new MyAntTask());

    myTask.execute();

    Mockito.verify(myTask).fire();
}

想要模拟您正在测试的对象似乎并不正确。通常最好设计测试，以便您验证对单独对象的调用。

Answer 2

我在这里看到更多设计问题：

为什么你需要一种方法，而且这两种方法都是公开的？
模拟用于模拟依赖关系，而不是用于测试中的类
如果你把火（名字很不清楚）当作私人。你不应该测试你班级的私人行为

JUnit / Mockito测试失败的原因很奇怪

2 个答案: