cruise-control
cruise-control copied to clipboard
Missing EasyMock.verify in unit tests
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.
I'll take this on.
Hey, is this issue still open?
Hey I would like to work on this. Can I get started