refactor: address quality issues

Author: mariajgrimaldi

openedx_authz/api/decorators.py

@@ -37,8 +37,9 @@ def get_all_roles():
         def get_roles_in_scope(scope: ScopeData):
             return enforcer.get_filtered_roles(scope.namespaced_key)
     """
+
     FILTER_DATA_CLASSES = { # Consider empty for no filtering
-        "scope": ScopeData,
+        # "scope": ScopeData,
         # TODO: currently ALLOW_FILTERED_POLICY_LOADING is False to avoid partial policy loads. We can
         # Allow filtering on scope (initially) once we have a CONF model that supports this so filtering is meaningful,
         # consistent and doesn't lead to partial policy loads.

openedx_authz/api/roles.py

@@ -176,6 +176,7 @@ def get_role_definitions_in_scope(scope: ScopeData) -> list[RoleData]:
         for role, permissions in permissions_per_role.items()
     ]
 
+
 @manage_policy_lifecycle()
 def get_all_roles_names() -> list[str]:
     """Get all the available roles names in the current environment.