feat(test): ALWAYS_RUN_ALL classification for coverage selection

Author: sbryngelson

toolchain/mfc/test/coverage.py

@@ -38,6 +38,22 @@ def save_map(path: Path, entries: dict, *, n_tests: int, git_sha: str, gfortran_
         json.dump(payload, f, indent=2, sort_keys=True)
 
 
+ALWAYS_RUN_ALL_EXACT = frozenset(["CMakeLists.txt"])
+ALWAYS_RUN_ALL_PREFIXES = (
+    "src/common/include/",  # GPU/Fypp macro & include files (CPU map can't line-attribute)
+    "toolchain/cmake/",  # build system
+    "toolchain/mfc/params/",  # parameter codegen -> emits Fortran broadly
+    "toolchain/bootstrap/",  # build/run scripts
+)
+
+
+def is_always_run_all(changed_files: set) -> bool:
+    """True if any changed file forces the full suite (un-attributable by the CPU map)."""
+    if changed_files & ALWAYS_RUN_ALL_EXACT:
+        return True
+    return any(f.startswith(ALWAYS_RUN_ALL_PREFIXES) for f in changed_files)
+
+
 def load_map(path: Path) -> Tuple[Optional[dict], Optional[dict]]:
     """Return (entries_without_meta, meta), or (None, None) if missing/corrupt."""
     if not Path(path).exists():

toolchain/mfc/test/test_coverage_unit.py

@@ -1,7 +1,7 @@
 import tempfile
 from pathlib import Path
 
-from mfc.test.coverage import load_map, param_hash, save_map
+from mfc.test.coverage import is_always_run_all, load_map, param_hash, save_map
 
 
 def test_param_hash_is_order_independent():
@@ -46,3 +46,24 @@ def test_load_corrupt_returns_none():
         p = Path(d) / "m.json.gz"
         p.write_bytes(b"not gzip")
         assert load_map(p) == (None, None)
+
+
+def test_macro_file_forces_all():
+    assert is_always_run_all({"src/common/include/parallel_macros.fpp"})
+
+
+def test_cmake_forces_all():
+    assert is_always_run_all({"CMakeLists.txt"})
+    assert is_always_run_all({"toolchain/cmake/foo.cmake"})
+
+
+def test_param_codegen_forces_all():
+    assert is_always_run_all({"toolchain/mfc/params/definitions.py"})
+
+
+def test_ordinary_common_module_does_not_force_all():
+    assert not is_always_run_all({"src/common/m_helper.fpp"})
+
+
+def test_ordinary_sim_module_does_not_force_all():
+    assert not is_always_run_all({"src/simulation/m_rhs.fpp"})