refactor: address quality issues

Author: mariajgrimaldi

openedx_authz/api/roles.py

@@ -242,7 +242,7 @@ def assign_role_to_subject_in_scope(subject: SubjectData, role: RoleData, scope:
                 subject=subject.namespaced_key,
                 role=role.namespaced_key,
                 scope=scope.namespaced_key,
-                actor=getattr(get_current_user(), "username", None)
+                actor=getattr(get_current_user(), "username", None),
             )
         )
     )
@@ -293,7 +293,7 @@ def unassign_role_from_subject_in_scope(subject: SubjectData, role: RoleData, sc
                     subject=subject.namespaced_key,
                     role=role.namespaced_key,
                     scope=scope.namespaced_key,
-                    actor=getattr(get_current_user(), "username", None)
+                    actor=getattr(get_current_user(), "username", None),
                 )
             )
         )

openedx_authz/engine/utils.py

@@ -405,7 +405,10 @@ def migrate_authz_to_legacy_course_roles(
     _validate_migration_input(course_id_list, org_id)
 
     role_assignments = get_all_role_assignments_per_scope_type(
-        scope_types=(CourseOverviewData, OrgCourseOverviewGlobData)
+        scope_types=(
+            CourseOverviewData,
+            OrgCourseOverviewGlobData,
+        )
     )
 
     # Two cases here:

openedx_authz/migrations/0008_roleassignmentaudit.py

@@ -4,7 +4,6 @@
 
 
 class Migration(migrations.Migration):
-
     dependencies = [
         ("openedx_authz", "0007_coursescope"),
     ]
@@ -47,8 +46,7 @@ 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."
+                            "Namespaced key of the scope (e.g. 'course-v1^course-v1:org+course+run') or glob pattern."
                         ),
                         max_length=255,
                     ),
@@ -58,8 +56,7 @@ class Migration(migrations.Migration):
                     models.CharField(
                         blank=True,
                         help_text=(
-                            "Username of the user who performed the operation,"
-                            " or None for system-initiated actions."
+                            "Username of the user who performed the operation, or None for system-initiated actions."
                         ),
                         max_length=150,
                         null=True,

openedx_authz/models/core.py

@@ -12,6 +12,7 @@
 from openedx_authz.constants import AUTHZ_POLICY_ATTRIBUTES_SEPARATOR
 from openedx_authz.engine.filter import Filter
 
+
 class BaseRegistryModel(models.Model):
     """Base model that supports automatic subclass registration.