Skip to content

Commit 1b7c536

Browse files
test: pin HELP_TOKENS_VERSION in instructor-dashboard SUPPORT_URL tests
The asserted help-tokens URL includes the doc version, which varies between master ("latest") and release branches ("open-release-<name>.master"), causing the tests to fail on release branches. Pin HELP_TOKENS_VERSION to "latest" so the tests are deterministic regardless of which branch they run on. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent c53db77 commit 1b7c536

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

lms/djangoapps/mfe_config_api/tests/test_views.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -297,6 +297,7 @@ def side_effect(key, default=None):
297297
# Value in original MFE_CONFIG not overridden by catalog config should be preserved
298298
self.assertEqual(data["PRESERVED_SETTING"], "preserved") # noqa: PT009
299299

300+
@override_settings(HELP_TOKENS_VERSION="latest")
300301
@patch("lms.djangoapps.mfe_config_api.views.configuration_helpers")
301302
def test_legacy_overrides_instructor_dashboard(self, configuration_helpers_mock):
302303
"""Legacy help-tokens SUPPORT_URL is included for instructor-dashboard when no explicit override is set."""
@@ -769,6 +770,7 @@ def side_effect(key, default=None):
769770
brand_new = apps_by_id["org.openedx.frontend.app.brand.new"]["config"]
770771
self.assertEqual(brand_new["BRAND_NEW_KEY"], "value") # noqa: PT009
771772

773+
@override_settings(HELP_TOKENS_VERSION="latest")
772774
@patch("lms.djangoapps.mfe_config_api.views.configuration_helpers")
773775
def test_legacy_overrides_instructor_dashboard_support_url(self, configuration_helpers_mock):
774776
"""Instructor dashboard gets SUPPORT_URL from help-tokens when no explicit override is set."""

0 commit comments

Comments
 (0)