feat: redirect to exam on tab (#382)

varshamenon4 · web-flow · commit 364e582f7ff5 · 2023-06-09T12:00:13.000-04:00
diff --git a/CHANGELOG.rst b/CHANGELOG.rst
@@ -16,6 +16,9 @@ Please See the `releases tab <https://github.com/openedx/xblock-lti-consumer/rel
 Unreleased
 ~~~~~~~~~~
 
+9.5.3 - 2023-06-09
+------------------
+* Redirect to exam on same LTI proctoring launch tab once ready to start.
 * Show a warning in Studio if an LTI 1.1 consumer has an invalid `lti_id`.
 
 9.5.2 - 2023-05-24
diff --git a/lti_consumer/plugin/views.py b/lti_consumer/plugin/views.py
@@ -4,6 +4,7 @@
 import logging
 import urllib
 
+from django.conf import settings
 from django.contrib.auth import get_user_model
 from django.core.exceptions import ObjectDoesNotExist, PermissionDenied, ValidationError
 from django.db import transaction
@@ -835,4 +836,12 @@ def start_proctoring_assessment_endpoint(request):
         user_id=request.user.id,
     )
 
-    return render(request, 'html/lti_start_assessment.html', status=HTTP_200_OK)
+    context_url = "/".join([settings.LEARNING_MICROFRONTEND_URL, "course",
+                            launch_data.context_id,
+                            launch_data.context_label])
+    context = {}
+    context.update({
+        "context_url": context_url,
+    })
+
+    return render(request, 'html/lti_start_assessment.html', context, status=HTTP_200_OK)
diff --git a/lti_consumer/templates/html/lti_start_assessment.html b/lti_consumer/templates/html/lti_start_assessment.html
@@ -1,19 +1,17 @@
 {% load i18n %}
+
 <!DOCTYPE html>
 <html>
     <head>
-        <meta http-equiv="Content-Type" content="text/html; charset=UTF-8" >
+        <meta http-equiv="refresh" content="5; url='{{ context_url }}'" >
         <title>LTI</title>
     </head>
     <body>
         <p>
             <b>{% trans "Sending you back to your exam." %}</b>
         </p>
-        <script>
-            setTimeout(function () {window.close()}, 5000);
-        </script>
         <p>
-            <a href="javascript:window.close();">
+            <a href="{{ context_url }}">
                 {% trans "Return to exam." %}
             </a>
         </p>
diff --git a/lti_consumer/tests/unit/plugin/test_proctoring.py b/lti_consumer/tests/unit/plugin/test_proctoring.py
@@ -85,6 +85,9 @@ def _setup_cached_data(self):
             config_id=self.lti_config.config_id,
             resource_link_id="resource_link_id",
             proctoring_launch_data=proctoring_launch_data,
+            context_id="course-v1:testU+DemoX+Demo_Course",
+            context_title="http://localhost:2000",
+            context_label="block-v1:testU+DemoX+Demo_Course+type@sequential+block@1234",
         )
 
         self.launch_data_key = get_cache_key(
@@ -293,4 +296,4 @@ def test_start_assessment_endpoint_returns_valid_html(self):
         )
 
         self.assertEqual(response.status_code, 200)
-        self.assertIn('window.close', response.content.decode('utf-8'))
+        self.assertIn('Return to exam', response.content.decode('utf-8'))
diff --git a/test_settings.py b/test_settings.py
@@ -28,3 +28,6 @@
 
 # Platform name for LTI 1.1 and 1.3 claims testing
 PLATFORM_NAME = "Your platform name here"
+
+# Learning MFE URL for start assessment testing
+LEARNING_MICROFRONTEND_URL = 'http://test.learning:2000'

Original file line number	Diff line number	Diff line change
`@@ -85,6 +85,9 @@ def _setup_cached_data(self):`
`85`	`85`	`config_id=self.lti_config.config_id,`
`86`	`86`	`resource_link_id="resource_link_id",`
`87`	`87`	`proctoring_launch_data=proctoring_launch_data,`
	`88`	`+ context_id="course-v1:testU+DemoX+Demo_Course",`
	`89`	`+ context_title="http://localhost:2000",`
	`90`	`+ context_label="block-v1:testU+DemoX+Demo_Course+type@sequential+block@1234",`
`88`	`91`	`)`
`89`	`92`
`90`	`93`	`self.launch_data_key = get_cache_key(`
`@@ -293,4 +296,4 @@ def test_start_assessment_endpoint_returns_valid_html(self):`
`293`	`296`	`)`
`294`	`297`
`295`	`298`	`self.assertEqual(response.status_code, 200)`
`296`		`- self.assertIn('window.close', response.content.decode('utf-8'))`
	`299`	`+ self.assertIn('Return to exam', response.content.decode('utf-8'))`