darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-08-14 23:08:07 -04:00
Code Issues Packages Projects Releases Wiki Activity

Reorder and label test cases

Browse Source 
This commit takes the existing test cases, removes duplicates, and tries
to add a human-readable comment explaining what each one is testing.
This commit is contained in:
Chris Hill-Scott
2022-06-06 15:07:53 +01:00
parent ea9c7e6102
commit 30eebf3586
1 changed files with 19 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
tests/app/utils/test_user.py
View File

@@ -6,15 +6,25 @@ from app.utils.user import user_has_permissions
@pytest.mark.parametrize('permissions', (
pytest.param(
['send_messages'],
marks=pytest.mark.xfail(raises=Forbidden),
),
['manage_service'],
['send_messages', 'manage_service'],
['manage_templates', 'manage_service'],
['manage_service', 'manage_templates'],
[],
pytest.param([
# Route has a permission which the user doesn’t have
'send_messages'
], marks=pytest.mark.xfail(raises=Forbidden)),
[
# Route has one of the permissions which the user has
'manage_service'
],
[
# Route has more than one of the permissions which the user has
'manage_templates', 'manage_service'
],
[
# Route has one of the permissions which the user has, and one they do not
'manage_service', 'send_messages',
],
[
# Route has no specific permissions required
],
))
def test_permissions(
client_request,