Mockito：如果传递给 mock 的参数被修改了怎么办？

Question

我们在 Mockito 上遇到了非常糟糕的问题。

代码：

public class Baz{
    private Foo foo;
    private List list;

    public Baz(Foo foo){
        this.foo = foo;
    }

    public void invokeBar(){
        list = Arrays.asList(1,2,3);
        foo.bar(list);
        list.clear();
    }

}


public class BazTest{

   @Test
   void testBarIsInvoked(){
        Foo mockFoo = mock(Foo.class);
        Baz baz = new Baz(mockFoo);           
        baz.invokeBar();
        verify(mockFoo).bar(Arrays.asList(1,2,3));
   }
}

这会导致如下错误消息：

Arguments are different! Wanted: 
foo.bar([1,2,3]); 
Actual invocation has different arguments:
foo.bar([]);

刚刚发生的事情：

Mockito 记录 reference 到 list 而不是 list 的副本，因此在上面的代码中，Mockito 验证 修改版本（空列表，[]）而不是调用期间实际传递的那个 ([1,2,3])！

问题：

除了像下面那样做一个防御性副本（这实际上有帮助，但我们不喜欢这个解决方案）之外，还有什么优雅而干净的解决方案吗？

   public void fun(){
        list = Arrays.asList(1,2,3);
        foo.bar(new ArrayList(list));
        list.clear();
    }

我们不想修改正确生产代码并降低其性能只是为了解决测试的技术问题。

我在这里问这个问题是因为这似乎是 Mockito 的常见问题。还是我们做错了什么？

附言。这不是一个真实的代码，所以请不要问我们为什么要创建一个列表然后清除它等等。在真实的代码中，我们确实需要做类似的事情:-)。

Answer 1

这里的解决方案是使用自定义答案。两个代码示例：第一个是使用的测试类，第二个是测试。

一、测试类：

private interface Foo
{
    void bar(final List<String> list);
}

private static final class X
{
    private final Foo foo;

    X(final Foo foo)
    {
        this.foo = foo;
    }

    void invokeBar()
    {
        // Note: using Guava's Lists here
        final List<String> list = Lists.newArrayList("a", "b", "c");
        foo.bar(list);
        list.clear();
    }
}

开始测试：

@Test
@SuppressWarnings("unchecked")
public void fooBarIsInvoked()
{
    final Foo foo = mock(Foo.class);
    final X x = new X(foo);

    // This is to capture the arguments with which foo is invoked
    // FINAL IS NECESSARY: non final method variables cannot serve
    // in inner anonymous classes
    final List<String> captured = new ArrayList<String>();

    // Tell that when foo.bar() is invoked with any list, we want to swallow its
    // list elements into the "captured" list
    doAnswer(new Answer()
    {
        @Override
        public Object answer(final InvocationOnMock invocation)
            throws Throwable
        {
            final List<String> list
                = (List<String>) invocation.getArguments()[0];
            captured.addAll(list);
            return null;
        }
    }).when(foo).bar(anyList());

    // Invoke...
    x.invokeBar();

    // Test invocation...
    verify(foo).bar(anyList());

    // Test arguments: works!
    assertEquals(captured, Arrays.asList("a", "b", "c"));
}

当然，能够编写这样的测试需要你能够注入足够的状态到你的“外部对象”中，这样​​测试才有意义……这里就比较简单了。