Skip to content
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

SetSecurityManager will be removed in a future update #815

Open
Bukama opened this issue Apr 20, 2024 · 1 comment
Open

SetSecurityManager will be removed in a future update #815

Bukama opened this issue Apr 20, 2024 · 1 comment
Labels

Comments

@Bukama
Copy link
Member

Bukama commented Apr 20, 2024

When I run our build locally with JDK 21 I get the following warnings about the SecurityManager of the @SetEnvironmentVariable (last line highlighted)

WARNING: A terminally deprecated method in java.lang.System has been called
WARNING: System::setSecurityManager has been called by org.junitpioneer.jupiter.EnvironmentVariableUtilsTests$With_SecurityManager (file:/D:/Github/junit-pioneer/build/classes/java/test/)
WARNING: Please consider reporting this to the maintainers of org.junitpioneer.jupiter.EnvironmentVariableUtilsTests$With_SecurityManager
WARNING: System::setSecurityManager will be removed in a future release

Seems we have to rethink our current approach.

@beatngu13
Copy link
Member

I like the approach by system stubs mentioned here:

#509 (comment)

This requires Mockito, but we could pull that in the same way as we use Jackson, i.e., as an optional runtime dependency / Gralde feature variant.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants