fix(test): force full run on src/**/include/ changes (gcov can't attribute includes)

Author: sbryngelson

toolchain/mfc/test/coverage.py

@@ -52,7 +52,13 @@ 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)
+    if any(f.startswith(ALWAYS_RUN_ALL_PREFIXES) for f in changed_files):
+        return True
+    # gcov rolls #:include'd .fpp into the parent compilation unit, so include files
+    # (inline_*.fpp, HardcodedIC, macros) are not reliably attributed in the map. Force a
+    # full run for ANY src/**/include/ change so this attribution gap can never cause
+    # under-inclusion — by rule, not by relying on the file being absent from the map.
+    return any(f.startswith("src/") and "/include/" in f and f.endswith(".fpp") for f in changed_files)
 
 
 def load_map(path: Path) -> Tuple[Optional[dict], Optional[dict]]:

toolchain/mfc/test/test_coverage_unit.py

@@ -261,3 +261,9 @@ def test_changed_files_explicit_space_and_comma_separated():
 
     assert get_changed_files("/r", "master", explicit="src/a.fpp src/b.fpp") == {"src/a.fpp", "src/b.fpp"}
     assert get_changed_files("/r", "master", explicit="src/a.fpp,src/b.fpp") == {"src/a.fpp", "src/b.fpp"}
+
+
+def test_sim_include_fpp_forces_all():
+    # gcov can't reliably attribute #:include'd files; any src include change runs all.
+    assert is_always_run_all({"src/simulation/include/inline_riemann.fpp"})
+    assert is_always_run_all({"src/pre_process/include/2dHardcodedIC.fpp"})