d8tltanc commented on a change in pull request #9485: URL: https://github.com/apache/kafka/pull/9485#discussion_r534983615
########## File path: core/src/test/scala/unit/kafka/security/authorizer/AclAuthorizerTest.scala ########## @@ -82,16 +86,17 @@ class AclAuthorizerTest extends ZooKeeperTestHarness { override def setUp(): Unit = { super.setUp() + val authorizers = Seq(aclAuthorizer, aclAuthorizer2, interfaceDefaultAuthorizer.authorizer) Review comment: Move the interface default test to a new class. Also, created a util class for code sharing. commit 6c550fd04a0c1912e669bf18d60dee27dd03e53c ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org