diff --git a/lms/djangoapps/branding/api.py b/lms/djangoapps/branding/api.py index 671ee63e00d66d0bbfec984f5f8f8e86a30b38ad..7cb8409e210c8e675ab32fb57f42533be22bf40e 100644 --- a/lms/djangoapps/branding/api.py +++ b/lms/djangoapps/branding/api.py @@ -587,12 +587,6 @@ def get_about_url(): def get_home_url(): """ - Lookup and return home page url, lookup is performed in the following order - - 1. return marketing root URL, If marketing is enabled - 2. Otherwise return dashboard URL. + Return Dashboard page url """ - if settings.FEATURES.get('ENABLE_MKTG_SITE', False): - return marketing_link('ROOT') - return reverse('dashboard') diff --git a/lms/djangoapps/branding/tests/test_api.py b/lms/djangoapps/branding/tests/test_api.py index 883fe4ee5d7f6a59c5cb145ae6cfa5898d8eb39c..23ee29cf542c7307a92e563418a1b6a0f5b7dff9 100644 --- a/lms/djangoapps/branding/tests/test_api.py +++ b/lms/djangoapps/branding/tests/test_api.py @@ -37,18 +37,10 @@ class TestHeader(TestCase): self.assertEqual(logo_url, cdn_url) - def test_home_url_with_mktg_disabled(self): + def test_home_url(self): expected_url = get_home_url() self.assertEqual(reverse('dashboard'), expected_url) - @mock.patch.dict('django.conf.settings.FEATURES', {'ENABLE_MKTG_SITE': True}) - @mock.patch.dict('django.conf.settings.MKTG_URLS', { - "ROOT": "https://edx.org", - }) - def test_home_url_with_mktg_enabled(self): - expected_url = get_home_url() - self.assertEqual(marketing_link('ROOT'), expected_url) - class TestFooter(TestCase): """Test retrieving the footer. """