fix: add parallel UT coverage reporting and recover Codecov (#3323)

* Revert "fix: Parallel unit test runner with file-level subprocess isolation (#3285)"

This reverts commit b6e59cf.

* fix: restore parallel UT runner with coverage reporting

* fix: pin codecov action for sonar

* ci: add repo controlled codecov gates

Author: JasonW404

.github/workflows/auto-unit-test.yml

@@ -19,6 +19,10 @@ on:
         description: 'per-file timeout in seconds (0=disabled)'
         required: false
         default: '300'
+      codecov_os:
+        description: 'Codecov uploader OS override (e.g. linux-arm64, linux, macos, windows)'
+        required: false
+        default: 'linux-arm64'
   pull_request:
     branches: [develop, 'release/**', 'hotfix/**']
     paths:
@@ -95,11 +99,13 @@ jobs:
           fi
 
       - name: Upload coverage to Codecov
-        uses: codecov/codecov-action@v4
+        # codecov/codecov-action@v7
+        uses: codecov/codecov-action@fb8b3582c8e4def4969c97caa2f19720cb33a72f
         with:
           files: test/coverage.xml
           token: ${{ secrets.CODECOV_TOKEN }}
           flags: unittests
           name: codecov-umbrella
-          fail_ci_if_error: false
+          fail_ci_if_error: true
+          os: ${{ github.event.inputs.codecov_os || 'linux-arm64' }}
           verbose: true

codecov.yml

@@ -0,0 +1,22 @@
+coverage:
+  precision: 2
+  round: down
+  range: "90...100"
+  status:
+    project:
+      default:
+        target: auto
+        threshold: 0%
+        if_ci_failed: error
+        only_pulls: true
+    patch:
+      default:
+        target: 90%
+        threshold: 0%
+        if_ci_failed: error
+        only_pulls: true
+
+comment:
+  layout: "diff, flags, files"
+  behavior: default
+  require_changes: true

test/run_all_test.py

@@ -271,8 +271,7 @@ def generate_error_report(results: list[dict]) -> None:
 
 def _combine_coverage(current_dir: Path, project_root: Path) -> bool:
     coverage_data_file = current_dir / ".coverage"
-    coverage_xml_file = current_dir / "coverage.xml"
-    for path in (coverage_data_file, coverage_xml_file):
+    for path in (coverage_data_file,):
         if path.exists():
             path.unlink()
 
@@ -282,23 +281,12 @@ def _combine_coverage(current_dir: Path, project_root: Path) -> bool:
         str(coverage_data_file),
         str(current_dir),
     )
-    xml_cmd = _coverage_command(
-        "xml",
-        "-o",
-        str(coverage_xml_file),
-        "--data-file",
-        str(coverage_data_file),
-    )
     coverage_env = _coverage_env(project_root)
     combine = subprocess.run(combine_cmd, cwd=project_root, env=coverage_env, text=True, capture_output=True)
     if combine.returncode != 0:
         logger.error("Coverage combine failed:\n%s\n%s", combine.stdout, combine.stderr)
         return False
-    xml = subprocess.run(xml_cmd, cwd=project_root, env=coverage_env, text=True, capture_output=True)
-    if xml.returncode != 0:
-        logger.error("Coverage XML generation failed:\n%s\n%s", xml.stdout, xml.stderr)
-        return False
-    logger.info("Coverage XML file generated: %s", coverage_xml_file)
+    logger.info("Coverage data combined: %s", coverage_data_file)
     return True
 
 
@@ -318,6 +306,9 @@ def _report_coverage(current_dir: Path) -> bool:
         html_dir = current_dir / "coverage_html"
         cov.html_report(directory=str(html_dir))
         logger.info("\nHTML coverage report generated in: %s", html_dir)
+        xml_file = current_dir / "coverage.xml"
+        cov.xml_report(outfile=str(xml_file))
+        logger.info("XML coverage report generated: %s", xml_file)
     except Exception as exc:
         logger.error("Coverage report failed: %s", exc)
         return False