refactor: abstract process in _get_course_keys_from_scopes

Author: MaferMazu

cms/djangoapps/contentstore/views/course.py

@@ -29,7 +29,7 @@
 from opaque_keys.edx.keys import CourseKey
 from opaque_keys.edx.locator import BlockUsageLocator
 from openedx_authz.api import get_scopes_for_user_and_permission
-from openedx_authz.api.data import CourseOverviewData, OrgCourseOverviewGlobData
+from openedx_authz.api.data import CourseOverviewData, OrgCourseOverviewGlobData, ScopeData
 from openedx_authz.constants.permissions import (
     COURSES_MANAGE_COURSE_UPDATES,
     COURSES_MANAGE_GROUP_CONFIGURATIONS,
@@ -824,18 +824,10 @@ def _get_course_keys_for_org_scope(org_keys: set[str]):
 
     return CourseOverview.get_all_courses(orgs=org_keys).values_list('id', flat=True)
 
-
-def _get_authz_accessible_courses_list(request):
+def _get_course_keys_from_scopes(authz_scopes: list[ScopeData]):
     """
-    List all courses available to the logged in user by
-    evaluating Authz scopes for course access.
+    Convert a set of Authz scopes into specific course keys.
     """
-    user = request.user
-    authz_scopes = get_scopes_for_user_and_permission(
-        user.username,
-        COURSES_VIEW_COURSE.identifier
-    )
-
     course_keys = set()
     org_keys = set()
     for access in authz_scopes:
@@ -849,9 +841,20 @@ def _get_authz_accessible_courses_list(request):
             key for key in _get_course_keys_for_org_scope(org_keys)
             if core_toggles.enable_authz_course_authoring(key)
         )
-
     return course_keys
 
+def _get_authz_accessible_courses_list(request):
+    """
+    List all courses available to the logged in user by
+    evaluating Authz scopes for course access.
+    """
+    user = request.user
+    authz_scopes = get_scopes_for_user_and_permission(
+        user.username,
+        COURSES_VIEW_COURSE.identifier
+    )
+
+    return _get_course_keys_from_scopes(authz_scopes)
 
 def _get_legacy_accessible_courses_list(request):
     """