chia7712 commented on code in PR #23931:
URL: https://github.com/apache/kafka/pull/23931#discussion_r4381590334
##########
clients/clients-integration-tests/src/test/java/org/apache/kafka/clients/admin/AdminClientTimeoutIntegrationTest.java:
##########
@@ -75,6 +86,125 @@ public void testCallInFlightTimeouts() throws Exception {
}
}
+ @ClusterTest
+ public void testDescribeConfigWithOptionTimeoutMs() {
+ try (Admin admin = createInvalidAdminClient()) {
Review Comment:
It seems those tests don't need the cluster instance, so would you mind
using `@Test` instead? `testCallInFlightTimeouts` and `testClose` still need
the cluster, so please inject `ClusterInstance` as a method parameter for those
two instead of via the constructor :)
--
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]
