strangelookingnerd opened a new pull request, #309: URL: https://github.com/apache/commons-jcs/pull/309
In c05870d82501295890f6fcea48b9770d06983dc0 @garydgregory already began the transition towards JUnit5. This PR aims to finish that. The main points and changes are * Use JUnit5 annotations * Use JUnit5 `Assertions` and `Assumptions` * Remove `public` visibility from tests classes and methods (best-practise) * Remove `junit-vintage-engine` dependency as it is no longer required I validated that all tests succeeded before and after my changes and the overall number of tests did not change. Test coverage is also identical. There are some _oddly_ formatted classes - I tried my very best to keep the original formatting intact to minimize the changeset. -- 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]
