good first issueproductivity
倉庫指標
- Star
- (3,036 star)
- PR 合併指標
- (PR 指標待抓取)
描述
In some unit tests, before resetting mocked objects or after mocked objects get used, EasyMock.verify does not get invoked on mocked objects. So, we should add it.
For example, in UserTaskManagerTest.java and KafkaCruiseControlServletEndpointTest.java. There could be more cases. So, I suggest thoroughly iterate all cases where EasyMock gets used.
It may require re-writing some unit tests since naively inserting verify before resetting mocks or at the end of unit tests fail in most cases.