TaiJuWu commented on code in PR #18555:
URL: https://github.com/apache/kafka/pull/18555#discussion_r1921759305


##########
core/src/test/scala/integration/kafka/server/GssapiAuthenticationTest.scala:
##########
@@ -110,8 +110,8 @@ class GssapiAuthenticationTest extends 
IntegrationTestHarness with SaslSetup {
    * are able to connect after the second re-login. Verifies that logout is 
performed only once
    * since duplicate logouts without successful login results in NPE from Java 
9 onwards.
    */
-  @ParameterizedTest
-  @ValueSource(strings = Array("kraft"))
+  @ParameterizedTest(name = 
TestInfoUtils.TestWithParameterizedQuorumAndGroupProtocolNames)
+  @MethodSource(Array("getTestQuorumAndGroupProtocolParametersAll"))
   def testLoginFailure(quorum: String): Unit = {

Review Comment:
   @kirktrue  All comments are address, thanks!



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

Reply via email to