Skip to content

Commit fa7ea64

Browse files
committed
fixup! feat: add AuthZ permissions for schedule and course details
1 parent 878dda8 commit fa7ea64

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

cms/djangoapps/contentstore/rest_api/v1/views/course_details.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -150,7 +150,7 @@ def put(self, request: Request, course_id: str):
150150
along with all the course's details similar to a ``GET`` request.
151151
"""
152152
course_key = CourseKey.from_string(course_id)
153-
is_schedule_update, is_details_update = self.classify_update(request, course_key)
153+
is_schedule_update, is_details_update = self._classify_update(request, course_key)
154154

155155
if not is_schedule_update and not is_details_update:
156156
# No updatable fields provided in the request
@@ -182,7 +182,7 @@ def put(self, request: Request, course_id: str):
182182
serializer = CourseDetailsSerializer(updated_data)
183183
return Response(serializer.data)
184184

185-
def classify_update(self, request: Request, course_key: str) -> tuple[bool, bool]:
185+
def _classify_update(self, request: Request, course_key: str) -> tuple[bool, bool]:
186186
"""
187187
Determine whether the payload is updating schedule fields, detail fields, or both.
188188

openedx/core/djangoapps/authz/tests/mixins.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,11 +55,11 @@ def setUp(self):
5555
self.unauthorized_client = APIClient()
5656
self.unauthorized_client.force_authenticate(user=self.unauthorized_user)
5757

58-
self.super_user = UserFactory(is_superuser=True)
58+
self.super_user = UserFactory(is_superuser=True, password=self.password)
5959
self.super_client = APIClient()
6060
self.super_client.force_authenticate(user=self.super_user)
6161

62-
self.staff_user = UserFactory(is_staff=True)
62+
self.staff_user = UserFactory(is_staff=True, password=self.password)
6363
self.staff_client = APIClient()
6464
self.staff_client.force_authenticate(user=self.staff_user)
6565

0 commit comments

Comments
 (0)