Fix lint: remove separator comments flagged by new source linter (#1193)

Author: sbryngelson

toolchain/mfc/test/test_coverage_unit.py

@@ -21,10 +21,8 @@
 from pathlib import Path
 from unittest.mock import patch
 
-# ---------------------------------------------------------------------------
 # Import the module under test.
 # We patch the module-level imports that require the full toolchain.
-# ---------------------------------------------------------------------------
 
 
 # Create minimal stubs for toolchain modules so coverage.py can be imported
@@ -160,9 +158,7 @@ class _FakeMFCException(Exception):
     COVERAGE_CACHE_PATH = _globals["COVERAGE_CACHE_PATH"]
 
 
-# ---------------------------------------------------------------------------
 # Helper: minimal fake test case
-# ---------------------------------------------------------------------------
 
 
 class FakeCase:
@@ -175,9 +171,7 @@ def get_uuid(self) -> str:
         return self._uuid
 
 
-# ===========================================================================
 # Group 1: _parse_diff_files — git diff --name-only parsing
-# ===========================================================================
 
 
 class TestParseDiffFiles(unittest.TestCase):
@@ -211,9 +205,7 @@ def test_parse_mixed_extensions(self):
         assert "CMakeLists.txt" in result
 
 
-# ===========================================================================
 # Group 2: should_run_all_tests — ALWAYS_RUN_ALL detection
-# ===========================================================================
 
 
 class TestShouldRunAllTests(unittest.TestCase):
@@ -280,9 +272,7 @@ def test_mixed_one_trigger_fires_all(self):
         )
 
 
-# ===========================================================================
 # Group 3: filter_tests_by_coverage — core file-level selection logic
-# ===========================================================================
 
 
 class TestFilterTestsByCoverage(unittest.TestCase):
@@ -428,9 +418,7 @@ def test_incomplete_coverage_not_triggered_by_preprocess(self):
         assert len(skipped) == 1
 
 
-# ===========================================================================
 # Group 4: Corner cases from design discussion
-# ===========================================================================
 
 
 class TestDesignCornerCases(unittest.TestCase):
@@ -487,9 +475,7 @@ def test_niche_feature_pruning(self):
         assert len(skipped) == 3
 
 
-# ===========================================================================
 # Group 5: _parse_gcov_json_output — gcov JSON parsing (file-level)
-# ===========================================================================
 
 
 class TestParseGcovJsonOutput(unittest.TestCase):
@@ -650,9 +636,7 @@ def test_concatenated_json_skips_zero_coverage(self):
         assert result == {"src/simulation/m_rhs.fpp"}
 
 
-# ===========================================================================
 # Group 6: _normalize_cache — old format conversion
-# ===========================================================================
 
 
 class TestNormalizeCache(unittest.TestCase):
@@ -690,9 +674,7 @@ def test_empty_coverage_dict_becomes_empty_list(self):
         assert result["TEST_A"] == []
 
 
-# ===========================================================================
 # Group 7: Cache path format
-# ===========================================================================
 
 
 class TestCachePath(unittest.TestCase):
@@ -701,9 +683,7 @@ def test_cache_path_is_gzipped(self):
         assert str(COVERAGE_CACHE_PATH).endswith(".json.gz")
 
 
-# ===========================================================================
 # Group 8: _compute_gcov_prefix_strip
-# ===========================================================================
 
 
 class TestComputeGcovPrefixStrip(unittest.TestCase):
@@ -724,9 +704,7 @@ def test_deep_path(self):
         assert result == "5"
 
 
-# ===========================================================================
 # Group 9: load_coverage_cache
-# ===========================================================================
 
 
 class TestLoadCoverageCache(unittest.TestCase):

toolchain/mfc/viz/interactive.py

@@ -434,7 +434,6 @@ def _get_cached_3d_mesh(
         return _mesh3_cache.get(key)
 
 
-
 # (PyVista code removed — server-side rendering uses kaleido on Linux)