diff --git a/lms/djangoapps/courseware/tests/test_date_summary.py b/lms/djangoapps/courseware/tests/test_date_summary.py index 66974670fa..d4619d882d 100644 --- a/lms/djangoapps/courseware/tests/test_date_summary.py +++ b/lms/djangoapps/courseware/tests/test_date_summary.py @@ -96,10 +96,17 @@ class CourseDateSummaryTest(SharedModuleStoreTestCase): def test_course_info_feature_flag(self): SelfPacedConfiguration(enable_course_home_improvements=False).save() self.setup_course_and_user() + self.client.login(username='mrrobot', password='test') url = reverse('info', args=(self.course.id,)) response = self.client.get(url) 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 def assert_block_types(self, expected_blocks): """Assert that the enabled block types for this course are as expected.""" diff --git a/lms/djangoapps/courseware/views/views.py b/lms/djangoapps/courseware/views/views.py index c08c2d9ede..00ae4bdbfe 100644 --- a/lms/djangoapps/courseware/views/views.py +++ b/lms/djangoapps/courseware/views/views.py @@ -334,10 +334,10 @@ def course_info(request, course_id): # Construct the dates fragment 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 SelfPacedConfiguration.current().enable_course_home_improvements: + dates_fragment = CourseDatesFragmentView().render_to_fragment(request, course_id=course_id) show_upgrade_notification = False if request.GET.get('upgrade', 'false') == 'true': store_upgrade_cookie = True