From 8b3f285d049ff6e40132dab8133736c7fe14e346 Mon Sep 17 00:00:00 2001 From: Gregory Martin Date: Tue, 18 Jul 2017 11:13:01 -0400 Subject: [PATCH] fix tests --- .../certificates/tests/test_signals.py | 32 ++++++++++++++ .../djangoapps/signals/tests/test_handlers.py | 43 ------------------- 2 files changed, 32 insertions(+), 43 deletions(-) delete mode 100644 openedx/core/djangoapps/signals/tests/test_handlers.py diff --git a/lms/djangoapps/certificates/tests/test_signals.py b/lms/djangoapps/certificates/tests/test_signals.py index 31dd1ec0a0..05d123724f 100644 --- a/lms/djangoapps/certificates/tests/test_signals.py +++ b/lms/djangoapps/certificates/tests/test_signals.py @@ -21,6 +21,38 @@ from xmodule.modulestore.tests.django_utils import ModuleStoreTestCase from xmodule.modulestore.tests.factories import CourseFactory +class SelfGeneratedCertsSignalTest(ModuleStoreTestCase): + """ + Tests for enabling/disabling self-generated certificates according to course-pacing. + """ + + def setUp(self): + super(SelfGeneratedCertsSignalTest, self).setUp() + SelfPacedConfiguration(enabled=True).save() + self.course = CourseFactory.create(self_paced=True) + # Enable the feature + CertificateGenerationConfiguration.objects.create(enabled=True) + + def test_cert_generation_flag_on_pacing_toggle(self): + """ + Verify that signal enables or disables self-generated certificates + according to course-pacing. + """ + #self-generation of cert disables by default + self.assertFalse(certs_api.cert_generation_enabled(self.course.id)) + + _listen_for_course_pacing_changed('store', self.course.id, self.course.self_paced) + #verify that self-generation of cert is enabled for self-paced course + self.assertTrue(certs_api.cert_generation_enabled(self.course.id)) + + self.course.self_paced = False + self.store.update_item(self.course, self.user.id) + + _listen_for_course_pacing_changed('store', self.course.id, self.course.self_paced) + # verify that self-generation of cert is disabled for instructor-paced course + self.assertFalse(certs_api.cert_generation_enabled(self.course.id)) + + class SelfGeneratedCertsSignalTest(ModuleStoreTestCase): """ Tests for enabling/disabling self-generated certificates according to course-pacing. diff --git a/openedx/core/djangoapps/signals/tests/test_handlers.py b/openedx/core/djangoapps/signals/tests/test_handlers.py deleted file mode 100644 index 872c66cea9..0000000000 --- a/openedx/core/djangoapps/signals/tests/test_handlers.py +++ /dev/null @@ -1,43 +0,0 @@ -""" -Unit tests for enabling self-generated certificates for self-paced courses -and disabling for instructor-paced courses. -""" - -from certificates import api as certs_api -from certificates.models import CertificateGenerationConfiguration -from openedx.core.djangoapps.self_paced.models import SelfPacedConfiguration -from xmodule.modulestore.tests.django_utils import ModuleStoreTestCase -from xmodule.modulestore.tests.factories import CourseFactory -from openedx.core.djangoapps.signals.handlers import _listen_for_course_pacing_changed - - -class SelfGeneratedCertsSignalTest(ModuleStoreTestCase): - """ - Tests for enabling/disabling self-generated certificates according to course-pacing. - """ - - def setUp(self): - super(SelfGeneratedCertsSignalTest, self).setUp() - SelfPacedConfiguration(enabled=True).save() - self.course = CourseFactory.create(self_paced=True) - # Enable the feature - CertificateGenerationConfiguration.objects.create(enabled=True) - - def test_cert_generation_flag_on_pacing_toggle(self): - """ - Verify that signal enables or disables self-generated certificates - according to course-pacing. - """ - #self-generation of cert disables by default - self.assertFalse(certs_api.cert_generation_enabled(self.course.id)) - - _listen_for_course_pacing_changed('store', self.course.id, self.course.self_paced) - #verify that self-generation of cert is enabled for self-paced course - self.assertTrue(certs_api.cert_generation_enabled(self.course.id)) - - self.course.self_paced = False - self.store.update_item(self.course, self.user.id) - - _listen_for_course_pacing_changed('store', self.course.id, self.course.self_paced) - # verify that self-generation of cert is disabled for instructor-paced course - self.assertFalse(certs_api.cert_generation_enabled(self.course.id))