[GitHub] [kafka] jeff303 commented on a change in pull request #11792: Replace EasyMock/PowerMock with Mockito in DistributedHerderTest

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

jeff303 commented on a change in pull request #11792:
URL: https://github.com/apache/kafka/pull/11792#discussion_r819904687



##########
File path: connect/runtime/src/test/java/org/apache/kafka/connect/runtime/distributed/DistributedHerderTest.java
##########
@@ -917,6 +955,7 @@ public void testDestroyConnector() throws Exception {
         PowerMock.verifyAll();
     }
 
+    @Disabled("Disabled until EasyMock->Mockito conversion is done")

Review comment:
       This is only temporary, as a way to keep all the source code in place while iterating on the file (since it's a large file).  I will be removing the annotation one-by-one as I fix each case.  Just wanted to validate the basic approach before going to all the trouble.




-- 
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-unsubscribe@kafka.apache.org

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