ci: remove case_validator.py from ALWAYS_RUN_ALL for smarter pruning (#1330)

Author: sbryngelson

toolchain/mfc/test/coverage.py

@@ -44,6 +44,8 @@
 #     are conservatively included (not in cache -> always runs).
 #   - definitions.py: adding a parameter doesn't affect tests that don't use it;
 #     the PR's .fpp changes trigger the relevant tests via coverage overlap.
+#   - case_validator.py: validation only affects user-facing error messages for
+#     invalid configs, not test outputs (tests use valid configs).
 ALWAYS_RUN_ALL = frozenset(
     [
         "CMakeLists.txt",
@@ -56,7 +58,6 @@
         "toolchain/mfc/test/case.py",
         "toolchain/mfc/test/coverage.py",
         "toolchain/mfc/run/input.py",
-        "toolchain/mfc/case_validator.py",
     ]
 )
 

toolchain/mfc/test/test_coverage_unit.py

@@ -237,8 +237,9 @@ def test_definitions_py_does_not_trigger_all(self):
     def test_input_py_triggers_all(self):
         assert should_run_all_tests({"toolchain/mfc/run/input.py"}) is True
 
-    def test_case_validator_triggers_all(self):
-        assert should_run_all_tests({"toolchain/mfc/case_validator.py"}) is True
+    def test_case_validator_does_not_trigger_all(self):
+        """case_validator.py removed: validation only affects error messages, not test outputs."""
+        assert should_run_all_tests({"toolchain/mfc/case_validator.py"}) is False
 
     def test_cmakelists_triggers_all(self):
         assert should_run_all_tests({"CMakeLists.txt"}) is True