fix: fix revocation utility comparison (#32751)

* fix: fix revocation utility comparison
openedx · Jul 14, 2023 · 1af9803 · 1af9803
1 parent 3476647
commit 1af9803
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/common/djangoapps/entitlements/rest_api/v1/tests/test_views.py b/common/djangoapps/entitlements/rest_api/v1/tests/test_views.py
@@ -1299,7 +1299,7 @@ def test_already_completed_course(self, mock_get_courses_completion_status):
             is_active=True,
             mode=CourseMode.VERIFIED
         )
-        mock_get_courses_completion_status.return_value = ([enrollment.course_id], False)
+        mock_get_courses_completion_status.return_value = ([str(enrollment.course_id)], False)
         course_entitlement = CourseEntitlementFactory.create(user=self.user, enrollment_course_run=enrollment)
         url = reverse(self.REVOKE_VERIFIED_ACCESS_PATH)
 

diff --git a/common/djangoapps/entitlements/utils.py b/common/djangoapps/entitlements/utils.py
@@ -77,7 +77,7 @@ def revoke_entitlements_and_downgrade_courses_to_audit(course_entitlements, user
         if course_entitlement.enrollment_course_run is None:
             if course_entitlement.expired_at is None:
                 course_entitlement.expire_entitlement()
-        elif course_entitlement.enrollment_course_run.course_id not in awarded_cert_course_ids:
+        elif str(course_entitlement.enrollment_course_run.course_id) not in awarded_cert_course_ids:
             course_id = course_entitlement.enrollment_course_run.course_id
             enrollment_mode = course_entitlement.enrollment_course_run.mode
             username = course_entitlement.enrollment_course_run.user.username