首页 > 技术文章 > java.lang.IllegalStateException: 1 matchers expected, 5 recorded.

ld-mars 2019-12-19 16:17 原文

这是一个很神奇的错误。

常规的出错是因为在mock方法里,其中某一个或者几个参数使用了EasyMock.anyxx(),而其他的使用了具体的值。

java.lang.IllegalStateException: 1 matchers expected, 5 recorded.
This exception usually occurs when matchers are mixed with raw values when recording a method:
    foo(5, eq(6));    // wrong
You need to use no matcher at all or a matcher for every single param:
    foo(eq(5), eq(6));    // right
    foo(5, 6);    // also right
    at org.easymock.internal.ExpectedInvocation.createMissingMatchers(ExpectedInvocation.java:52)
    at org.easymock.internal.ExpectedInvocation.<init>(ExpectedInvocation.java:41)
    at org.easymock.internal.RecordState.invoke(RecordState.java:51)
    at org.easymock.internal.MockInvocationHandler.invoke(MockInvocationHandler.java:40)
    at org.easymock.internal.ObjectMethodsFilter.invoke(ObjectMethodsFilter.java:101)
    at org.easymock.internal.ClassProxyFactory$MockMethodInterceptor.intercept(ClassProxyFactory.java:97)
    at com.unit_test_easymock_mockito.database.MyDatabase$$EnhancerByCGLIB$$f0fe0b8a.updateBook(<generated>)
    at com.unit_test_easymock_mockito.zexception.BookDaoImplTest.updateBookTest(BookDaoImplTest.java:85)

但本例中出现的错误并不是这个原因,附上代码:

/**
  * 更新书本信息 单元测试
  */
@Test
public void updateBookTest() {
        
    Book book = new Book();
        
    myDatabase.updateBook(EasyMock.anyObject());
    EasyMock.expectLastCall();
        
    mockControl.replay();
        
    bookDaoImpl.updateBook(book);
        
    mockControl.verify();
}

可以看到,这里mock的myDatabase.updateBook(EasyMock.anyObject());只有一个参数,不存在一个为anyObject(),一个为具体值的情况。

并且,单独执行junit是正常通过的:

 

 但通过maven clean install打包时,就报错了。

我们来通过debug的方式,看看错误到底出在了什么地方。

 

 

 

 

 

 

 

 

 可以看到在这里报错了,错误信息也和之前报出的一致。

再仔细分析,发现是这一句代码导致的抛异常:

if (matchers.size() != invocation.getArguments().length) {
invocation.getArguments().length是调用方法的参数个数,那么matchers.size()又是什么呢,从哪里获取的呢?
继续分析代码:
public ExpectedInvocation(Invocation invocation, List<IArgumentMatcher> matchers) {
    this.invocation = invocation;
    this.matchers = createMissingMatchers(invocation, matchers);
}
createMissingMatchers方法是在ExpectedInvocation构造方法里调用的,再来看:
public Object invoke(Invocation invocation) {
    closeMethod();
    List<IArgumentMatcher> lastMatchers = LastControl.pullMatchers();
    lastInvocation = new ExpectedInvocation(invocation, lastMatchers);
    lastInvocationUsed = false;
    return emptyReturnValueFor(invocation.getMethod().getReturnType());
}

在invoke方法里,会将matchers传递给ExpectedInvocation,而matchers是通过LastControl.pullMatchers()获取的:

public static List<IArgumentMatcher> pullMatchers() {
    List<IArgumentMatcher> stack = threadToArgumentMatcherStack.get();
    if (stack == null) {
        return null;
    }
    threadToArgumentMatcherStack.remove();
    return new ArrayList<>(stack);
}

这里可以看出,matchers最终是从threadToArgumentMatcherStack里获取的,并且,在获取后,会及时的threadToArgumentMatcherStack.remove();

再来了解下threadToArgumentMatcherStack是什么以及matchers是什么时候放到threadToArgumentMatcherStack里的:

private static final ThreadLocal<List<IArgumentMatcher>> threadToArgumentMatcherStack = new ThreadLocal<>();

threadToArgumentMatcherStack是一个ThreadLocal

public static void reportMatcher(IArgumentMatcher matcher) {
    List<IArgumentMatcher> stack = threadToArgumentMatcherStack.get();
    if (stack == null) {
        stack = new ArrayList<>(5); // methods of more than 5 parameters are quite rare
        threadToArgumentMatcherStack.set(stack);
    }
    stack.add(matcher);
}

在reportMatcher方法里,会把matchers放到threadToArgumentMatcherStack里。

那么,reportMatcher方法又是在什么时候调用的呢?

 

 

 

 可以看出,在进行对参数的Mock的时候,anyxx(),都会调用,添加matcher。

那么问题来了,在pullMatchers()方法里,每次获取时都会及时进行清空,为什么出现“1 matchers expected, 5 recorded.”,明明只有一个参数,matchers却被添加了5次的情况?

思考分析一下,发现有一种可能性,就是调用了reportMatcher方法,但是没有调用pullMatchers()方法,这样,对于ThreadLocal类型的threadToArgumentMatcherStack,在同一个线程里,matchers会一直增加。

但对于EasyMock来说,Mock一个方法必然会调用pullMatchers()方法,那么是不是可能没有使用EasyMock来Mock方法,但却使用了EasyMock.anyObject()来Mock参数了呢?

结果果然如此:

@Test
public void queryBookByIdTest() {
        
    Integer id = 1;
        
    Mockito.when(bookDao.queryBookById(EasyMock.anyObject())).thenReturn(null);
        
    bookServiceImpl.queryBookById(id);
}

在这个单元测试里,使用了Mockito来mock方法,却同时使用了EasyMock.anyObject()来mock参数,这样就导致执行到真正EasyMock来mock方法的时候,出问题了,matchers和方法的参数对应不上,导致了问题出现。

最后还存在一个疑问:为什么单独执行单元测试不会报错,而执行clean install打包时就报出这个错呢?

个人推测执行单元测试时,是对各个单元测试方法单个执行的,即单元测试之间不会相互影响;而clean install打包时,所有的单元测试方法在同一个线程里具有相同的上下文,导致了问题出现。

所以说,在项目里写单元测试时,尽量只使用一种单元测试框架,混合使用多种单元测试框架,可能会造成很神奇的问题出现。

推荐阅读