Skip to content

Commit 32c731f

Browse files
committed
refactor: replace external_key initialization with build_external_key method
1 parent 68e4042 commit 32c731f

2 files changed

Lines changed: 7 additions & 7 deletions

File tree

cms/djangoapps/contentstore/tests/test_course_listing.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -724,7 +724,7 @@ def test_course_listing_with_org_scope(self):
724724
the AuthZ course authoring toggle is enabled.
725725
"""
726726
_, _, authz_courses, legacy_courses = self._create_courses()
727-
org_scope = OrgCourseOverviewGlobData(external_key='course-v1:Org1+*')
727+
org_scope = OrgCourseOverviewGlobData.build_external_key("Org1")
728728
assign_role_to_user_in_scope(
729729
self.authorized_user.username,
730730
COURSE_STAFF.external_key,
@@ -757,7 +757,7 @@ def test_course_listing_with_org_scope_with_toggle(self):
757757
authz_keys, _, _, _ = self._create_courses()
758758
# enable only the first and third course keys
759759
enabled_keys = {str(authz_keys[0]), str(authz_keys[2])}
760-
org_scope = OrgCourseOverviewGlobData(external_key='course-v1:Org1+*')
760+
org_scope = OrgCourseOverviewGlobData.build_external_key("Org1")
761761
assign_role_to_user_in_scope(
762762
self.authorized_user.username,
763763
COURSE_STAFF.external_key,
@@ -784,7 +784,7 @@ def test_course_listing_with_org_scope_without_courses(self):
784784
courses, `get_courses_accessible_to_user` should return an empty
785785
list.
786786
"""
787-
org_scope = OrgCourseOverviewGlobData(external_key='course-v1:Org2+*')
787+
org_scope = OrgCourseOverviewGlobData.build_external_key("Org2")
788788
assign_role_to_user_in_scope(
789789
self.authorized_user.username,
790790
COURSE_STAFF.external_key,
@@ -806,8 +806,8 @@ def test_course_listing_with_org_scope_fetched_once(self):
806806
"""
807807
Verify that course overviews are fetched once with all authorized orgs.
808808
"""
809-
org_scope1 = OrgCourseOverviewGlobData(external_key='course-v1:Org1+*')
810-
org_scope2 = OrgCourseOverviewGlobData(external_key='course-v1:Org2+*')
809+
org_scope1 = OrgCourseOverviewGlobData.build_external_key("Org1")
810+
org_scope2 = OrgCourseOverviewGlobData.build_external_key("Org2")
811811
assign_role_to_user_in_scope(
812812
self.authorized_user.username,
813813
COURSE_STAFF.external_key,

common/djangoapps/student/tests/test_roles.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -310,7 +310,7 @@ def test_get_authz_compat_course_access_roles_for_user_org_scope(self):
310310
"""
311311
Org-wide AuthZ assignments should map to legacy org-level course access roles.
312312
"""
313-
org_scope = OrgCourseOverviewGlobData(external_key="course-v1:OpenedX+*")
313+
org_scope = OrgCourseOverviewGlobData.build_external_key("OpenedX")
314314
assignment = RoleAssignmentData(
315315
subject=UserData(external_key=self.student.username),
316316
roles=[RoleData(external_key=COURSE_ADMIN.external_key)],
@@ -338,7 +338,7 @@ def test_org_scope_authz_role_grants_instructor_dashboard_permissions(self):
338338
"""
339339
# pylint: disable=protected-access
340340
course_key = CourseKey.from_string("course-v1:OpenedX+DemoX+DemoCourse")
341-
org_scope = OrgCourseOverviewGlobData(external_key="course-v1:OpenedX+*")
341+
org_scope = OrgCourseOverviewGlobData.build_external_key("OpenedX")
342342
assignment = RoleAssignmentData(
343343
subject=UserData(external_key=self.student.username),
344344
roles=[RoleData(external_key=COURSE_ADMIN.external_key)],

0 commit comments

Comments
 (0)