FrankChen021 opened a new pull request, #20062: URL: https://github.com/apache/druid/pull/20062
## Summary - Migrate the scoped services, embedded-tests, server HTTP, Jetty, and server-initialization tests to JUnit 5. - Preserve class-wide parameterization in SecurityResourceFilterTest with @ParameterizedClass and @MethodSource. - Replace JUnit 4 rules, runners, lifecycle annotations, expected exceptions, and internal Hamcrest matchers with Jupiter equivalents. - Keep PR3-owned top-level initialization files, server/pom.xml, and shared POMs unchanged. ## Validation - mvn -ntp -DskipTests -pl server,services,embedded-tests -am test-compile -Pskip-static-checks -Dweb.console.skip=true -T1C — PASS. - Focused services tests — PASS: 153 + 56 + 13 tests across the migrated classes; CreateTablesTest rerun after final Checkstyle fix: 1 test. - Focused server tests — PASS: 245 tests, 0 failures/errors, 2 skips; JettyCertRenewTest had one automatic flake retry and then passed. - Focused embedded tests — PASS: 25 tests, 0 failures/errors. - mvn -ntp -DskipTests -pl server,services,embedded-tests -am checkstyle:check -Dweb.console.skip=true -T1C — PASS, 0 violations. - SpotBugs — PASS for server/services; embedded-tests completed with no main classes to analyze. - git diff --check and added-static-import audit — PASS. No POM dependency was removed: services still consumes JUnit 4 through shared druid-server test fixtures, and embedded-tests retains live Hamcrest consumers. 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]
