-
Notifications
You must be signed in to change notification settings - Fork 12
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
System.exit
behaviour will not be supported from JDK 18+
#37
Comments
Hello, is there any plan to fix this ? |
If you would like to create a PR then please do. I'll look at it in the mid to long term otherwise. |
I think the best thing to do is look at a System Stubs v3 for JDK 18+. One that's built on JDK 17 - perhaps keeping the JDK8 branch alive just in case. |
Indeed is one of my idea was moving to jdk 17 as most of framework have move on this version I have fork the project to perform the changes |
Sounds good @zordym. If you can put up a PR with your best attempt, even if it doesn't work 100%, I don't mind joining in with you on it to try to get it over the line. Keep me posted on this issue. |
For sure, I will do my best |
@zordym - let's also make sure to update https://github.com/webcompere/system-stubs/blob/main/History.md to include your contribution as well as having you on the usual GH statistics. We do open source because we care about having the tool and sharing that. We deserve the credit. |
As far as I understand it's expected behavior, don't I?
My env:
|
@vitalijr2 - yes. You're stubbing If enough people need a solution to this, we may have to look at a new way of intercepting it from Java 18 onwards. |
The Security Manager will be removed in later Java and this will stop the SystemExit mocking from working.
The text was updated successfully, but these errors were encountered: