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

Expose the ArtemisSecurityManager methods via TestUtils #53

Open
MaisiKoleni opened this issue Sep 18, 2020 · 0 comments
Open

Expose the ArtemisSecurityManager methods via TestUtils #53

MaisiKoleni opened this issue Sep 18, 2020 · 0 comments
Labels
code quality The quality of the source code needs to be improved [no change of functionality] enhancement New feature or request

Comments

@MaisiKoleni
Copy link
Collaborator

MaisiKoleni commented Sep 18, 2020

The ArtemisSecurityManager contains methods required to whitelist threads and manage other security issues, such as proposed in #52. We should use an official class with static method for that which delegates to the methods of the security manager.

It should also be considered to rename TestUtils to something that is more associated with Ares, such as an Ares class.

All of that would improve readability and finding as well as documenting the methods and features that Aresprovides besides the annotations.

@MaisiKoleni MaisiKoleni added enhancement New feature or request code quality The quality of the source code needs to be improved [no change of functionality] labels Sep 18, 2020
@MaisiKoleni MaisiKoleni self-assigned this Sep 18, 2020
@MaisiKoleni MaisiKoleni removed their assignment Oct 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code quality The quality of the source code needs to be improved [no change of functionality] enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant