divijvaidya opened a new pull request, #12465: URL: https://github.com/apache/kafka/pull/12465
This PR build on top of https://github.com/apache/kafka/pull/11017. I have added the previous author's comment in this PR for attribution. I have also addressed the pending comments from @chia7712 in this PR. Notes to help the reviewer: 1. Mockito has `mockStatic` method which is equivalent to PowerMock's method. 2. When we run the tests using `@RunWith(MockitoJUnitRunner.StrictStubs.class)` Mockito performs a verify() for all stubs that are mentioned, hence, there is no need to explicitly verify the stubs (unless you want to verify the number of times etc.). Note that this does not work for static mocks. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
