fix: add context data to event attempt events

openedx · Sep 8, 2023 · c1978e5 · c1978e5
1 parent 8e990ee
commit c1978e5
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 14 deletions.
diff --git a/event_routing_backends/processors/xapi/event_transformers/exam_events.py b/event_routing_backends/processors/xapi/event_transformers/exam_events.py
@@ -32,20 +32,32 @@ def get_object(self):
         Returns:
             `Activity`
         """
-        object_id = 'dummy' ## TODO Load the object id from the event
+        object_id = self.get_data('data.exam_content_id') ## TODO Load the object id from the event
 
         return Activity(
             id=object_id,
             definition=ActivityDefinition(
                 type=constants.XAPI_ACTIVITY_ATTEMPT,
+                name=LanguageMap(**({constants.EN: self.get_data('data.exam_name')})),
                 extensions=Extensions({
                     constants.XAPI_ACTIVITY_ID: self.get_data('data.attempt_id'),
-                    constants.XAPI_ACTIVITY_ATTEMPT_STATUS: self.get_data('data.attempt_status'),
-                    constants.XAPI_ACTIVITY_ATTEMPT_TIME: self.get_data('data.attempt_event_elapsed_time_secs'),
+                    constants.XAPI_ACTIVITY_CODE: self.get_data('data.attempt_code'),
                 })
             ),
         )
 
+    def get_context_extensions(self):
+        extensions = super().get_context_extensions()
+
+        return extensions.update({
+            constants.XAPI_ATTEMPT_TYPE: "exam",
+            constants.XAPI_PRACTICE_EXAM: self.get_data('data.exam_is_practice_exam'),
+            constants.XAPI_PROCTORED_EXAM: self.get_data('data.exam_is_proctored'),
+            constants.XAPI_ATTEMPT_STARTED: self.get_data('data.attempt_started_at'),
+            constants.XAPI_ATTEMPT_COMPLETED: self.get_data('data.attempt_completed_at'),
+            constants.XAPI_ACTIVITY_ATTEMPT_TIME: self.get_data('data.attempt_event_elapsed_time_secs'),
+        })
+
 
 @XApiTransformersRegistry.register('edx.special_exam.practice.attempt.started')
 class EnrollmentActivatedTransformer(BaseExamAttemptTransformer):
@@ -57,11 +69,6 @@ class EnrollmentActivatedTransformer(BaseExamAttemptTransformer):
         display=LanguageMap({constants.EN: constants.STARTED}),
     )
 
-    def get_context_extensions(self):
-        extensions = super().get_context_extensions()
-        return extensions.update({
-            constants.XAPI_ATTEMPT_TYPE: "practice",
-        })
 
 
 @XApiTransformersRegistry.register('edx.special_exam.practice.attempt.submitted')
@@ -73,9 +80,3 @@ class EnrollmentDeactivatedTransformer(BaseExamAttemptTransformer):
         id=constants.XAPI_VERB_SUBMITTED,
         display=LanguageMap({constants.EN: constants.SUBMITTED}),
     )
-
-    def get_context_extensions(self):
-        extensions = super().get_context_extensions()
-        return extensions.update({
-            constants.XAPI_ATTEMPT_TYPE: "practice",
-        })
diff --git a/event_routing_backends/settings/common.py b/event_routing_backends/settings/common.py
@@ -74,6 +74,8 @@ def plugin_settings(settings):
                                     'edx.course.enrollment.mode_changed',
                                     'edx.grades.subsection.grade_calculated',
                                     'edx.grades.course.grade_calculated',
+                                    'edx.special_exam.timed_attempt.created',
+                                    'edx.special_exam.timed_attempt.submitted',
                                     'edx.forum.thread.created',
                                     'edx.forum.thread.deleted',
                                     'edx.forum.thread.edited',