-
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
ServletContextInitializer IT test #33018
Conversation
@gastaldi can you kick off this workflow I want to see if this IT fails or not in Native mode . I think this emulates my real world test case. |
Done, btw you can also check the workflow running in your own branch if you enable it 😉 |
Oh whoah how do I do that. a setting in my fork? |
Nevermind I think I found the setting! |
This comment has been minimized.
This comment has been minimized.
OK @stuartwdouglas something is different about how the unit tests run vs a real world app. If you run this real world app which does the exact same as the Integration test above fails and throws the RunTimeException. Reproducer: Run
|
I have confirmed that bug #28028 still exists and even though this IT is attempting to replicate it it seems to work but a real life scenario does not work. |
This comment has been minimized.
This comment has been minimized.
Status for workflow
|
@gsmet i think you fixed this bug! |
Test case for #28028 just to see if it fails in Native mode