-
Notifications
You must be signed in to change notification settings - Fork 2k
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
[Feature Request] Support JUnit 5 #17445
Comments
@dbwiddis I'm all for avoiding the PR churn you highlighted here. I honestly don't know the pros/cons of the solutions you listed. Would option 2 be something like creating a |
Or could we just add a JUnit5 dependency to |
I'll take a stab in my free time at what some of the current uses are and see if I can have a minimal JUnit5 Superclass to give at least some of the benefits. |
After initial research it may be as simple as just brining in JUnit5 dependency (thus getting the version) and using the vintage engine to run existing JUnit4 code/tests, with no additional tests needed. |
Went with this approach. See https://junit.org/junit5/docs/current/user-guide/#migrating-from-junit4 By adding the "vintage" engine, existing Junit 4 tests/code continue to run and allow a slow migration (or partial migration). |
Closing, see discussion on #17745 |
Is your feature request related to a problem? Please describe
Many plugins use JUnit5 for unit testing..
However, OpenSearch is still using JUnit 4, which is the version available in the version catalog:
https://github.com/opensearch-project/OpenSearch/blob/main/gradle/libs.versions.toml#L64
This results in a lot of version bump PR churn.
Describe the solution you'd like
Related component
Build
Describe alternatives you've considered
Additional context
Option 3 and alternative 2 are probably not easy, requiring upstream changes:
The text was updated successfully, but these errors were encountered: