Skip to content
Snippets Groups Projects
Commit c24236d4 authored by Douglas Hall's avatar Douglas Hall
Browse files

Merge pull request #11 from indagation/add_name_and_date

Added display name and due date to LTI Parameters
parents a9fe0cbd 91d4b1fd
Branches
No related tags found
No related merge requests found
...@@ -128,8 +128,14 @@ class LtiConsumer(object): ...@@ -128,8 +128,14 @@ class LtiConsumer(object):
u'lis_result_sourcedid': self.xblock.lis_result_sourcedid, u'lis_result_sourcedid': self.xblock.lis_result_sourcedid,
u'context_id': self.xblock.context_id, u'context_id': self.xblock.context_id,
u'custom_component_display_name': self.xblock.display_name,
} }
if self.xblock.due:
lti_parameters['custom_component_due_date'] = self.xblock.due.strftime('%Y-%m-%d %H:%M:%S')
if self.xblock.graceperiod:
lti_parameters['custom_component_graceperiod'] = str(self.xblock.graceperiod.total_seconds())
if self.xblock.has_score: if self.xblock.has_score:
lti_parameters.update({ lti_parameters.update({
u'lis_outcome_service_url': self.xblock.outcome_service_url u'lis_outcome_service_url': self.xblock.outcome_service_url
......
...@@ -119,6 +119,9 @@ LTI_PARAMETERS = [ ...@@ -119,6 +119,9 @@ LTI_PARAMETERS = [
'tool_consumer_instance_description', 'tool_consumer_instance_description',
'tool_consumer_instance_url', 'tool_consumer_instance_url',
'tool_consumer_instance_contact_email', 'tool_consumer_instance_contact_email',
'custom_component_due_date',
'custom_component_graceperiod',
'custom_component_display_name'
] ]
......
...@@ -5,8 +5,11 @@ Unit tests for lti_consumer.lti module ...@@ -5,8 +5,11 @@ Unit tests for lti_consumer.lti module
import unittest import unittest
from datetime import timedelta
from mock import Mock, PropertyMock, patch from mock import Mock, PropertyMock, patch
from django.utils import timezone
from lti_consumer.tests.unit.test_utils import FAKE_USER_ID, make_request from lti_consumer.tests.unit.test_utils import FAKE_USER_ID, make_request
from lti_consumer.tests.unit.test_lti_consumer import TestLtiConsumerXBlock from lti_consumer.tests.unit.test_lti_consumer import TestLtiConsumerXBlock
...@@ -146,6 +149,9 @@ class TestLtiConsumer(TestLtiConsumerXBlock): ...@@ -146,6 +149,9 @@ class TestLtiConsumer(TestLtiConsumerXBlock):
""" """
Test `get_signed_lti_parameters` returns the correct dict Test `get_signed_lti_parameters` returns the correct dict
""" """
self.lti_consumer.xblock.due = timezone.now()
self.lti_consumer.xblock.graceperiod = timedelta(days=1)
expected_lti_parameters = { expected_lti_parameters = {
u'user_id': self.lti_consumer.xblock.user_id, u'user_id': self.lti_consumer.xblock.user_id,
u'oauth_callback': u'about:blank', u'oauth_callback': u'about:blank',
...@@ -157,6 +163,9 @@ class TestLtiConsumer(TestLtiConsumerXBlock): ...@@ -157,6 +163,9 @@ class TestLtiConsumer(TestLtiConsumerXBlock):
u'lis_result_sourcedid': self.lti_consumer.xblock.lis_result_sourcedid, u'lis_result_sourcedid': self.lti_consumer.xblock.lis_result_sourcedid,
u'context_id': self.lti_consumer.xblock.context_id, u'context_id': self.lti_consumer.xblock.context_id,
u'lis_outcome_service_url': self.lti_consumer.xblock.outcome_service_url, u'lis_outcome_service_url': self.lti_consumer.xblock.outcome_service_url,
u'custom_component_display_name': self.lti_consumer.xblock.display_name,
u'custom_component_due_date': self.lti_consumer.xblock.due.strftime('%Y-%m-%d %H:%M:%S'),
u'custom_component_graceperiod': str(self.lti_consumer.xblock.graceperiod.total_seconds()),
'lis_person_sourcedid': 'edx', 'lis_person_sourcedid': 'edx',
'lis_person_contact_email_primary': 'edx@example.com', 'lis_person_contact_email_primary': 'edx@example.com',
u'custom_param_1': 'custom1', u'custom_param_1': 'custom1',
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment