Closed vuriaval closed 3 weeks ago
The changes involve migrating the test classes from JUnit 4 to JUnit 5 (JUnit Jupiter) across multiple files. This includes updating import statements, annotations, and assertion methods to their JUnit 5 equivalents. The updates aim to leverage the new features and improvements provided by JUnit 5.
File(s) | Change Summary |
---|---|
src/test/java/ai/elimu/dao/... |
Replaced JUnit 4 annotations (@RunWith , @ContextConfiguration , @FixMethodOrder ) with JUnit 5 equivalents (@SpringJUnitConfig , @TestMethodOrder ). Updated assertions from Hamcrest matchers to JUnit 5 assertions (assertEquals , assertTrue , assertNull , etc.). |
src/test/java/ai/elimu/rest/v2/content/... |
Replaced JUnit 4 annotations (@Test ) with JUnit 5 equivalents. Updated assertions from assertThat to JUnit 5 assertions (assertFalse , assertNotNull , etc.). |
src/test/java/ai/elimu/rest/v2/crowdsource/WordContributionRestControllerTest.java |
Replaced JUnit 4 annotations (@Test ) with JUnit 5 equivalents. Updated assertions from assertThat to assertFalse . |
Objective | Addressed | Explanation |
---|---|---|
Migrate from JUnit 4 to JUnit 5 (#1679) | ✅ |
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 15.26%. Comparing base (
9bf5510
) to head (aa87a08
). Report is 1 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Closes #1679
ScreenshotOnFailureStatement.java
andScreenshotOnFailureRule.java
, were removed.pom.xml
had some outdated dependencies that were bumped automatically, so they were kept.