diff --git a/openedx/features/personalized_learner_schedules/show_answer/show_answer_field_override.py b/openedx/features/personalized_learner_schedules/show_answer/show_answer_field_override.py index f4bb34ab196ec9238137d994abe029208b2a01b7..bd7bc4b0ba4ad8d47c07296b898ca16f235d682e 100644 --- a/openedx/features/personalized_learner_schedules/show_answer/show_answer_field_override.py +++ b/openedx/features/personalized_learner_schedules/show_answer/show_answer_field_override.py @@ -53,4 +53,8 @@ class ShowAnswerFieldOverride(FieldOverrideProvider): @classmethod def enabled_for(cls, course): """ Enabled only for Self-Paced courses using Personalized User Schedules. """ - return course and course.self_paced and RELATIVE_DATES_FLAG.is_enabled(course.id) + # Returning False while we figure out a bug where Course Level Show Answer settings are not + # being properly applied and are being overridden with AFTER_ALL_ATTEMPTS_OR_CORRECT + # *****IMPORTANT*****: comment the tests back in when this is re-enabled + return False + # return course and course.self_paced and RELATIVE_DATES_FLAG.is_enabled(course.id) diff --git a/openedx/features/personalized_learner_schedules/show_answer/tests/test_show_answer_override.py b/openedx/features/personalized_learner_schedules/show_answer/tests/test_show_answer_override.py index 752a8c56253960d7dca8035054af7dfb2cf6dbdf..fce7b52d2e4382ad56de97d347faf2e7ebf670dd 100644 --- a/openedx/features/personalized_learner_schedules/show_answer/tests/test_show_answer_override.py +++ b/openedx/features/personalized_learner_schedules/show_answer/tests/test_show_answer_override.py @@ -34,34 +34,34 @@ class ShowAnswerFieldOverrideTest(ModuleStoreTestCase): field_data_cache = FieldDataCache([], course.id, self.user) return get_module(self.user, request, course.location, field_data_cache, course=course) - @ddt.data(True, False) - def test_override_enabled_for(self, active): - with RELATIVE_DATES_FLAG.override(active=active): - # Instructor paced course will just have the default value - ip_course = self.setup_course() - course_module = self.get_course_module(ip_course) - self.assertEqual(course_module.showanswer, SHOWANSWER.FINISHED) + # @ddt.data(True, False) + # def test_override_enabled_for(self, active): + # with RELATIVE_DATES_FLAG.override(active=active): + # # Instructor paced course will just have the default value + # ip_course = self.setup_course() + # course_module = self.get_course_module(ip_course) + # self.assertEqual(course_module.showanswer, SHOWANSWER.FINISHED) - sp_course = self.setup_course(self_paced=True) - course_module = self.get_course_module(sp_course) - if active: - self.assertEqual(course_module.showanswer, SHOWANSWER.AFTER_ALL_ATTEMPTS_OR_CORRECT) - else: - self.assertEqual(course_module.showanswer, SHOWANSWER.FINISHED) + # sp_course = self.setup_course(self_paced=True) + # course_module = self.get_course_module(sp_course) + # if active: + # self.assertEqual(course_module.showanswer, SHOWANSWER.AFTER_ALL_ATTEMPTS_OR_CORRECT) + # else: + # self.assertEqual(course_module.showanswer, SHOWANSWER.FINISHED) - @ddt.data( - (SHOWANSWER.ATTEMPTED, SHOWANSWER.ATTEMPTED_NO_PAST_DUE), - (SHOWANSWER.CLOSED, SHOWANSWER.AFTER_ALL_ATTEMPTS), - (SHOWANSWER.CORRECT_OR_PAST_DUE, SHOWANSWER.ANSWERED), - (SHOWANSWER.FINISHED, SHOWANSWER.AFTER_ALL_ATTEMPTS_OR_CORRECT), - (SHOWANSWER.PAST_DUE, SHOWANSWER.NEVER), - (SHOWANSWER.NEVER, SHOWANSWER.NEVER), - (SHOWANSWER.AFTER_SOME_NUMBER_OF_ATTEMPTS, SHOWANSWER.AFTER_SOME_NUMBER_OF_ATTEMPTS), - (SHOWANSWER.ALWAYS, SHOWANSWER.ALWAYS), - ) - @ddt.unpack - @RELATIVE_DATES_FLAG.override(active=True) - def test_get(self, initial_value, expected_final_value): - course = self.setup_course(self_paced=True, showanswer=initial_value) - course_module = self.get_course_module(course) - self.assertEqual(course_module.showanswer, expected_final_value) + # @ddt.data( + # (SHOWANSWER.ATTEMPTED, SHOWANSWER.ATTEMPTED_NO_PAST_DUE), + # (SHOWANSWER.CLOSED, SHOWANSWER.AFTER_ALL_ATTEMPTS), + # (SHOWANSWER.CORRECT_OR_PAST_DUE, SHOWANSWER.ANSWERED), + # (SHOWANSWER.FINISHED, SHOWANSWER.AFTER_ALL_ATTEMPTS_OR_CORRECT), + # (SHOWANSWER.PAST_DUE, SHOWANSWER.NEVER), + # (SHOWANSWER.NEVER, SHOWANSWER.NEVER), + # (SHOWANSWER.AFTER_SOME_NUMBER_OF_ATTEMPTS, SHOWANSWER.AFTER_SOME_NUMBER_OF_ATTEMPTS), + # (SHOWANSWER.ALWAYS, SHOWANSWER.ALWAYS), + # ) + # @ddt.unpack + # @RELATIVE_DATES_FLAG.override(active=True) + # def test_get(self, initial_value, expected_final_value): + # course = self.setup_course(self_paced=True, showanswer=initial_value) + # course_module = self.get_course_module(course) + # self.assertEqual(course_module.showanswer, expected_final_value)