[GitHub] [rocketmq] caigy commented on pull request #4637: [ISSUE #4585] PlainPermissionManagerTest add unit test

[GitBox <gi...@apache.org>]

caigy commented on PR #4637:
URL: https://github.com/apache/rocketmq/pull/4637#issuecomment-1217451742

   @Oliverwqcwrw Please fix test errors:
   
   ```
   Tests run: 25, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 27.275 sec <<< FAILURE! - in org.apache.rocketmq.client.consumer.DefaultLitePullConsumerTest
   unnecessary Mockito stubbings(org.apache.rocketmq.client.consumer.DefaultLitePullConsumerTest)  Time elapsed: 0.039 sec  <<< ERROR!
   org.mockito.exceptions.misusing.UnnecessaryStubbingException: 
   Unnecessary stubbings detected in test class: DefaultLitePullConsumerTest
   Clean & maintainable test code requires zero unnecessary code.
   Following stubbings are unnecessary (click to navigate to relevant line of code):
     1. -> at org.apache.rocketmq.client.consumer.DefaultLitePullConsumerTest.testPullTaskImpl_ProcessQueueNull(DefaultLitePullConsumerTest.java:356)
   Please remove unnecessary stubbings or use 'lenient' strictness. More info: javadoc for UnnecessaryStubbingException class.
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org