Commit 908d5f91 by Giulio Gratta Committed by Clinton Blackburn

Makes regen_user fn respect ENABLE_OPENBADGES

- Uses pre-existing function to check if badging is enabled
- Patch ENABLE_OPENBADGES for failing test
parent 5026b4f9
...@@ -9,6 +9,7 @@ from django.core.management.base import BaseCommand, CommandError ...@@ -9,6 +9,7 @@ from django.core.management.base import BaseCommand, CommandError
from opaque_keys.edx.keys import CourseKey from opaque_keys.edx.keys import CourseKey
from badges.events.course_complete import get_completion_badge from badges.events.course_complete import get_completion_badge
from badges.utils import badges_enabled
from certificates.api import regenerate_user_certificates from certificates.api import regenerate_user_certificates
from xmodule.modulestore.django import modulestore from xmodule.modulestore.django import modulestore
...@@ -100,7 +101,7 @@ class Command(BaseCommand): ...@@ -100,7 +101,7 @@ class Command(BaseCommand):
course_id course_id
) )
if course.issue_badges: if badges_enabled() and course.issue_badges:
badge_class = get_completion_badge(course_id, student) badge_class = get_completion_badge(course_id, student)
badge = badge_class.get_for_user(student) badge = badge_class.get_for_user(student)
......
...@@ -169,6 +169,7 @@ class RegenerateCertificatesTest(CertificateManagementTest): ...@@ -169,6 +169,7 @@ class RegenerateCertificatesTest(CertificateManagementTest):
@ddt.data(True, False) @ddt.data(True, False)
@override_settings(CERT_QUEUE='test-queue') @override_settings(CERT_QUEUE='test-queue')
@patch.dict('django.conf.settings.FEATURES', {'ENABLE_OPENBADGES': True})
@patch('certificates.api.XQueueCertInterface', spec=True) @patch('certificates.api.XQueueCertInterface', spec=True)
def test_clear_badge(self, issue_badges, xqueue): def test_clear_badge(self, issue_badges, xqueue):
""" """
......
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