RANGER-4848: Migrate Ranger Junit tests to junit5 - #768
Conversation
c36a3fe to
a01c16b
Compare
vyommani
left a comment
There was a problem hiding this comment.
Overall, the changes look good to me. That said, I notice quite a few test modifications that aren't directly tied to the JUnit 4 to JUnit 5 migration. I think these additional test improvements are substantial enough to warrant their own separate PR. I'm not entirely sure how much effort it would take to split them out while keeping the migration changes intact in this PR.
Currently I am trying to keep the previous junit 5 version or lower(if needed in any module) and same maven-surefire-plugin version, trying to remove reference of junit 4 as well; |
a01c16b to
f6be842
Compare
f6be842 to
3ac457d
Compare
kishorgollapalliwar
left a comment
There was a problem hiding this comment.
Look good to me. Please make sure to have enough DEV testing before merging the change.
What changes were proposed in this pull request?
Ranger junit test code migration from junit 4 to junit 5; some of the the test code were already moved to junit 5 via RANGER-4559 and RANGER-4730
How was this patch tested?
mvn clean install