From bc64c2ce365fc3212b13eb86c2ada345a8b1f3bf Mon Sep 17 00:00:00 2001 From: Arunmozhi Date: Thu, 14 Sep 2023 11:43:29 +0530 Subject: [PATCH] fix: failing tests in the xapi module --- .../test_xapi/test_commands/test_send_course_enrollments.py | 5 +---- .../test_statements/test_learner_course_enrollment.py | 2 +- 2 files changed, 2 insertions(+), 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 index 2e339480c5..a02be9cd62 100644 --- 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 index 13408e527b..248e548392 100644 --- 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,