fix: wrong user id sent to event bus (#213)

edx · Nov 15, 2023 · 707e5be · 707e5be
1 parent c0d5c97
commit 707e5be
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/edx_exams/apps/core/signals/signals.py b/edx_exams/apps/core/signals/signals.py
@@ -17,7 +17,7 @@ def _create_user_data(user):
     Helper function to create a UserData object.
     """
     user_data = UserData(
-        id=user.id,
+        id=user.lms_user_id,
         is_active=user.is_active,
         pii=UserPersonalData(
             username=user.username,

diff --git a/edx_exams/apps/core/tests/test_api.py b/edx_exams/apps/core/tests/test_api.py
@@ -317,7 +317,7 @@ def test_attempt_event_emitted(self, event_name, status, expect_requesting_user)
             self.assertEqual(mock_event_send.call_count, 1)
 
             user_data = UserData(
-                id=self.user.id,
+                id=self.user.lms_user_id,
                 is_active=self.user.is_active,
                 pii=UserPersonalData(
                     username=self.user.username,
@@ -650,7 +650,7 @@ def test_event_emitted(self, mock_event_send):
         reset_exam_attempt(self.exam_attempt, self.user)
 
         user_data = UserData(
-            id=self.student_user.id,
+            id=self.student_user.lms_user_id,
             is_active=self.student_user.is_active,
             pii=UserPersonalData(
                 username=self.student_user.username,
@@ -659,7 +659,7 @@ def test_event_emitted(self, mock_event_send):
             )
         )
         requesting_user_data = UserData(
-            id=self.user.id,
+            id=self.user.lms_user_id,
             is_active=self.user.is_active,
             pii=UserPersonalData(
                 username=self.user.username,