Skip to content
Snippets Groups Projects
Commit 9b093ff1 authored by Eric Fischer's avatar Eric Fischer
Browse files

Add passthrough for context_title and context_label LTI parameters

parent 38f53585
Branches
Tags release-2020-10-02-15.01
No related merge requests found
......@@ -28,6 +28,8 @@ LTI_DEFAULT_PARAMS = {
}
LTI_OPTIONAL_PARAMS = {
'context_title': u'context title',
'context_label': u'context label',
'lis_result_sourcedid': u'result sourcedid',
'lis_outcome_service_url': u'outcome service URL',
'tool_consumer_instance_guid': u'consumer instance guid'
......@@ -95,6 +97,21 @@ class LtiLaunchTest(LtiTestMixin, TestCase):
views.lti_launch(request, unicode(COURSE_KEY), unicode(USAGE_KEY))
render.assert_called_with(request, USAGE_KEY)
@patch('lti_provider.views.render_courseware')
@patch('lti_provider.views.store_outcome_parameters')
@patch('lti_provider.views.authenticate_lti_user')
def test_valid_launch_with_optional_params(self, _authenticate, store_params, _render):
"""
Verifies that the LTI launch succeeds when passed a valid request.
"""
request = build_launch_request(extra_post_data=LTI_OPTIONAL_PARAMS)
views.lti_launch(request, unicode(COURSE_KEY), unicode(USAGE_KEY))
store_params.assert_called_with(
dict(ALL_PARAMS.items() + LTI_OPTIONAL_PARAMS.items()),
request.user,
self.consumer
)
@patch('lti_provider.views.render_courseware')
@patch('lti_provider.views.store_outcome_parameters')
@patch('lti_provider.views.authenticate_lti_user')
......
......@@ -28,8 +28,8 @@ REQUIRED_PARAMETERS = [
]
OPTIONAL_PARAMETERS = [
'lis_result_sourcedid', 'lis_outcome_service_url',
'tool_consumer_instance_guid'
'context_title', 'context_label', 'lis_result_sourcedid',
'lis_outcome_service_url', 'tool_consumer_instance_guid'
]
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment