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

Add security integration tests #258

Closed
Tracked by #58
lezzago opened this issue Aug 5, 2021 · 2 comments
Closed
Tracked by #58

Add security integration tests #258

lezzago opened this issue Aug 5, 2021 · 2 comments
Assignees
Labels

Comments

@lezzago
Copy link
Member

lezzago commented Aug 5, 2021

Is your feature request related to a problem? Please describe.
Since the notification plugin is going to support integration with security, please add integration tests that test security permission scenarios.
This should include make sure if user has permission to access a resource or not and checking for filterby backend roles works (reference).

You can refer to the Alerting plugin to see how security integration tests were done:
Security tests for destinations
Security tests for monitors

@lezzago
Copy link
Member Author

lezzago commented Apr 29, 2022

Please note this is an enhancement on the existing integration tests where it tests the security scenarios with the plugin to ensure only users with the correct permissions can access certain resources or APIs.

@lezzago
Copy link
Member Author

lezzago commented May 19, 2022

These tests are to check basic security use cases like a user must get permission denied error when trying to access a Notification API that they do not have cluster permissions for. Including these tests will help reduce manual testing for security cases.

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

3 participants