-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Run QuarkusTestResource in the runtime ClassLoader #7897
Conversation
This is only for QuarkusTest, QuarkusUnitTest will need a slightly different solution if we decide it is a problem, as the lifecycles don't match. Fixes quarkusio#7880
a900c7d
to
ea52f11
Compare
@lkrzyzanek @aldettinger @andrejpetras any chance you can test this? |
@geoand how can I get build of this PR? Is there a SNAPSHOT maven build to try? |
@lkrzyzanek you will need to build the PR locally and use |
@geoand 1.2.1.Final -> OK
1.3.0.CR2, 1.3.0.Final -> Error
[PR] -> OK
|
@andrejpetras excellent, that's what we expected to see :) |
Tried PR and tests works for me as well now. |
Thanks for checking @lkrzyzanek! |
This is only for QuarkusTest, QuarkusUnitTest will need
a slightly different solution if we decide it is a problem, as
the lifecycles don't match.
Fixes #7880