[GitHub] [kafka] ijuma commented on a change in pull request #11074: KAFKA-13101: Replace EasyMock and PowerMock with Mockito for RestServerTest

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

ijuma commented on a change in pull request #11074:
URL: https://github.com/apache/kafka/pull/11074#discussion_r702994755



##########
File path: connect/runtime/src/test/java/org/apache/kafka/connect/runtime/rest/RestServerTest.java
##########
@@ -61,18 +56,23 @@
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertNull;
 import static org.junit.Assert.assertTrue;
+import static org.mockito.Mockito.doReturn;
+import static org.mockito.Mockito.mock;
 
-@RunWith(PowerMockRunner.class)
-@PowerMockIgnore({"javax.net.ssl.*", "javax.security.*", "javax.crypto.*"})
 public class RestServerTest {
-    @MockStrict
+    
     private Herder herder;
-    @MockStrict
     private Plugins plugins;
     private RestServer server;
 
     protected static final String KAFKA_CLUSTER_ID = "Xbafgnagvar";
 
+    @Before
+    public void tearUp() {

Review comment:
       This method is usually called `setUp()`.




-- 
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