gitedmond opened a new pull request, #23745: URL: https://github.com/apache/kafka/pull/23745
Remove the explicit `testImplementation` dependency on `:test-common:test-common-util` from `:tools`. Its former compile-time use was the `@Flaky` annotation removed by #23531. The module remains available through the shared `testRuntimeOnly runtimeTestLibs`, preserving the JUnit runtime filters. This carries forward the dependency cleanup from #23466 following Viktor's review comment. The diff is one deleted line in `build.gradle`. Validation on Java 17: - Full `:tools:test`: 1,273 tests passed, zero failures or skips, including all 16 `MetadataQuorumCommandTest` invocations and 19 `JmxToolTest` tests. - `:tools:compileTestJava`, `:tools:checkstyleTest`, and `:tools:spotlessCheck` passed. - The resolved `testRuntimeClasspath` still includes `test-common-util`. - `git diff --check` passed. The full suite ran with one test worker, a 1 GiB test heap, and a temporary loopback hostname mapping for the execution environment's RMI networking. Without that mapping, `JmxToolTest` fails with `Network is unreachable` on unchanged trunk as well. This contribution is licensed under the project's open source license. Generated-by: OpenAI Codex (GPT-6) -- 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]
