TEMP: disable cases.py staleness check and ALWAYS_RUN_ALL to test pruning

Author: sbryngelson

toolchain/mfc/test/coverage.py

@@ -49,7 +49,8 @@
         "src/common/include/macros.fpp",
         "src/common/include/case.fpp",
         "toolchain/mfc/test/case.py",
-        "toolchain/mfc/test/cases.py",
+        # TEMP: cases.py removed to test pruning on this PR — restore before merge
+        # "toolchain/mfc/test/cases.py",
         "toolchain/mfc/test/coverage.py",
         "toolchain/mfc/params/definitions.py",
         "toolchain/mfc/run/input.py",
@@ -664,17 +665,17 @@ def load_coverage_cache(root_dir: str) -> Optional[dict]:
         cons.print("[yellow]Warning: Coverage cache has unexpected format.[/yellow]")
         return None
 
-    cases_py = Path(root_dir) / "toolchain/mfc/test/cases.py"
-    try:
-        current_hash = hashlib.sha256(cases_py.read_bytes()).hexdigest()
-    except OSError as exc:
-        cons.print(f"[yellow]Warning: Cannot read cases.py for cache staleness check: {exc}[/yellow]")
-        return None
-    stored_hash = cache.get("_meta", {}).get("cases_hash", "")
-
-    if current_hash != stored_hash:
-        cons.print("[yellow]Warning: Coverage cache is stale (cases.py changed).[/yellow]")
-        return None
+    # TEMP: staleness check disabled to test pruning on this PR — restore before merge
+    # cases_py = Path(root_dir) / "toolchain/mfc/test/cases.py"
+    # try:
+    #     current_hash = hashlib.sha256(cases_py.read_bytes()).hexdigest()
+    # except OSError as exc:
+    #     cons.print(f"[yellow]Warning: Cannot read cases.py for cache staleness check: {exc}[/yellow]")
+    #     return None
+    # stored_hash = cache.get("_meta", {}).get("cases_hash", "")
+    # if current_hash != stored_hash:
+    #     cons.print("[yellow]Warning: Coverage cache is stale (cases.py changed).[/yellow]")
+    #     return None
 
     cache = _normalize_cache(cache)
 

toolchain/mfc/test/test_coverage_unit.py

@@ -233,7 +233,8 @@ def test_macros_fpp_triggers_all(self):
         assert should_run_all_tests({"src/common/include/macros.fpp"}) is True
 
     def test_cases_py_triggers_all(self):
-        assert should_run_all_tests({"toolchain/mfc/test/cases.py"}) is True
+        # TEMP: cases.py removed from ALWAYS_RUN_ALL for testing — restore before merge
+        assert should_run_all_tests({"toolchain/mfc/test/cases.py"}) is False
 
     def test_case_py_triggers_all(self):
         assert should_run_all_tests({"toolchain/mfc/test/case.py"}) is True
@@ -467,7 +468,8 @@ def test_non_fpp_always_run_all_detected(self):
         _parse_diff_files includes it -> should_run_all_tests fires.
         """
         files = _parse_diff_files("toolchain/mfc/test/cases.py\n")
-        assert should_run_all_tests(files) is True
+        # TEMP: cases.py removed from ALWAYS_RUN_ALL for testing — restore before merge
+        assert should_run_all_tests(files) is False
 
     def test_niche_feature_pruning(self):
         """
@@ -775,9 +777,11 @@ def test_corrupt_gzip_returns_none(self):
         assert self._run() is None
 
     def test_stale_cache_returns_none(self):
-        """Cache with wrong cases_hash -> None."""
+        """Cache with wrong cases_hash -> None (staleness check).
+        TEMP: staleness check disabled for testing — restore assertion before merge."""
         self._write_cache({"_meta": {"cases_hash": "wrong_hash"}, "TEST1": ["src/simulation/m_rhs.fpp"]})
-        assert self._run() is None
+        # TEMP: staleness check disabled, so stale cache is accepted
+        assert self._run() is not None
 
     def test_empty_cache_returns_none(self):
         """Cache with only _meta and no test entries -> None."""