fix: PR fixes

Author: kiram15

enterprise/filters/enrollment.py

@@ -19,12 +19,20 @@ class EnterpriseEnrollmentPostProcessor(PipelineStep):
 
     When an enterprise customer user enrolls in a course, this step calls the enterprise and consent
     API clients to post the enrollment and provide consent on behalf of the enterprise customer.
+
+    This step is intended to be registered as a pipeline step for the
+    ``org.openedx.learning.course.enrollment.started.v1`` filter.
     """
 
     def run_filter(self, user: AbstractBaseUser, course_key: Any, mode: str) -> dict[str, Any]:  # pylint: disable=arguments-differ
         """
         Post enterprise enrollment and consent if the user is an enterprise customer user.
         """
+        log.info(
+            "EnterpriseEnrollmentPostProcessor running: user_id=%s, course_key=%s",
+            user.id,
+            str(course_key),
+        )
         try:
             from openedx.features.enterprise_support.api import (  # pylint: disable=import-outside-toplevel
                 ConsentApiServiceClient,

enterprise/settings/common.py

@@ -20,6 +20,10 @@
         "fail_silently": False,
         "pipeline": ["enterprise.filters.grades.GradeEventContextEnricher"],
     },
+    "org.openedx.learning.course.enrollment.started.v1": {
+        "fail_silently": False,
+        "pipeline": ["enterprise.filters.enrollment.EnterpriseEnrollmentPostProcessor"],
+    },
 }
 
 

tests/filters/test_enrollment.py

@@ -46,7 +46,7 @@ class TestEnterpriseEnrollmentPostProcessor(TestCase):
 
     def _make_step(self):
         return EnterpriseEnrollmentPostProcessor(
-            "org.openedx.learning.course.enrollment.created.v1",
+            "org.openedx.learning.course.enrollment.started.v1",
             [],
         )
 

tests/test_enterprise/test_settings.py

@@ -7,7 +7,7 @@
 import ddt
 import pytest
 
-from enterprise.settings.common import _merge_filters_config, plugin_settings
+from enterprise.settings.common import ENTERPRISE_FILTERS_CONFIG, _merge_filters_config, plugin_settings
 
 
 class TestPluginSettingsPipelineInjection(unittest.TestCase):
@@ -212,3 +212,29 @@ def test_additions_dict_isolated_from_subsequent_mutation(self):
         existing[FILTER_A]['pipeline'].append(STEP_Y)
 
         assert additions[FILTER_A]['pipeline'] == [STEP_X]
+
+
+class TestEnterpriseFiltersConfig(unittest.TestCase):
+    """
+    Smoke tests asserting that ``ENTERPRISE_FILTERS_CONFIG`` contains the expected
+    filter registrations.  These tests catch omissions when a new pipeline step is
+    added to ``enterprise/filters/`` but its filter-type key is never registered.
+    """
+
+    def test_plugin_settings_injects_all_enterprise_filters(self):
+        """
+        plugin_settings() should inject every filter key and pipeline step from
+        ENTERPRISE_FILTERS_CONFIG into OPEN_EDX_FILTERS_CONFIG.
+        """
+        settings = SimpleNamespace(
+            ENABLE_ENTERPRISE_INTEGRATION=True,
+            OPEN_EDX_FILTERS_CONFIG={},
+        )
+        plugin_settings(settings)
+
+        for filter_key, expected_filter_config in ENTERPRISE_FILTERS_CONFIG.items():
+            assert filter_key in settings.OPEN_EDX_FILTERS_CONFIG
+            actual_filter_config = settings.OPEN_EDX_FILTERS_CONFIG[filter_key]
+            assert actual_filter_config.get("fail_silently") == expected_filter_config.get("fail_silently")
+            for expected_step in expected_filter_config.get("pipeline", []):
+                assert expected_step in actual_filter_config.get("pipeline", [])