Skip to content

Commit 8adbabb

Browse files
mgwozdz-uniconfeanil
authored andcommitted
fix: make mfe_config_api tests portable across release branches (#38497)
(cherry picked from commit d327dd3)
1 parent 75522f7 commit 8adbabb

1 file changed

Lines changed: 5 additions & 2 deletions

File tree

lms/djangoapps/mfe_config_api/tests/test_views.py

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
from rest_framework.test import APITestCase
1414

1515
from lms.djangoapps.mfe_config_api.views import mfe_name_to_app_id
16+
from openedx.core.release import doc_version
1617

1718
# Default legacy configuration values, used in tests to build a correct expected response
1819
default_legacy_config = {
@@ -24,6 +25,8 @@
2425
"ENABLE_COURSE_DISCOVERY": False,
2526
}
2627

28+
INSTRUCTOR_SUPPORT_URL = f"https://docs.openedx.org/en/{doc_version()}/educators/index.html"
29+
2730

2831
@ddt.ddt
2932
class MFEConfigTestCase(APITestCase):
@@ -313,7 +316,7 @@ def side_effect(key, default=None):
313316
data = response.json()
314317
self.assertEqual( # noqa: PT009
315318
data["SUPPORT_URL"],
316-
"https://docs.openedx.org/en/latest/educators/index.html",
319+
INSTRUCTOR_SUPPORT_URL,
317320
)
318321

319322
@patch("lms.djangoapps.mfe_config_api.views.configuration_helpers")
@@ -787,7 +790,7 @@ def side_effect(key, default=None):
787790
instructor = apps_by_id["org.openedx.frontend.app.instructorDashboard"]
788791
self.assertEqual( # noqa: PT009
789792
instructor["config"]["SUPPORT_URL"],
790-
"https://docs.openedx.org/en/latest/educators/index.html",
793+
INSTRUCTOR_SUPPORT_URL,
791794
)
792795

793796
@patch("lms.djangoapps.mfe_config_api.views.configuration_helpers")

0 commit comments

Comments
 (0)