Skip to content

test: cover artifact registry historical row status - #58

Draft
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/test-coverage-regression-prevention-5a5e
Draft

test: cover artifact registry historical row status#58
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/test-coverage-regression-prevention-5a5e

Conversation

@cursor

@cursor cursor Bot commented Jun 3, 2026

Copy link
Copy Markdown

Pull Request Template

Summary

Adds a focused regression test for artifact registry status aggregation when historical duplicate rows exist for an artifact type.

Why

Risky behavior now covered: report artifact bundle completeness must be driven by the newest registry row per artifact type. A stale older complete row must not mask a newer missing row and incorrectly mark a campaign artifact bundle complete.

These tests materially reduce regression risk because artifact completeness feeds trust/reporting surfaces after campaign finalization, and the recent artifact registry extraction centralized this status projection.

Scope

  • Narrow change
  • Broad change

Test files added/updated:

  • test_artifact_contract.py

Risk

Test-only change. It exercises SQLite artifact registry rows and trust identity summary behavior without changing production code.

Validation

  • Dev contract verified (.\.venv\Scripts\python.exe .agent\scripts\helpers\verify_dev_contract.py --quick or CI equivalent)
  • Tests added or updated
  • Existing tests passed
  • Manual validation performed
  • Not validated yet

Validated with:

  • .venv/bin/python -m ruff check test_artifact_contract.py
  • QUANTMAP_LAB_ROOT=/tmp/quantmap-lab .venv/bin/python -m pytest -q test_artifact_contract.py::test_artifact_registry_latest_row_controls_bundle_status test_artifact_contract.py (27 passed)

Agent Surface

  • Instruction files reviewed if behavior/tooling changed
  • Workspace settings changes are intentional and minimal
  • If .agent/scripts/agent_surface_audit.py is present in this branch, agent surface audit passed

Notes

No production behavior changed.

Open in Web View Automation 

Co-authored-by: Joshua L. Davis Sr. <Mad-Labs42@users.noreply.github.com>
@sonarqubecloud

sonarqubecloud Bot commented Jun 3, 2026

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant