diff --git a/toolchain/mfc/test/coverage.py b/toolchain/mfc/test/coverage.py index eca6f3fae5..429129afc3 100644 --- a/toolchain/mfc/test/coverage.py +++ b/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", ] ) diff --git a/toolchain/mfc/test/test_coverage_unit.py b/toolchain/mfc/test/test_coverage_unit.py index 9b9302f84b..6327a31e17 100644 --- a/toolchain/mfc/test/test_coverage_unit.py +++ b/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