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

AbstractDSpaceIntegrationTest: consider removing or replacing method System.setSecurityManager #9592

Open
saschaszott opened this issue May 16, 2024 · 1 comment
Labels
bug code task Code cleanup task help wanted Needs a volunteer to claim to move forward testing framework Related specifically to Unit or Integration Tests

Comments

@saschaszott
Copy link
Contributor

Bug Description

When running JUnit integration tests on the main branch the following (red) message appears in IntelliJ:

WARNING: A terminally deprecated method in java.lang.System has been called
WARNING: System::setSecurityManager has been called by org.dspace.AbstractDSpaceIntegrationTest (file:/local-path-to-dspace-clone/dspace-api/target/test-classes/)
WARNING: Please consider reporting this to the maintainers of org.dspace.AbstractDSpaceIntegrationTest
WARNING: System::setSecurityManager will be removed in a future release

It is caused by System.setSecurityManager() in AbstractDSpaceIntegrationTest:

System.setSecurityManager(new NoExitSecurityManager());

@saschaszott saschaszott added bug needs triage New issue needs triage and/or scheduling labels May 16, 2024
@tdonohue tdonohue added code task Code cleanup task and removed needs triage New issue needs triage and/or scheduling labels May 16, 2024
@tdonohue
Copy link
Member

I've seen this as well. It's just a warning at this point, and I'm not sure myself how to solve the issue (removing these calls causes tests to fail). But, it needs a volunteer to investigate & find a solution.

@tdonohue tdonohue added help wanted Needs a volunteer to claim to move forward testing framework Related specifically to Unit or Integration Tests labels May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug code task Code cleanup task help wanted Needs a volunteer to claim to move forward testing framework Related specifically to Unit or Integration Tests
Projects
Status: 📋 To Do
Development

No branches or pull requests

2 participants