-
Notifications
You must be signed in to change notification settings - Fork 162
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
NoClassDefFoundError with junit jupiter 5.9 #136
Comments
Thanks for the report. This needs to be fixed. However I am very busy at the moment - even on weekends. How urgent is it @jenschude? Or can you maybe provide a PR yourself? |
My main concern would be that implementing the interface would mean to stop support for junit < 5.9 |
Yes, that is not nice but I do not see another opportunity at the moment. So we will release a new version and document it properly on https://github.com/TNG/junit-dataprovider/wiki/Version-compatibility |
Opened a PR. |
Also to note here. The PR only fixes the the issue atm. It still uses an internal class of the jupiter engine. The InterceptionExecutableInvoker is also doesn't have an interface so it could be possible that in future versions it may break again. It may be worth to take a closer look at the other changes made with 5.9. The introduction of the ExecutableInvoker interface also added the invoker to the Context object. So it may be possible to refactor this part to not use the internal classes anymore. |
Thanks for the PR, should we merge and release it for now or should we immediately try to use the invoker from the context. @jenschude? |
I would merge it, to open up the upgrade path for junit 5.9. The investigation regarding the context invoker could be a future option for improvements. |
Thanks @jenschude, fixed with v2.10 which should appear shortly on Maven Central. |
Overview
With the release of junit 5.9 the ExecutableInvoker class was changed to an Interface. The class itself has been replaced with the DefaultExecutableInvoker class. Therefor the class can't be found any more.
See also release notes for https://junit.org/junit5/docs/current/release-notes/#release-notes-5.9.0
The text was updated successfully, but these errors were encountered: