diff options
| author | Frederick Muriuki Muriithi | 2022-11-24 13:53:59 +0300 |
|---|---|---|
| committer | Frederick Muriuki Muriithi | 2022-11-24 13:53:59 +0300 |
| commit | 9810731b8432624c3817633ea0877dd80d0eea39 (patch) | |
| tree | e30a620e9d549c6df19c80e3e27f0ac59cd7cb6e /tests/unit/auth/test_privileges.py | |
| parent | 021b8dfcb99928b363e4546f626e3deb5793e392 (diff) | |
| download | genenetwork3-9810731b8432624c3817633ea0877dd80d0eea39.tar.gz | |
auth: Check for authentication and fix errors
* gn3/auth/authorisation/groups.py: base `MembershipError` on new
`AuthorisationError` base exception. Use new authentication checking
decorator.
* gn3/auth/authorisation/privileges.py: Change argument to User object rather
than UUID object
* gn3/auth/authorisation/roles.py: Use new authentication checking decorator.
* tests/unit/auth/test_groups.py: use `conftest.TEST_USER`
* tests/unit/auth/test_privileges.py: use `conftest.TEST_USER`
* tests/unit/auth/test_roles.py: use `conftest.TEST_USER`
Diffstat (limited to 'tests/unit/auth/test_privileges.py')
| -rw-r--r-- | tests/unit/auth/test_privileges.py | 16 |
1 files changed, 7 insertions, 9 deletions
diff --git a/tests/unit/auth/test_privileges.py b/tests/unit/auth/test_privileges.py index 2514f9d..3586e90 100644 --- a/tests/unit/auth/test_privileges.py +++ b/tests/unit/auth/test_privileges.py @@ -6,6 +6,8 @@ import pytest from gn3.auth import db from gn3.auth.authorisation.privileges import Privilege, user_privileges +from tests.unit.auth import conftest + SORT_KEY = lambda x: x.privilege_name PRIVILEGES = sorted( @@ -33,18 +35,14 @@ PRIVILEGES = sorted( @pytest.mark.unit_test @pytest.mark.parametrize( - "user_id,expected", ( - ("ecb52977-3004-469e-9428-2a1856725c7f", PRIVILEGES), - ("21351b66-8aad-475b-84ac-53ce528451e3", []), - ("ae9c6245-0966-41a5-9a5e-20885a96bea7", []), - ("9a0c7ce5-2f40-4e78-979e-bf3527a59579", []), - ("e614247d-84d2-491d-a048-f80b578216cb", []))) -def test_user_privileges(auth_testdb_path, test_users, user_id, expected):# pylint: disable=[unused-argument] + "user,expected", tuple(zip( + conftest.TEST_USERS, (PRIVILEGES, [], [], [], [])))) +def test_user_privileges(auth_testdb_path, test_users, user, expected):# pylint: disable=[unused-argument] """ - GIVEN: A user_id + GIVEN: A user WHEN: An attempt is made to fetch the user's privileges THEN: Ensure only """ with db.connection(auth_testdb_path) as conn: assert sorted( - user_privileges(conn, UUID(user_id)), key=SORT_KEY) == expected + user_privileges(conn, user), key=SORT_KEY) == expected |