Rancho-7 commented on code in PR #19968:
URL: https://github.com/apache/kafka/pull/19968#discussion_r2154684232


##########
clients/clients-integration-tests/src/test/java/org/apache/kafka/clients/security/GroupAuthorizerIntegrationTest.java:
##########
@@ -255,15 +280,15 @@ public void 
testUnauthorizedProduceAndConsume(ClusterInstance clusterInstance, G
 
     @ClusterTest
     public void 
testClassicConsumeUnsubscribeWithoutGroupPermission(ClusterInstance 
clusterInstance) throws ExecutionException, InterruptedException {
-        testConsumeUnsubscribeWithGroupPermission(clusterInstance, 
GroupProtocol.CLASSIC);
+        testConsumeUnsubscribeWithoutGroupPermission(clusterInstance, 
GroupProtocol.CLASSIC);
     }
 
     @ClusterTest
     public void 
testAsyncConsumeUnsubscribeWithoutGroupPermission(ClusterInstance 
clusterInstance) throws ExecutionException, InterruptedException {
-        testConsumeUnsubscribeWithGroupPermission(clusterInstance, 
GroupProtocol.CONSUMER);
+        testConsumeUnsubscribeWithoutGroupPermission(clusterInstance, 
GroupProtocol.CONSUMER);
     }
 
-    private void testConsumeUnsubscribeWithGroupPermission(ClusterInstance 
clusterInstance, GroupProtocol groupProtocol) throws InterruptedException, 
ExecutionException {
+    private void testConsumeUnsubscribeWithoutGroupPermission(ClusterInstance 
clusterInstance, GroupProtocol groupProtocol) throws InterruptedException, 
ExecutionException {

Review Comment:
   Sounds good,I will keep both.



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

Reply via email to