2026-04-28 11:50:18 +00:00
2 changed files with 10 additions and 1 deletions
@@ -338,7 +338,9 @@ jobs:
        runs-on: docker
        container:
            image: ${{vars.docker_prefix}}python:3.13-slim
-        needs: [lint, typecheck, security, quality]
+        # unit_tests is included so coverage only runs after tests pass,
+        # preventing misleading results when tests are still in-flight or failing.
+        needs: [lint, typecheck, security, quality, unit_tests]
        steps:
            - name: Install system dependencies (nodejs for checkout, git for merge tests)
              run: |
@@ -7,6 +7,13 @@ The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).

 ### Changed

+- **CI coverage job now waits for unit_tests** (#10714): Added `unit_tests` to the
+  `needs` list of the `coverage` job in `ci.yml`. Previously the coverage job ran
+  in parallel with unit tests, which could produce misleading pass results when
+  tests were still in-flight or had already failed. Coverage now only starts after
+  unit tests succeed, eliminating redundant parallel test execution and ensuring
+  coverage results are always meaningful.
+
 - **Diagnostics spec examples expanded to all 9 providers** (#5320): Updated the
  `agents diagnostics` command examples in the specification to show all 9 supported
  providers (OpenAI, Anthropic, Google, Gemini, Azure, OpenRouter, Cohere, Groq,