FrankChen021 opened a new pull request, #19915:
URL: https://github.com/apache/druid/pull/19915

   Part of [#13948](https://github.com/apache/druid/issues/13948).
   
   ## Summary
   
   This PR migrates the second independent `extensions-contrib` batch from 
JUnit 4 to JUnit 5:
   
   - `moving-average-query`
   - `aliyun-oss-extensions`
   - `rabbit-stream-indexing-service`
   - `opentsdb-emitter`
   
   The module POMs remove JUnit 4 and Hamcrest test dependencies, and the test 
sources use Jupiter annotations, assertions, and parameterization. 
`MovingAverageQueryTest` now owns its small no-op scheduler fixture instead of 
loading the JUnit-4-based `QueryStackTests` class.
   
   This branch is based directly on `master`; it does not stack on 
[#19908](https://github.com/apache/druid/pull/19908) and contains no shared 
SQL/processing fixture changes. It can be merged independently. The shared 
fixtures are intentionally owned only by #19908.
   
   Related migration PRs: [#19908](https://github.com/apache/druid/pull/19908), 
[#19875](https://github.com/apache/druid/pull/19875), 
[#19876](https://github.com/apache/druid/pull/19876), 
[#19877](https://github.com/apache/druid/pull/19877), 
[#19878](https://github.com/apache/druid/pull/19878), 
[#19879](https://github.com/apache/druid/pull/19879), 
[#19880](https://github.com/apache/druid/pull/19880), 
[#19881](https://github.com/apache/druid/pull/19881), and 
[#19882](https://github.com/apache/druid/pull/19882).
   
   ## Validation
   
   - `test-compile` passed for all four modules and their required reactor 
dependencies.
   - Focused tests passed: moving-average (65), Aliyun OSS (68), Rabbit stream 
(39), and OpenTSDB (11).
   - Checkstyle passed with 0 violations for all four modules.
   - SpotBugs passed with 0 bugs/errors for all four modules.
   - No JUnit 4 or Hamcrest source imports or direct POM dependencies remain in 
these modules.
   - `git diff --check` passed.
   


-- 
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]

Reply via email to