greenlight
greenlight copied to clipboard
ManageRoles Permission
Description
- Adding check for ManageRoles permission before accessing roles endpoints
- Added rspec tests to check for forbidden status return
- Removes the Roles tab from AdminNavSideBar if current user doesn't have permission
Testing Steps
Screenshots (if appropriate):
SonarCloud Quality Gate failed.
14 Bugs
0 Vulnerabilities
0 Security Hotspots
80 Code Smells
No Coverage information
1.6% Duplication
If possible, let us try to be consistent with the spec template introduced by @scouillard ,
@KH-Amir-TN This is following the same template. The only difference is that it is using the context to remove the permission instead since most of the tests in the file require the permission. We agreed that it what we will do in this case
If possible, let us try to be consistent with the spec template introduced by @scouillard ,
@KH-Amir-TN This is following the same template. The only difference is that it is using the context to remove the permission instead since most of the tests in the file require the permission. We agreed that it what we will do in this case
Gotcha, My bad, I didn't pay enough attention 😅