FrankChen021 opened a new pull request, #19980: URL: https://github.com/apache/druid/pull/19980
## Summary - Migrate Batch 4 processing query and aggregation tests to current JUnit 5 APIs and extensions. - Preserve shared Hamcrest matcher APIs, fixture factory behavior, and the existing processing JUnit 4/Vintage/Hamcrest dependencies. - Add the current `JupiterAssertions` and `ExpectThrows` adapters required by the exact migration base. ## Scope - 74 assigned Java files under `processing/src/test/java/org/apache/druid/query`, plus the two current test adapters. - Covers aggregation and the remaining processing query families. - Leaves `NestedDataTestUtils.java`, `GroupByQueryRunnerTestHelper.java`, `processing/pom.xml`, excluded filter tests, and expression tests unchanged. - No duplicate `org.apache.druid.testing.matchers` framework or CI automation added. Resolves #13948 ## Checks - `mvn -ntp -pl processing -am -DskipTests -Dweb.console.skip=true -T1C test-compile` — passed with Checkstyle, PMD, main/test forbidden-API scans, and compilation. - Focused aggregation/failure tests — 134 passed. - Focused GroupBy parameterized cases — 208 passed. - Focused Unnest GroupBy case — 1 passed. - Focused Timeseries parameterized cases — 52 passed. - Focused MultiValuedDimension parameterized cases — 32 passed. - `mvn -ntp -pl processing -am -DskipTests -Dweb.console.skip=true -T1C spotbugs:check` — passed with zero SpotBugs bugs/errors. -- 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]
