Skip to content

Commit f15ab79

Browse files
committed
test: format code and uncomment tests
1 parent b33e2a3 commit f15ab79

1 file changed

Lines changed: 7 additions & 10 deletions

File tree

openedx_authz/tests/rest_api/test_views.py

Lines changed: 7 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -38,10 +38,7 @@ class ViewTestMixin(BaseRolesTestCase):
3838
"""Mixin providing common test utilities for view tests."""
3939

4040
@classmethod
41-
def _assign_roles_to_users(
42-
cls,
43-
assignments: list[dict] | None = None,
44-
):
41+
def _assign_roles_to_users(cls, assignments: list[dict] | None = None):
4542
"""Helper method to assign roles to multiple users.
4643
4744
This method can be used to assign a role to a single user or multiple users
@@ -114,7 +111,7 @@ def setUpClass(cls):
114111
},
115112
{
116113
"subject_name": "regular_7",
117-
"role_name": "library_collaborator",
114+
"role_name": "library_contributor",
118115
"scope_name": "lib:Org3:LIB3",
119116
},
120117
{
@@ -167,9 +164,9 @@ def setUp(self):
167164
([{"action": "view_library", "scope": "lib:Org1:LIB1"}], [True]),
168165
# Single permission - denied (scope not assigned to user)
169166
([{"action": "view_library", "scope": "lib:Org2:LIB2"}], [False]),
170-
# # Single permission - denied (action not assigned to user)
167+
# Single permission - denied (action not assigned to user)
171168
([{"action": "edit_library", "scope": "lib:Org1:LIB1"}], [False]),
172-
# # Multiple permissions - mixed results
169+
# Multiple permissions - mixed results
173170
(
174171
[
175172
{"action": "view_library", "scope": "lib:Org1:LIB1"},
@@ -792,9 +789,9 @@ def test_get_roles_pagination(self, query_params: dict, expected_count: int, has
792789
# Library Admin user
793790
("regular_5", status.HTTP_200_OK),
794791
# Library Author user
795-
# ("regular_6", status.HTTP_200_OK), # TODO: uncomment this when we have the explicit permissions
796-
# Library Collaborator user
797-
# ("regular_7", status.HTTP_200_OK), # TODO: uncomment this when we have the explicit permissions
792+
("regular_6", status.HTTP_200_OK), # TODO: uncomment this when we have the explicit permissions
793+
# Library Contributor user
794+
("regular_7", status.HTTP_200_OK), # TODO: uncomment this when we have the explicit permissions
798795
# Library User user
799796
("regular_8", status.HTTP_200_OK),
800797
# Regular user without permission

0 commit comments

Comments
 (0)