Add unit learning objectives report API endpoint

kolibri/core/logger/api.py

@@ -1,6 +1,5 @@
 import hashlib
 import logging
-import uuid as uuid_module
 from datetime import timedelta
 from itertools import groupby
 from math import ceil
@@ -54,6 +53,7 @@
 from kolibri.core.logger.constants.exercise_attempts import MAPPING
 from kolibri.core.logger.evaluation import attempts_diff
 from kolibri.core.logger.evaluation import LOG_ORDER_BY
+from kolibri.core.logger.utils.pre_post_test import get_synthetic_content_id
 from kolibri.core.notifications.api import create_summarylog
 from kolibri.core.notifications.api import parse_attemptslog
 from kolibri.core.notifications.api import parse_summarylog
@@ -389,10 +389,8 @@ def _get_context(self, user, validated_data):
                 "test_type": test_type,
             }
 
-            # Synthetic content_id: UUID5 so pre and post are distinct
-            content_id = uuid_module.uuid5(
-                uuid_module.UUID(deterministic_hash), test_type
-            ).hex
+            # Synthetic content_id: shared across all learners for this test instance.
+            content_id = get_synthetic_content_id(course_session_id, unit_id, test_type)
             channel_id = None
             kind = content_kinds.QUIZ
 

kolibri/core/logger/utils/pre_post_test.py

@@ -0,0 +1,15 @@
+"""
+Single authoritative implementation of get_synthetic_content_id, shared between
+the learner-side logger API (kolibri.core.logger.api) and the coach-side unit
+report API (kolibri.plugins.coach.unit_report_api).
+"""
+import uuid
+
+_PRE_POST_TEST_SYNTHETIC_CONTENT_ID_NAMESPACE = uuid.UUID(
+    "7c9e4b1a-3d5f-4a8e-9c2b-6d0e1f2a3b4c"
+)
+
+
+def get_synthetic_content_id(course_session_id, unit_id, test_type):
+    key = "{}:{}:{}".format(course_session_id, unit_id, test_type)
+    return uuid.uuid5(_PRE_POST_TEST_SYNTHETIC_CONTENT_ID_NAMESPACE, key).hex

kolibri/plugins/coach/api_urls.py

@@ -8,6 +8,7 @@
 from .api import PracticeQuizDifficultQuestionsViewset
 from .api import QuizDifficultQuestionsViewset
 from .class_summary_api import ClassSummaryViewSet
+from .unit_report_api import UnitReportViewSet
 
 router = routers.DefaultRouter()
 
@@ -30,4 +31,11 @@
     basename="practicequizdifficulties",
 )
 
-urlpatterns = [re_path(r"^", include(router.urls))]
+urlpatterns = [
+    re_path(r"^", include(router.urls)),
+    re_path(
+        r"^coursesession/(?P<course_session_id>[0-9a-fA-F]{32})/unit/(?P<unit_contentnode_id>[0-9a-fA-F]{32})/report/$",
+        UnitReportViewSet.as_view({"get": "retrieve"}),
+        name="unitreport",
+    ),
+]