为什么mockito在java 7而不是java 6中的then回报（）上报告错误

Question

Mockito在模拟客户端时报告未完成的存根错误。getPrograms（）应该返回SortedSet

下面是模拟客户端时触发错误的代码。getPrograms（）：

private void prepareScheduleChangePreconditions() {
    Client client = mock(Client.class);
    TimeTable tt = BuilderUtil.buildTable(AcceleratedScheduleTimeTable.Schedule.NORMAL, "08:00");
    when(clientRepository.findByCode(anyString())).thenReturn(client);
    //Error is reported for next line of code
    when(client.getPrograms()).thenReturn(new TreeSet<Program>(Collections.singleton(program)));
    when(event.getTimeTable()).thenReturn(tt);      
}

以下是错误输出：

Tests in error:
  testExampleScheduleChangeNotify1(com.example.service.impl.ExampleServiceImplTest):
Unfinished stubbing detected here:
-> at com.example.service.impl.ExampleServiceImplTest.prepareScheduleChangePreconditions(ExampleServiceImplTest.java:134)

E.g. thenReturn() may be missing.
Examples of correct stubbing:
    when(mock.isOk()).thenReturn(true);
    when(mock.isOk()).thenThrow(exception);
    doThrow(exception).when(mock).someVoidMethod();
Hints:
 1. missing thenReturn()
 2. you are trying to stub a final method, you naughty developer!

方法不是最终的。任何帮助或线索都将不胜感激。

更新1

根据Mike B的请求，我成功地将其隔离到在Java 7中失败的更简单的测试用例中。

@RunWith(MockitoJUnitRunner.class)
public class MockitoTest {

    @Mock
    Program program;

    private void preparePreconditions() {
        Client client = mock(Client.class);
        when(client.getPrograms()).thenReturn(new TreeSet<Program>(Collections.singleton(program)));
    }

    public static class Client {
        public SortedSet<Program> getPrograms() {
            return new TreeSet<Program>();
        }
    }

    public static class Program implements Comparable<Program> {
        private String name;

        public String getName() {
            return name;
        }

        public void setName(String name) {
            this.name = name;
        }

        @Override
        public int compareTo(Program program) {
            return 0;
        }
    }

    @Test
    public void test() {
        preparePreconditions();
    }

}

更新2

奇怪的是，如果我这样做的话，它会起作用：

TreeSet<Program> programs = new TreeSet<Program>();
programs.add(program);
when(client.getPrograms()).thenReturn(programs);




Answer 1

这实际上并不能回答你的问题，但请记住嘲弄的基本指令：

1. 模拟角色，而不是对象

实际上，这意味着嘲笑类（即不是接口）是闪烁的红色警告信号，表明您的设计有问题。

如果您无法控制与之接口的代码（例如，它是第三方库或遗留软件），我喜欢将类包装在接口中并模拟它。作为一个额外的好处，这也允许您在其他代码中重命名名称不佳的方法（和类）。从您的示例中，客户端返回哪些“程序”？是活动程序集，正在运行的程序，还是什么？让我们假设它是活动程序集：

public interface Programs {
    SortedSet<Program> active();
}

class SimpleClientWrapper implements Programs {
    private final Client wrapped;

    SimpleClientWrapper(Client c) { wrapped = c; }

    public SortedSet<Program> active() { return wrapped.getPrograms(); }
}

如果这感觉工作太多（我承认有时这是过度的），另一种可能性，如果你想模拟的方法不是最终的，是在你的测试中暂时覆盖它们：

Client client = new Client() {
    @Override
    public SortedSet<Program> getPrograms() {
        return new TreeSet<Program>(Collections.singleton(program));
    }
};

我有时会使用这种方法来重写错误处理程序等。

Answer 2

我相信这里真正发生的是在（…）的情况下，在单个中调用多个模拟方法。然后（…） 语句。

考虑下面的例子：

when(program.getName()).thenReturn(String.valueOf(program.compareTo(null)));

它返回与您相同的异常。这是因为在mock上调用了两个方法：getName（）和compareTo（）在singlewhen（…）中。然后返回（…） 语句。

在本页（mockito文档）中，您还可以阅读：

默认情况下，对于返回值的所有方法，mock返回null、空集合或适当的基元/基元包装值（例如：0、false、...对于int/intger、boolean/Boolean、...）。

因此，mockito必须有某种机制来检测对某个被模拟对象的哪个调用要做什么（返回）。

在您的示例中，第二次调用由new TreeSet进行

TreeSet的实现似乎从Java6变成了Java7。这就是为什么它本可以更早起作用。