refactor: clean up unused imports in roles and test views

Author: BryanttV

openedx_authz/api/roles.py

@@ -18,8 +18,6 @@
     RoleData,
     ScopeData,
     SubjectData,
-    UserData,
-    SubjectRoleAssignmentData,
 )
 from openedx_authz.api.permissions import get_permission_from_policy
 from openedx_authz.engine.enforcer import enforcer

openedx_authz/tests/rest_api/test_views.py

@@ -15,14 +15,7 @@
 from rest_framework.test import APITestCase
 
 from openedx_authz import api
-from openedx_authz.api.data import (
-    ActionData,
-    PermissionData,
-    RoleAssignmentData,
-    RoleData,
-    ScopeData,
-    UserData,
-)
+from openedx_authz.api.data import ActionData, PermissionData, RoleAssignmentData, RoleData, ScopeData, UserData
 
 User = get_user_model()
 
@@ -329,7 +322,7 @@ def test_put_add_users_to_role(
         expected_errors: int,
         mock_assign_role_side_effect: list[bool],
         mock_assign_role,
-    ):
+    ):  # pylint: disable=too-many-positional-arguments
         """Test adding users to a role within a scope.
 
         Expected result:
@@ -430,7 +423,7 @@ def test_delete_remove_users_from_role(
         expected_errors: int,
         mock_unassign_role_side_effect: list[bool],
         mock_unassign_role,
-    ):
+    ):  # pylint: disable=too-many-positional-arguments
         """Test removing users from a role within a scope.
 
         Expected result: