-
Notifications
You must be signed in to change notification settings - Fork 112
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade tests to JUnit 5 #939
Comments
@0crat in |
@krzyk Merged, please close |
@paulodamaso thanks |
@ypshenychka/z please review this job completed by @paulodamaso/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed |
The job #939 is now out of scope |
@0crat quality good |
@ypshenychka The project doesn't have enough funds, can't make a payment |
Order was finished, quality is "good": +35 point(s) just awarded to @paulodamaso/z |
Quality review completed: +8 point(s) just awarded to @ypshenychka/z |
Tests should be upgrade JUnit 4.x -> JUnit 5.
All old constructs should be replaced with new ones (e.g. ExpectedException should become assertThrows).
And remove
DefaultPackage
PMD rule, as all the JUnit 5 test methods should have default scope, and it is generally good to limit scopes.The text was updated successfully, but these errors were encountered: