adoroszlai opened a new pull request #2909: URL: https://github.com/apache/ozone/pull/2909
## What changes were proposed in this pull request? These unit tests are skipped currently, because they are annotated with JUnit5's `@Test`, which the JUnit4 runner does not know about: ``` hadoop-hdds/common/src/test/java/org/apache/hadoop/hdds/ratis/conf/TestRatisClientConfig.java hadoop-hdds/common/src/test/java/org/apache/hadoop/hdds/ratis/conf/TestRaftClientConfig.java hadoop-hdds/common/src/test/java/org/apache/hadoop/hdds/tracing/TestStringCodec.java ``` This PR adds JUnit5 test dependency in the affected module (`hdds-common`). It also tweaks `TestStringCodec`, which was broken (without noticing it, since it was not being run) by the most recent Jaeger update (HDDS-4980). https://issues.apache.org/jira/browse/HDDS-6094 ## How was this patch tested? Verified that previously missing tests were executed: https://github.com/adoroszlai/hadoop-ozone/runs/4484919834#step:4:651 https://github.com/adoroszlai/hadoop-ozone/runs/4484919834#step:4:679 Full CI: https://github.com/adoroszlai/hadoop-ozone/actions/runs/1563924309 -- 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]
