Mockito/JMockit & Hamcrest 匹配器：如何验证列表/集合？

Question

2013 post on SO 询问如何使用 Hamcrest 匹配器来验证 Mockito 中的列表/集合调用。公认的解决方案是将 Matcher 转换为 (Collection)。

我正在尝试做类似的事情，但遇到了类转换错误。我不确定我是否在滥用 Hamcrest 匹配器，或者 Mockito 是否根本不支持这种用法。就我而言，我正在尝试使用匹配器列表作为我的论点：

static class Collaborator
{
   void doSomething(Iterable<String> values) {}
}

@Test
public void usingMockito()
{
   Collaborator mock = Mockito.mock(Collaborator.class);
   mock.doSomething(Arrays.asList("a", "b"));

   // legal cast
   Mockito.verify(mock).doSomething((Collection<String>)argThat(Matchers.contains("a", "b")));
   // legal cast
   Mockito.verify(mock).doSomething((Collection<String>)argThat(Matchers.contains(Matchers.equalTo("a"), Matchers.equalTo("b"))));

   // illegal cast!!! Cannot cast from Iterable<capture#3-of ? extends List<Matcher<String>>> to Collection<String>
   Mockito.verify(mock).doSomething((Collection<String>)argThat(Matchers.contains(Arrays.asList(Matchers.equalTo("a"), Matchers.equalTo("b")))));
}

但我得到了转换错误：

Cannot cast from Iterable<capture#3-of ? extends List<Matcher<String>>> to Collection<String>

我在做一些不受支持的事情吗？

Answer 1

我更喜欢使用allOf

import static org.hamcrest.Matchers.allOf;
import static org.hamcrest.Matchers.equalTo;
import static org.hamcrest.Matchers.hasItems;
import static org.hamcrest.Matchers.hasProperty;
import static org.hamcrest.Matchers.hasSize;
import static org.hamcrest.Matchers.notNullValue;
import static org.hamcrest.Matchers.nullValue;

...

    Mockito.verify(mock).doSomething(
        argThat(
            allOf(
                hasItems(equalTo("a")),
                hasItems(equalTo("b"))
            )
        )
    );

Answer 2

正如 Jeff Bowman 已经指出的那样，问题在于编译器不知道您尝试调用的 4 个 contains 方法中的哪一个。

你正在构建的列表

Arrays.asList(Matchers.equalTo("a"), Matchers.equalTo("b"))

是类型

List<Matcher<String>>

但是您要调用的 contains 方法 (<E> Matcher<Iterable<? extends E>> contains(List<Matcher<? super E>> itemMatchers)) 需要一个类型

List<Matcher<? super String>>

作为参数。由于您的列表类型与预期的不匹配，编译器实际上认为您正在尝试调用

<E> Matcher<Iterable<? extends E>> contains(E... items)

解决方案：给编译器它想要的东西。创建一个List<Matcher<? super String>> 而不是List<Matcher<String>>：

        List<Matcher<? super String>> matchersList = new ArrayList<>();
        matchersList.add(Matchers.equalTo("a"));
        matchersList.add(Matchers.equalTo("b"));

        // no illegal cast anymore
        Mockito.verify(mock).doSomething(
            (Collection<String>) argThat(Matchers.contains(matchersList)));

编辑：

从他的评论中添加 Jeff Bowman 的内联解决方案，这样可以使用Arrays.asList，如问题中所述：

Mockito.verify(mock).doSomething(
   (Collection<String>) argThat(
        Matchers.contains(
            Arrays.<Matcher<? super String>> asList(
                Matchers.equalTo("a"), Matchers.equalTo("b")
            )
        )
    )
);

Answer 3

我相信这是由于 Hamcrest 中令人讨厌的模棱两可，它的 Matchers class：

1. <E> Matcher<Iterable<? extends E>> contains(E... items)
2. <E> Matcher<Iterable<? extends E>> contains(Matcher<? super E> itemMatcher)
3. <E> Matcher<Iterable<? extends E>> contains(Matcher<? super E>... itemMatchers)
4. <E> Matcher<Iterable<? extends E>> contains(List<Matcher<? super E>> itemMatchers)

没错，取决于你传递 Hamcrest 一个项目、一个匹配器、一个可变参数数组还是一个匹配器列表，你会得到不同的行为。因为 Java 不反对 Hamcrest 匹配器的匹配列表，所以一个语句有很多机会匹配多个这些重载，并且它们之间的选择是由 JLS 18.5.4 中令人眼花缭乱的类型代数确定的最具体的重载。

我认为您打算上面的第 4 项 — 传递 contains 的列表 Matcher<E> (Matcher<String>) 并返回 Matcher<Iterable<? extends String>> — 但编译器将其视为 #1 — 传递 contains E (List<Matcher<String>>) 类型的 值 并返回 Matcher<Iterable<? extends List<Matcher<String>>>>。

有几个解决方法，我还没有测试过：

• 将Matcher 提取到一个变量中，您可以使用像contains 这样的Hamcrest 匹配器，但不能使用像argThat 这样的Mockito 匹配器：

  Matcher<Iterable<String>> matchesAAndB = Matchers.contains(
      Arrays.asList(Matchers.equalTo("a"), Matchers.equalTo("b")));
  Mockito.verify(mock).doSomething((Collection<String>)argThat(matchesAAndB));
  

• 明确选择 E：

  Mockito.verify(mock).doSomething((Collection<String>)argThat(
      Matchers.<String>contains(Arrays.asList(
          Matchers.equalTo("a"), Matchers.equalTo("b")))));
  

Answer 4

最好的方法是使用标准的assertThat 方法（来自Hamcrest 或JUnit），它最适合任何Hamcrest 匹配器。使用 JMockit，您可以这样做：

@Test
public void usingJMockit(@Mocked final Collaborator mock) {
    mock.doSomething(asList("a", "b"));

    new Verifications() {{
        List<String> values;
        mock.doSomething(values = withCapture());

        // Now check the list of captured values using JUnit/Hamcrest:
        assertThat(values, contains("a", "b"));

        // Alternatively, could have used Asser4J, FEST Assert, etc.
    }};
}