From 6c31268a9d61d03f980792db0f4459282620fe7e Mon Sep 17 00:00:00 2001 From: Serhii Nanai Date: Tue, 30 Sep 2025 10:07:40 +0300 Subject: [PATCH] test: use ENABLE_CATALOG_MICROFRONTEND from settings --- lms/djangoapps/branding/tests/test_views.py | 14 ++------------ lms/djangoapps/courseware/tests/test_views.py | 6 +----- 2 files changed, 3 insertions(+), 17 deletions(-) diff --git a/lms/djangoapps/branding/tests/test_views.py b/lms/djangoapps/branding/tests/test_views.py index 33626a4b47..36ebcd7350 100644 --- a/lms/djangoapps/branding/tests/test_views.py +++ b/lms/djangoapps/branding/tests/test_views.py @@ -298,13 +298,8 @@ class TestIndex(SiteMixin, TestCase): @ddt.unpack def test_index_redirects_to_mfe(self, catalog_mfe_enabled, expected_redirect): """Test that index view redirects to MFE when both flags are enabled.""" - old_features = settings.FEATURES.copy() - old_features.update({ - "ENABLE_CATALOG_MICROFRONTEND": catalog_mfe_enabled, - "COURSES_ARE_BROWSABLE": True, - }) new_settings = { - "FEATURES": old_features, + "ENABLE_CATALOG_MICROFRONTEND": catalog_mfe_enabled, "CATALOG_MICROFRONTEND_URL": "http://example.com/catalog", } with override_settings(**new_settings): @@ -339,13 +334,8 @@ class TestCourses(SiteMixin, TestCase): @ddt.unpack def test_courses_redirect_to_mfe(self, catalog_mfe_enabled, expected_redirect): """Test that courses view redirects to MFE when both flags are enabled""" - old_features = settings.FEATURES.copy() - old_features.update({ - "ENABLE_CATALOG_MICROFRONTEND": catalog_mfe_enabled, - "COURSES_ARE_BROWSABLE": True, - }) new_settings = { - "FEATURES": old_features, + "ENABLE_CATALOG_MICROFRONTEND": catalog_mfe_enabled, "CATALOG_MICROFRONTEND_URL": "http://example.com/catalog", } with override_settings(**new_settings): diff --git a/lms/djangoapps/courseware/tests/test_views.py b/lms/djangoapps/courseware/tests/test_views.py index ba85abf4b7..4e3d1be9bd 100644 --- a/lms/djangoapps/courseware/tests/test_views.py +++ b/lms/djangoapps/courseware/tests/test_views.py @@ -3410,12 +3410,8 @@ class CourseAboutViewTests(ModuleStoreTestCase): """ Test that the CourseAboutView redirects to the MFE when appropriate. """ - old_features = settings.FEATURES.copy() - old_features.update({ - "ENABLE_CATALOG_MICROFRONTEND": catalog_mfe_enabled, - }) new_settings = { - "FEATURES": old_features, + "ENABLE_CATALOG_MICROFRONTEND": catalog_mfe_enabled, "CATALOG_MICROFRONTEND_URL": "http://example.com/catalog", } with override_settings(**new_settings):