sachinnn99 opened a new pull request, #15815: URL: https://github.com/apache/iceberg/pull/15815
### What Remove the explicit `flink-test-utils-junit` dependency from all three Flink version modules (v1.20, v2.0, v2.1) and add `exclude group: 'junit'` to `flink-connector-test-utils` to prevent JUnit4 from leaking onto the test classpath transitively. ### Why All Flink test code has already been migrated to JUnit5 in prior work (#13021, #10770), making the `flink-test-utils-junit` dependency vestigial. The `flink-connector-test-utils` dependency was missing a `junit` group exclusion, allowing JUnit4 to still reach the classpath transitively. ### Changes - **`flink/v1.20/build.gradle`**, **`flink/v2.0/build.gradle`**, **`flink/v2.1/build.gradle`**: Added `exclude group: 'junit'` to `connector-test-utils`, removed `flink-test-utils-junit` dependency - **`gradle/libs.versions.toml`**: Removed 3 unused `flink-test-utils-junit` library aliases ### Safety - Zero test files import JUnit4 APIs (`org.junit.Test`, `org.junit.Assume`, `@RunWith`) - `MiniClusterExtension` and other test utilities come from `flink-test-utils`, not `flink-test-utils-junit` - `flink-test-utils-junit` remains available transitively through `flink-test-utils` (compile scope) - Verified: `compileTestJava` passes for Flink 2.1 Fixes #12937 -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
