FrankChen021 opened a new pull request, #20059: URL: https://github.com/apache/druid/pull/20059
## Summary - Migrate 71 owned server coordinator and compaction tests from JUnit 4 to JUnit 5. - Preserve class-wide parameterization, lifecycle behavior, timeouts, assumptions, execution counts, and assertion semantics. - Keep shared POMs and helpers unchanged; no JUnit 4/Vintage dependencies were removed. ## Validation - `mvn -pl server -am -DskipTests test-compile -Pskip-static-checks -Dweb.console.skip=true -T1C -ntp` — PASS (`BUILD SUCCESS`). - `mvn -pl server -am -DskipTests -Dweb.console.skip=true -T1C -ntp checkstyle:check spotbugs:check` — PASS; Checkstyle reported 0 violations and SpotBugs reported 0 bugs/errors. SpotBugs emitted the existing processing analysis warning for missing `org.apache.logging.log4j.MarkerManager`. - `mvn test -pl server -am -Dtest="org.apache.druid.server.coordinator.**,org.apache.druid.server.compaction.**" -Dsurefire.failIfNoSpecifiedTests=false -DforkCount=0 -Pskip-static-checks -Dweb.console.skip=true -ntp` — 823 tests executed, 0 failures, 24 errors, 1 skipped. The errors are Mockito inline Byte Buddy self-attach failures under local OpenJDK 25 (`AttachNotSupportedException`), not migration assertion failures. The forked focused-test mode could not start in the local sandbox because Surefire's fork channel failed with `SocketException: Operation not permitted` while binding its socket. Part of #13948 -- 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]
