Commit c0b7f9b8 by Diana Huang

Only try to render dates when user is authenticated.

parent 15b11141
...@@ -96,10 +96,17 @@ class CourseDateSummaryTest(SharedModuleStoreTestCase): ...@@ -96,10 +96,17 @@ class CourseDateSummaryTest(SharedModuleStoreTestCase):
def test_course_info_feature_flag(self): def test_course_info_feature_flag(self):
SelfPacedConfiguration(enable_course_home_improvements=False).save() SelfPacedConfiguration(enable_course_home_improvements=False).save()
self.setup_course_and_user() self.setup_course_and_user()
self.client.login(username='mrrobot', password='test')
url = reverse('info', args=(self.course.id,)) url = reverse('info', args=(self.course.id,))
response = self.client.get(url) response = self.client.get(url)
self.assertNotIn('date-summary', response.content) self.assertNotIn('date-summary', response.content)
def test_course_info_logged_out(self):
self.setup_course_and_user()
url = reverse('info', args=(self.course.id,))
response = self.client.get(url)
self.assertEqual(200, response.status_code)
# Tests for which blocks are enabled # Tests for which blocks are enabled
def assert_block_types(self, expected_blocks): def assert_block_types(self, expected_blocks):
"""Assert that the enabled block types for this course are as expected.""" """Assert that the enabled block types for this course are as expected."""
......
...@@ -334,10 +334,10 @@ def course_info(request, course_id): ...@@ -334,10 +334,10 @@ def course_info(request, course_id):
# Construct the dates fragment # Construct the dates fragment
dates_fragment = None dates_fragment = None
if SelfPacedConfiguration.current().enable_course_home_improvements:
dates_fragment = CourseDatesFragmentView().render_to_fragment(request, course_id=course_id)
if request.user.is_authenticated(): if request.user.is_authenticated():
if SelfPacedConfiguration.current().enable_course_home_improvements:
dates_fragment = CourseDatesFragmentView().render_to_fragment(request, course_id=course_id)
show_upgrade_notification = False show_upgrade_notification = False
if request.GET.get('upgrade', 'false') == 'true': if request.GET.get('upgrade', 'false') == 'true':
store_upgrade_cookie = True store_upgrade_cookie = True
......
Markdown is supported
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