feat: ProctoringConsumer basic launch request (#388)

CHANGELOG.rst

@@ -16,6 +16,10 @@ Please See the `releases tab <https://github.com/openedx/xblock-lti-consumer/rel
 Unreleased
 ~~~~~~~~~~
 
+9.5.4 - 2023-06-28
+------------------
+* Allow basic LtiResourceLink launch for an LtiProctoringConsumer
+
 9.5.3 - 2023-06-09
 ------------------
 * Redirect to exam on same LTI proctoring launch tab once ready to start.

lti_consumer/__init__.py

@@ -4,4 +4,4 @@
 from .apps import LTIConsumerApp
 from .lti_xblock import LtiConsumerXBlock
 
-__version__ = '9.5.3'
+__version__ = '9.5.4'

lti_consumer/lti_1p3/consumer.py

@@ -885,8 +885,13 @@ def generate_launch_request(
                 self.set_extra_claim(self.get_assessment_control_claim())
         elif launch_data.message_type == "LtiEndAssessment":
             proctoring_claims = self.get_end_assessment_claims()
+        elif launch_data.message_type == "LtiResourceLinkRequest":
+            proctoring_claims = {}
         else:
-            raise ValueError('lti_message_hint must \"LtiStartProctoring\" or \"LtiEndAssessment\".')
+            raise ValueError(
+                'lti_message_hint must be \"LtiStartProctoring\" or \"LtiEndAssessment\"'
+                'or \"LtiResourceLinkRequest\"'
+            )
 
         self.set_extra_claim(proctoring_claims)
 

lti_consumer/lti_1p3/tests/test_consumer.py

@@ -1087,6 +1087,19 @@ def test_generate_launch_request(self, message_type, mock_get_data_from_cache):
         for claim in expected_claims.items():
             self.assertIn(claim, decoded_token_claims)
 
+    @patch('lti_consumer.lti_1p3.consumer.get_data_from_cache')
+    def test_generate_basic_launch_request(self, mock_get_data_from_cache):
+        mock_launch_data = self.get_launch_data(message_type="LtiResourceLinkRequest")
+        mock_get_data_from_cache.return_value = mock_launch_data
+
+        self._setup_proctoring()
+        token = self.lti_consumer.generate_launch_request(
+            self.preflight_response,
+        )['id_token']
+
+        # just check token is valid
+        self.lti_consumer.key_handler.validate_and_decode(token)
+
     @patch('lti_consumer.lti_1p3.consumer.get_data_from_cache')
     def test_enable_assessment_control(self, mock_get_data_from_cache):
         """
@@ -1120,11 +1133,11 @@ def test_enable_assessment_control(self, mock_get_data_from_cache):
     @patch('lti_consumer.lti_1p3.consumer.get_data_from_cache')
     def test_generate_launch_request_invalid_message(self, mock_get_data_from_cache):
         """
-        Ensures that a ValueError is raised if the launch_data.message_type is not LtiStartProctoring or
-        LtiEndAssessment.
+        Ensures that a ValueError is raised if the launch_data.message_type is not LtiStartProctoring,
+        LtiEndAssessment, or LtiResourceLinkRequest.
         """
 
-        mock_launch_data = self.get_launch_data(message_type="LtiResourceLinkRequest")
+        mock_launch_data = self.get_launch_data(message_type="LtiDeepLinkingRequest")
         mock_get_data_from_cache.return_value = mock_launch_data
 
         self._setup_proctoring()