refactor: address quality issues

Author: mariajgrimaldi

openedx_authz/admin.py

@@ -59,7 +59,8 @@ class ScopeTypeFilter(admin.SimpleListFilter):
     parameter_name = "scope_type"
 
     def lookups(self, request, model_admin):
-        """Return the available scope type choices.
+        """
+        Return the available scope type choices.
 
         Audit records are independent from live Casbin tables and scope objects:
         there are no FK references to filter on. The namespace prefix in the

openedx_authz/api/roles.py

@@ -12,7 +12,6 @@
 
 from crum import get_current_user
 from django.db import transaction
-
 from openedx_events.authz.data import RoleAssignmentData as RoleAssignmentEventData
 from openedx_events.authz.signals import ROLE_ASSIGNMENT_CREATED, ROLE_ASSIGNMENT_DELETED
 

openedx_authz/migrations/0008_roleassignmentaudit.py

@@ -46,15 +46,21 @@ class Migration(migrations.Migration):
                 (
                     "scope",
                     models.CharField(
-                        help_text="Namespaced key of the scope (e.g. 'course-v1^course-v1:org+course+run') or glob pattern.",
+                        help_text=(
+                            "Namespaced key of the scope"
+                            " (e.g. 'course-v1^course-v1:org+course+run') or glob pattern."
+                        ),
                         max_length=255,
                     ),
                 ),
                 (
                     "actor",
                     models.CharField(
                         blank=True,
-                        help_text="Username of the user who performed the operation, or None for system-initiated actions.",
+                        help_text=(
+                            "Username of the user who performed the operation,"
+                            " or None for system-initiated actions."
+                        ),
                         max_length=150,
                         null=True,
                     ),

openedx_authz/models/core.py

@@ -13,7 +13,6 @@
 from openedx_authz.constants import AUTHZ_POLICY_ATTRIBUTES_SEPARATOR
 from openedx_authz.engine.filter import Filter
 
-
 User = get_user_model()
 
 class BaseRegistryModel(models.Model):

openedx_authz/tests/api/test_roles.py

@@ -13,7 +13,6 @@
 from ddt import ddt, unpack
 from django.test import TestCase
 
-
 from openedx_authz.api.data import (
     ActionData,
     ContentLibraryData,
@@ -54,7 +53,6 @@
 from openedx_authz.models import ExtendedCasbinRule, Scope, Subject
 
 
-
 def _mock_get_or_create_scope(scope_data):
     """Mock implementation that creates actual Scope instances."""
     scope, _ = Scope.objects.get_or_create(id=hash(scope_data.external_key) % 10000)