FrankChen021 opened a new pull request, #19908: URL: https://github.com/apache/druid/pull/19908
Part of #13948. ### Description Migrates the remaining JUnit 4 tests under `extensions-contrib` to JUnit 5 in one batch. The migration covers 31 contrib modules and removes JUnit 4, JUnitParams, Vintage, MigrationSupport, and Hamcrest usage from their Java test sources and module POMs where the dependency is no longer required. Four modules temporarily retain a direct test-scoped Hamcrest dependency because the current `sql` test fixture `BaseCalciteQueryTest` exposes `Matcher<Throwable>` in inherited method signatures. Their source contains no Hamcrest usage; those dependency declarations can be removed after the coordinated SQL fixture migration lands. This follows the earlier migration PRs #19875, #19876, #19877, #19878, #19879, #19880, #19881, and #19882. ### Validation - All migrated module tests passed. - Checkstyle passed for all 31 migrated modules. - SpotBugs passed for all 31 migrated modules. - Exact source audits found no JUnit 4, JUnitParams, or `org.hamcrest` imports in the migrated scope. - `git diff --check` passed. This PR has: - [x] been self-reviewed. - [x] migrated existing tests without changing production behavior. -- 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]
