Small test style improvements

Author: dolfinus

tests/test_unit/test_groups/test_read_groups.py

@@ -148,10 +148,10 @@ async def test_filter_groups_by_role(
     expected_total: int,
     expected_roles: set,
 ):
-    role_query = f"?role={role}" if role else ""
     response = await client.get(
-        f"v1/groups{role_query}",
+        "v1/groups",
         headers={"Authorization": f"Bearer {user_with_many_roles.token}"},
+        params={"role": role} if role else None,
     )
 
     assert response.status_code == 200, response.text
@@ -181,10 +181,10 @@ async def test_filter_groups_not_applied_to_superuser(
     role: str,
     expected_total: int,
 ):
-    role_query = f"?role={role}" if role else ""
     response = await client.get(
-        f"v1/groups{role_query}",
+        "v1/groups",
         headers={"Authorization": f"Bearer {superuser.token}"},
+        params={"role": role} if role else None,
     )
 
     assert response.status_code == 200, response.text

tests/test_unit/test_queues/test_read_queues.py

@@ -21,7 +21,7 @@ async def test_group_member_can_read_queues(
 ):
     user = mock_group.get_member_of_role(role_guest_plus)
     response = await client.get(
-        f"v1/queues?group_id={mock_group.id}",
+        f"v1/queues",
         headers={"Authorization": f"Bearer {user.token}"},
         params={"group_id": mock_group.id},
     )