Commit e67975b2 by Will Daly

Feature flag credit provider messaging on the dashboard.

When the credit eligibility is disabled, skip the code
that access the CreditProvider table.  This will allow us
to safely delete a column from the table in rc/2015-07-08.
parent 39334db4
...@@ -531,7 +531,10 @@ def dashboard(request): ...@@ -531,7 +531,10 @@ def dashboard(request):
for course, __ in course_enrollment_pairs: for course, __ in course_enrollment_pairs:
enrolled_courses_dict[unicode(course.id)] = course enrolled_courses_dict[unicode(course.id)] = course
credit_messages = _create_credit_availability_message(enrolled_courses_dict, user) if settings.FEATURES.get("ENABLE_CREDIT_ELIGIBILITY"):
credit_messages = _create_credit_availability_message(enrolled_courses_dict, user)
else:
credit_messages = {}
course_optouts = Optout.objects.filter(user=user).values_list('course_id', flat=True) course_optouts = Optout.objects.filter(user=user).values_list('course_id', flat=True)
......
...@@ -5,6 +5,8 @@ import unittest ...@@ -5,6 +5,8 @@ import unittest
import datetime import datetime
import ddt import ddt
import pytz import pytz
from mock import patch
from django.test import TestCase from django.test import TestCase
from django.test.utils import override_settings from django.test.utils import override_settings
from django.db import connection, transaction from django.db import connection, transaction
...@@ -697,6 +699,7 @@ class CreditMessagesTests(ModuleStoreTestCase, CreditApiTestBase): ...@@ -697,6 +699,7 @@ class CreditMessagesTests(ModuleStoreTestCase, CreditApiTestBase):
request_status = api.get_credit_request_status(self.student.username, self.course.id) request_status = api.get_credit_request_status(self.student.username, self.course.id)
self.assertEqual(len(request_status), 0) self.assertEqual(len(request_status), 0)
@patch.dict(settings.FEATURES, {"ENABLE_CREDIT_ELIGIBILITY": True})
def test_credit_messages(self): def test_credit_messages(self):
self._set_creditcourse() self._set_creditcourse()
......
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