clolov commented on PR #12818: URL: https://github.com/apache/kafka/pull/12818#issuecomment-1302324578
Three points for this pull request: * I have left `mock()` instead of `@Mock` where abstracting it wouldn't have proved easy. * I have used `lenient()` with some strict stubs because otherwise Mockito complains we do not use that code in some tests. I chose to not break the abstraction. * If I have deleted expectations that’s because Mockito said they are not used. -- 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: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
