Skip to content

Commit

Permalink
fix: change variable name
Browse files Browse the repository at this point in the history
  • Loading branch information
luisfelipec95 committed Mar 5, 2024
1 parent 33a8e35 commit 3b2f939
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions lms/djangoapps/discussion/tests/test_views.py
Original file line number Diff line number Diff line change
Expand Up @@ -2280,8 +2280,8 @@ class ForumMFETestCase(ForumsEnableMixin, SharedModuleStoreTestCase):
Tests that the MFE upgrade banner and MFE is shown in the correct situation with the correct UI
"""

features_with_enable_mfe_settings = settings.FEATURES.copy()
features_with_enable_mfe_settings['ENABLE_MFE_FOR_TESTING'] = True
FEATURES_WITH_DISCUSSION_MFE_ENABLED = settings.FEATURES.copy()
FEATURES_WITH_DISCUSSION_MFE_ENABLED['ENABLE_MFE_FOR_TESTING'] = True

def setUp(self):
super().setUp()
Expand All @@ -2290,7 +2290,7 @@ def setUp(self):
self.staff_user = AdminFactory.create()
CourseEnrollmentFactory.create(user=self.user, course_id=self.course.id)

@override_settings(DISCUSSIONS_MICROFRONTEND_URL="http://test.url", FEATURES=features_with_enable_mfe_settings)
@override_settings(DISCUSSIONS_MICROFRONTEND_URL="http://test.url", FEATURES=FEATURES_WITH_DISCUSSION_MFE_ENABLED)
def test_redirect_from_legacy_base_url_to_new_experience(self):
"""
Verify that the legacy url is redirected to MFE homepage when
Expand All @@ -2305,7 +2305,7 @@ def test_redirect_from_legacy_base_url_to_new_experience(self):
expected_url = f"{settings.DISCUSSIONS_MICROFRONTEND_URL}/{str(self.course.id)}"
assert response.url == expected_url

@override_settings(DISCUSSIONS_MICROFRONTEND_URL="http://test.url", FEATURES=features_with_enable_mfe_settings)
@override_settings(DISCUSSIONS_MICROFRONTEND_URL="http://test.url", FEATURES=FEATURES_WITH_DISCUSSION_MFE_ENABLED)
def test_redirect_from_legacy_profile_url_to_new_experience(self):
"""
Verify that the requested user profile is redirected to MFE learners tab when
Expand All @@ -2320,7 +2320,7 @@ def test_redirect_from_legacy_profile_url_to_new_experience(self):
expected_url = f"{settings.DISCUSSIONS_MICROFRONTEND_URL}/{str(self.course.id)}/learners"
assert response.url == expected_url

@override_settings(DISCUSSIONS_MICROFRONTEND_URL="http://test.url", FEATURES=features_with_enable_mfe_settings)
@override_settings(DISCUSSIONS_MICROFRONTEND_URL="http://test.url", FEATURES=FEATURES_WITH_DISCUSSION_MFE_ENABLED)
def test_redirect_from_legacy_single_thread_to_new_experience(self):
"""
Verify that a legacy single url is redirected to corresponding MFE thread url when the ENABLE_DISCUSSIONS_MFE
Expand Down

0 comments on commit 3b2f939

Please sign in to comment.