search

ponder-lab / Imperative-DL-Study-Web-App

0 stars 0 forks source link

Add automated tests for roles #133

Closed mZneit closed 2 years ago

mZneit commented 2 years ago

Test if a certain role has the permission to access a certain page or directed to a 403 page for example. This wiki page has the list of all permissions for each role. https://github.com/ponder-lab/Imperative-DL-Study-Web-App/wiki/Roles

ZhongweiL commented 2 years ago

@mZneit I figured that to test the roles, we'll have to fill those roles into the test database and manually assign permissions to them, that wouldn't be too much different from testing the permissions directly. Do you think it would be a better idea to just test the specific permissions?

mZneit commented 2 years ago

Yes, I think it's better to just test specific permissions.

khatchad commented 2 years ago

@mZneit and I discussed this. This test would not be considered a unit test and thus would be quite involved. We don't feel that the benefits outweigh the effort required.