fix: don't show langauges without any catalog visible course

cms/models.py

@@ -91,8 +91,8 @@
     Program,
     ProgramCertificate,
     ProgramRun,
-    CourseLanguage,
 )
+from courses.utils import get_catalog_languages
 from ecommerce.models import Product
 from mitol.olposthog.features import is_enabled
 from mitxpro.features import CATALOG_LANGUAGE_FILTER
@@ -547,6 +547,7 @@ def get_context(self, request, *args, **kwargs):  # noqa: ARG002
         )
         external_program_qset = (
             ExternalProgramPage.objects.live()
+            .filter(program__live=True)
             .select_related("program", "language")
             .order_by("title")
         )
@@ -559,6 +560,7 @@ def get_context(self, request, *args, **kwargs):  # noqa: ARG002
         )
         external_course_qset = (
             ExternalCoursePage.objects.live()
+            .filter(course__live=True)
             .select_related("course", "language")
             .order_by("title")
         )
@@ -572,7 +574,6 @@ def get_context(self, request, *args, **kwargs):  # noqa: ARG002
             external_course_qset = external_course_qset.filter(
                 language__name=language_filter
             )
-
         if topic_filter != ALL_TOPICS:
             program_page_qset = program_page_qset.related_pages(topic_filter)
             external_program_qset = external_program_qset.related_pages(topic_filter)
@@ -665,13 +666,7 @@ def get_context(self, request, *args, **kwargs):  # noqa: ARG002
             ],
             show_language_filter=is_language_filter_enabled,
             selected_language=language_filter,
-            language_options=[
-                ALL_LANGUAGES,
-                *[
-                    course_language.name
-                    for course_language in CourseLanguage.objects.filter(is_active=True)
-                ],
-            ]
+            language_options=[ALL_LANGUAGES, *get_catalog_languages()]
             if is_language_filter_enabled
             else [],
         )

cms/models_test.py

@@ -92,6 +92,7 @@
 )
 from courses.models import CourseLanguage
 from ecommerce.factories import ProductFactory, ProductVersionFactory
+from mitxpro.utils import now_in_utc
 
 pytestmark = [pytest.mark.django_db]
 
@@ -266,10 +267,16 @@ def test_catalog_page_language_context(
     mocker.patch("cms.models.is_enabled", return_value=True)
     CourseLanguage.objects.all().delete()
     catalog_page = CatalogPageFactory.create()
+    now = now_in_utc()
+
     if languages:
-        CourseLanguageFactory.create_batch(
-            len(languages), name=factory.Iterator(languages)
-        )
+        for language in languages:
+            created_language = CourseLanguageFactory.create(name=language)
+            CourseRunFactory.create(
+                start_date=now + timedelta(days=1),
+                course__page__language=created_language,
+                course__program__page__language=created_language,
+            )
 
     rf = RequestFactory()
     request = rf.get(f"/?language={selected_language}")
@@ -299,6 +306,13 @@ def test_catalog_page_language_feature_flag(mocker, staff_user, is_enabled):
     CourseLanguage.objects.all().delete()
     catalog_page = CatalogPageFactory.create()
     languages = CourseLanguageFactory.create_batch(2)
+    now = now_in_utc()
+    CourseRunFactory.create_batch(
+        2,
+        start_date=now + timedelta(days=1),
+        course__page__language=factory.Iterator(languages),
+        course__program__page__language=factory.Iterator(languages),
+    )
 
     rf = RequestFactory()
     request = rf.get("/")

cms/views_test.py

@@ -502,7 +502,9 @@ def test_catalog_page_languages(  # noqa: PLR0913
     resp = client.get(f"{catalog_page.get_url()}?language={selected_language}")
 
     assert resp.status_code == status.HTTP_200_OK
-    assert resp.context_data["language_options"] == [ALL_LANGUAGES] + language_options
+    assert resp.context_data["language_options"] == [ALL_LANGUAGES] + [
+        assign_language.name
+    ]
     assert resp.context_data["selected_language"] == selected_language
     assert len(resp.context_data["course_pages"]) == expected_courses_count
     assert len(resp.context_data["program_pages"]) == expected_program_count

courses/utils.py

@@ -18,6 +18,7 @@
     Program,
     ProgramCertificate,
     ProgramEnrollment,
+    CourseLanguage,
 )
 from courseware.api import get_edx_api_course_detail_client
 from mitxpro.utils import has_equal_properties, now_in_utc
@@ -344,3 +345,29 @@ def get_catalog_course_filter(relative_filter=""):
         Q(**courseware_live_filter)
         & Q(Q(**courserun_start_date_filter) | Q(**courserun_enrollment_end_filter))
     )
+
+
+def get_catalog_languages():
+    """
+    Returns the languages that are associated with courses or programs visible in the catalog
+    """
+
+    course_languages = (
+        CourseLanguage.objects.filter(
+            Q(get_catalog_course_filter("coursepage__"))
+            | Q(
+                get_catalog_course_filter("programpage__program__courses__coursepage__")
+            )
+            | Q(get_catalog_course_filter("externalcoursepage__"))
+            | Q(
+                get_catalog_course_filter(
+                    "externalprogrampage__program__courses__coursepage__"
+                )
+            )
+        )
+        .distinct()
+        .order_by("priority")
+        .values_list("name", flat=True)
+    )
+
+    return list(course_languages)

courses/utils_test.py

@@ -2,8 +2,11 @@
 Tests for signals
 """
 
+from datetime import timedelta
+
 from unittest.mock import Mock
 
+import factory
 import pytest
 from edx_api.course_detail import CourseDetail, CourseDetails
 from requests.exceptions import HTTPError
@@ -16,13 +19,16 @@
     ProgramCertificateFactory,
     ProgramFactory,
     UserFactory,
+    CourseLanguageFactory,
 )
 from courses.models import ProgramCertificate
 from courses.utils import (
     generate_program_certificate,
     process_course_run_grade_certificate,
     sync_course_runs,
+    get_catalog_languages,
 )
+from mitxpro.utils import now_in_utc
 
 pytestmark = pytest.mark.django_db
 
@@ -208,3 +214,36 @@ def test_sync_course_runs(settings, mocker, mocked_api_response, expect_success)
     else:
         assert success_count == 0
         assert failure_count == 1
+
+
+def test_catalog_visible_languages():
+    """Test that get_catalog_languages returns the expected languages"""
+
+    catalog_visible_languages = CourseLanguageFactory.create_batch(
+        2, name=factory.Iterator(["Language1", "Language2"])
+    )
+    catalog_invisble_languages = CourseLanguageFactory.create_batch(
+        2, name=factory.Iterator(["Language3", "Language4"])
+    )
+    now = now_in_utc()
+    # Creates active courses (Course runs will create underlying course and page objects)
+    CourseRunFactory.create_batch(
+        2,
+        start_date=now + timedelta(days=1),
+        enrollment_end=now + timedelta(days=2),
+        course__page__language=factory.Iterator(catalog_visible_languages),
+        course__program__page__language=factory.Iterator(catalog_visible_languages),
+    )
+    # Creates expired courses (Course runs will create underlying course and page objects)
+    CourseRunFactory.create_batch(
+        2,
+        start_date=now - timedelta(days=1),
+        enrollment_end=now - timedelta(days=2),
+        course__page__language=factory.Iterator(catalog_invisble_languages),
+        course__program__page__language=factory.Iterator(catalog_invisble_languages),
+    )
+
+    # The expected languages are the ones that are associated with unexpuired/catalog visible courses
+    assert get_catalog_languages() == [
+        catalog_language.name for catalog_language in catalog_visible_languages
+    ]