fix: failing tests in the xapi module

open-craft · Sep 14, 2023 · bc64c2c · bc64c2c
1 parent c9872ee
commit bc64c2c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/tests/test_integrated_channels/test_xapi/test_commands/test_send_course_enrollments.py b/tests/test_integrated_channels/test_xapi/test_commands/test_send_course_enrollments.py
@@ -287,10 +287,7 @@ def test_is_already_transmitted(self):
 
         Command.is_already_transmitted(xapi_transmissions, user_id, course_id)
 
-    @mock.patch(
-        'integrated_channels.xapi.utils.is_success_response',
-        mock.MagicMock(return_value=False)
-    )
+    @mock.patch(MODULE_PATH + 'is_success_response', mock.MagicMock(return_value=False))
     @mock.patch(MODULE_PATH + 'send_course_enrollment_statement')
     def test_transmit_course_enrollments_transmit_fail_skip(self, mock_send_statement):
         # pylint: disable=import-outside-toplevel

diff --git a/tests/test_integrated_channels/test_xapi/test_statements/test_learner_course_enrollment.py b/tests/test_integrated_channels/test_xapi/test_statements/test_learner_course_enrollment.py
@@ -39,7 +39,7 @@ def setUp(self):
         self.object_id_course = 'https://{domain}/xapi/activities/course/{activity_id}'.format(
             domain=self.site.domain,
             activity_id=self.course_overview.course_key)
-        self.plain_course_id = self.course_overview.course_key
+        self.plain_course_id = self.course_overview.id
 
         self.object_id_courserun = 'https://{domain}/xapi/activities/courserun/{activity_id}'.format(
             domain=self.site.domain,