From 92bc7ba7219b8c3b10c438b9bf22aada06b15ebc Mon Sep 17 00:00:00 2001 From: Seongho Bae Date: Thu, 3 Sep 2026 23:43:29 +0900 Subject: [PATCH] fix(tests): sync stale runner-count assertion with #1805's consolidation test_quality_workflow_pins_supported_runner_images still asserted "runs-on: ubuntu-24.04" appears twice, from before #1805 combined the quality workflow's two jobs into one (docs/doctoring/exact-artifact-sbom -quality-runner-consolidation-20260903.md). #1805 added a new dedicated test_exact_artifact_quality_single_runner.py for the consolidated shape but left this older, now-stale assertion in place, failing on current main for anyone running the full suite -- confirmed against a fresh clean-main clone with no other changes. --- tests/test_exact_artifact_sbom_attestation_contract.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/tests/test_exact_artifact_sbom_attestation_contract.py b/tests/test_exact_artifact_sbom_attestation_contract.py index 08fa9b1460..0710102cb5 100644 --- a/tests/test_exact_artifact_sbom_attestation_contract.py +++ b/tests/test_exact_artifact_sbom_attestation_contract.py @@ -253,7 +253,10 @@ def test_quality_workflow_pins_supported_runner_images() -> None: """Keep exact supply-chain evidence on an explicit runner image.""" workflow = _required_text(QUALITY_WORKFLOW, "attestation quality workflow") assert "ubuntu-latest" not in workflow - assert workflow.count("runs-on: ubuntu-24.04") == 2 + # One job, not two, since the 20260903 single-runner consolidation + # (test_exact_artifact_quality_single_runner.py owns that invariant in + # detail); this assertion only needs the image itself to stay pinned. + assert workflow.count("runs-on: ubuntu-24.04") == 1 def test_doctoring_records_claim_boundary_recovery_and_primary_sources() -> None: