Update `user_roles`: Return roles for user by resource.

3 files changed, 92 insertions, 49 deletions

diff --git a/tests/unit/auth/fixtures/group_fixtures.py b/tests/unit/auth/fixtures/group_fixtures.py
index 66edf5e..93133d2 100644
--- a/tests/unit/auth/fixtures/group_fixtures.py
+++ b/tests/unit/auth/fixtures/group_fixtures.py
@@ -21,11 +21,14 @@ GROUP_CATEGORY = ResourceCategory(
     "A group resource.")
 GROUPS_AS_RESOURCES = tuple({
     "group_id": str(group.group_id),
-    "resource_id": str(uuid.uuid4()),
+    "resource_id": res_id,
     "resource_name": group.group_name,
     "category_id": str(GROUP_CATEGORY.resource_category_id),
     "public": "1"
-} for group in TEST_GROUPS)
+} for res_id, group in zip(
+    ("38d1807d-105f-44a7-8327-7e2d973b6d8d",
+     "89458ef6-e090-4b53-8c2c-59eaf2785f11"),
+    TEST_GROUPS))
 
 TEST_RESOURCES_GROUP_01 = (
     Resource(uuid.UUID("26ad1668-29f5-439d-b905-84d551f85955"),
diff --git a/tests/unit/auth/fixtures/migration_fixtures.py b/tests/unit/auth/fixtures/migration_fixtures.py
index 16a31cd..954b5a9 100644
--- a/tests/unit/auth/fixtures/migration_fixtures.py
+++ b/tests/unit/auth/fixtures/migration_fixtures.py
@@ -40,6 +40,11 @@ def conn_after_auth_migrations(backend, auth_testdb_path, all_migrations): # pyl
     """Run all migrations and return a connection to the database after"""
     apply_migrations(backend, all_migrations)
     with db.connection(auth_testdb_path) as conn:
+        with db.cursor(conn) as cursor:
+            cursor.execute(
+                "UPDATE resources "
+                "SET resource_id='0248b289-b277-4eaa-8c94-88a434d14b6e' "
+                "WHERE resource_name='GeneNetwork System'")
         yield conn
 
     rollback_migrations(backend, all_migrations)
diff --git a/tests/unit/auth/test_roles.py b/tests/unit/auth/test_roles.py
index 227cb9e..6c8dbae 100644
--- a/tests/unit/auth/test_roles.py
+++ b/tests/unit/auth/test_roles.py
@@ -64,53 +64,88 @@ def test_create_role_raises_exception_for_unauthorised_users(# pylint: disable=[
 @pytest.mark.parametrize(
     "user,expected",
     (zip(TEST_USERS,
-         ((Role(
-             role_id=uuid.UUID('a0e67630-d502-4b9f-b23f-6805d0f30e30'),
-             role_name='group-leader', user_editable=False,
-             privileges=(
-                 Privilege(privilege_id='group:resource:create-resource',
-                           privilege_description='Create a resource object'),
-                 Privilege(privilege_id='group:resource:delete-resource',
-                           privilege_description='Delete a resource'),
-                 Privilege(privilege_id='group:resource:edit-resource',
-                           privilege_description='edit/update a resource'),
-                 Privilege(
-                     privilege_id='group:resource:view-resource',
-                     privilege_description=(
-                         'view a resource and use it in computations')),
-                 Privilege(privilege_id='group:role:create-role',
-                           privilege_description='Create a new role'),
-                 Privilege(privilege_id='group:role:delete-role',
-                           privilege_description='Delete an existing role'),
-                 Privilege(privilege_id='group:role:edit-role',
-                           privilege_description='edit/update an existing role'),
-                 Privilege(privilege_id='group:user:add-group-member',
-                           privilege_description='Add a user to a group'),
-                 Privilege(privilege_id='group:user:assign-role',
-                           privilege_description=(
-                               'Assign a role to an existing user')),
-                 Privilege(privilege_id='group:user:remove-group-member',
-                           privilege_description='Remove a user from a group'),
-                 Privilege(privilege_id='system:group:delete-group',
-                           privilege_description='Delete a group'),
-                 Privilege(privilege_id='system:group:edit-group',
-                           privilege_description='Edit the details of a group'),
-                 Privilege(
-                     privilege_id='system:group:transfer-group-leader',
-                     privilege_description=(
-                         'Transfer leadership of the group to some other '
-                         'member')),
-                 Privilege(privilege_id='system:group:view-group',
-                           privilege_description='View the details of a group'),
-                 Privilege(privilege_id='system:user:list',
-                           privilege_description='List users in the system'))),
-           Role(
-               role_id=uuid.UUID("ade7e6b0-ba9c-4b51-87d0-2af7fe39a347"),
-               role_name="group-creator", user_editable=False,
-               privileges=(
-                   Privilege(privilege_id='system:group:create-group',
-                             privilege_description = "Create a group"),))),
-          tuple(), tuple(), tuple()))))
+         (({"resource_id": uuid.UUID("38d1807d-105f-44a7-8327-7e2d973b6d8d"),
+            "user_id": uuid.UUID("ecb52977-3004-469e-9428-2a1856725c7f"),
+            "roles": (Role(
+                role_id=uuid.UUID('a0e67630-d502-4b9f-b23f-6805d0f30e30'),
+                role_name='group-leader', user_editable=False,
+                privileges=(
+                    Privilege(
+                        privilege_id='group:resource:create-resource',
+                        privilege_description='Create a resource object'),
+                    Privilege(
+                        privilege_id='group:resource:delete-resource',
+                        privilege_description='Delete a resource'),
+                    Privilege(
+                        privilege_id='group:resource:edit-resource',
+                        privilege_description='edit/update a resource'),
+                    Privilege(
+                        privilege_id='group:resource:view-resource',
+                        privilege_description=(
+                            'view a resource and use it in computations')),
+                    Privilege(
+                        privilege_id='group:role:create-role',
+                        privilege_description='Create a new role'),
+                    Privilege(
+                        privilege_id='group:role:delete-role',
+                        privilege_description='Delete an existing role'),
+                    Privilege(
+                        privilege_id='group:role:edit-role',
+                        privilege_description='edit/update an existing role'),
+                    Privilege(
+                        privilege_id='group:user:add-group-member',
+                        privilege_description='Add a user to a group'),
+                    Privilege(
+                        privilege_id='group:user:assign-role',
+                        privilege_description=(
+                            'Assign a role to an existing user')),
+                    Privilege(
+                        privilege_id='group:user:remove-group-member',
+                        privilege_description='Remove a user from a group'),
+                    Privilege(
+                        privilege_id='system:group:delete-group',
+                        privilege_description='Delete a group'),
+                    Privilege(
+                        privilege_id='system:group:edit-group',
+                        privilege_description='Edit the details of a group'),
+                    Privilege(
+                        privilege_id='system:group:transfer-group-leader',
+                        privilege_description=(
+                            'Transfer leadership of the group to some other '
+                            'member')),
+                    Privilege(
+                        privilege_id='system:group:view-group',
+                        privilege_description='View the details of a group'),
+                    Privilege(
+                        privilege_id='system:user:list',
+                        privilege_description='List users in the system'))),)
+            },
+           {
+               "resource_id": uuid.UUID("0248b289-b277-4eaa-8c94-88a434d14b6e"),
+               "user_id": uuid.UUID("ecb52977-3004-469e-9428-2a1856725c7f"),
+               "roles": (Role(
+                   role_id=uuid.UUID("ade7e6b0-ba9c-4b51-87d0-2af7fe39a347"),
+                   role_name="group-creator",
+                   user_editable=False,
+                   privileges=(
+                       Privilege(
+                           privilege_id="system:group:create-group",
+                           privilege_description="Create a group"),)),)}),
+          ({"resource_id": uuid.UUID("2130aec0-fefd-434d-92fd-9ca342348b2d"),
+            "user_id": uuid.UUID("21351b66-8aad-475b-84ac-53ce528451e3"),
+            "roles": (Role(
+                role_id=uuid.UUID('89819f84-6346-488b-8955-86062e9eedb7'),
+                role_name='resource_editor',
+                user_editable=True,
+                privileges=(
+                    Privilege(
+                        privilege_id='group:resource:edit-resource',
+                        privilege_description='edit/update a resource'),
+                    Privilege(
+                        privilege_id='group:resource:view-resource',
+                        privilege_description='view a resource and use it in computations'))),)},),
+          tuple(),
+          tuple()))))
 def test_user_roles(fxtr_group_user_roles, user, expected):
     """
     GIVEN: an authenticated user