Commit a2f476a3 by Dennis Jen

Disabling the learner lens still enables navigating to it.

parent e8ff7547
......@@ -79,6 +79,10 @@ class LearnersViewTests(ViewTestMixin, SwitchMixin, TestCase):
})
self.assertNotContains(response, self.TABLE_ERROR_TEXT)
def test_success_if_hidden(self):
self.toggle_switch('enable_learner_analytics', False)
self.test_success()
@data(Timeout, ConnectionError, ValueError)
def test_data_api_error(self, RequestExceptionClass):
learners_payload = {'should_not': 'return this value'}
......
......@@ -368,8 +368,10 @@ class CourseNavBarMixin(object):
# Get the active primary item and remove it from the list
primary_nav_item = None
if self.active_primary_nav_item:
primary_nav_item = [i for i in primary_nav_items if i['name'] == self.active_primary_nav_item][0]
primary_nav_items.remove(primary_nav_item)
navs = [i for i in primary_nav_items if i['name'] == self.active_primary_nav_item]
if navs:
primary_nav_item = navs[0]
primary_nav_items.remove(primary_nav_item)
context.update({
'primary_nav_item': primary_nav_item,
......
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