diff options
| author | Frederick Muriuki Muriithi | 2023-04-25 09:42:36 +0300 |
|---|---|---|
| committer | Frederick Muriuki Muriithi | 2023-04-25 09:42:36 +0300 |
| commit | 8471ed1187a8abc5e28207776c5f49a59ba24b92 (patch) | |
| tree | e5df42404938c078313b5b039ab0269b437bb49f /tests/unit/auth/test_roles.py | |
| parent | 3e2198e39bc229553d118f367fbd2f9932a9a76b (diff) | |
| download | genenetwork3-8471ed1187a8abc5e28207776c5f49a59ba24b92.tar.gz | |
auth: Roles: Check for editability
Some roles should not be user-editable, and as such, we need to check before
allowing any edits on such roles. This commit makes that possible.
Diffstat (limited to 'tests/unit/auth/test_roles.py')
| -rw-r--r-- | tests/unit/auth/test_roles.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/tests/unit/auth/test_roles.py b/tests/unit/auth/test_roles.py index 0a3ba19..02fd9f7 100644 --- a/tests/unit/auth/test_roles.py +++ b/tests/unit/auth/test_roles.py @@ -27,7 +27,7 @@ PRIVILEGES = ( @pytest.mark.parametrize( "user,expected", tuple(zip(conftest.TEST_USERS[0:1], ( Role(uuid.UUID("d32611e3-07fc-4564-b56c-786c6db6de2b"), "a_test_role", - PRIVILEGES),)))) + True, PRIVILEGES),)))) def test_create_role(# pylint: disable=[too-many-arguments] fxtr_app, auth_testdb_path, mocker, fxtr_users, user, expected):# pylint: disable=[unused-argument] """ @@ -68,7 +68,7 @@ def test_create_role_raises_exception_for_unauthorised_users(# pylint: disable=[ (zip(TEST_USERS, ((Role( role_id=uuid.UUID('a0e67630-d502-4b9f-b23f-6805d0f30e30'), - role_name='group-leader', + role_name='group-leader', user_editable=False, privileges=( Privilege(privilege_id='group:resource:create-resource', privilege_description='Create a resource object'), @@ -108,7 +108,7 @@ def test_create_role_raises_exception_for_unauthorised_users(# pylint: disable=[ privilege_description='List users in the system'))), Role( role_id=uuid.UUID("ade7e6b0-ba9c-4b51-87d0-2af7fe39a347"), - role_name="group-creator", + role_name="group-creator", user_editable=False, privileges=( Privilege(privilege_id='system:group:create-group', privilege_description = "Create a group"),))), |