Skip to content

fix(tests): use whitelisted smoke job in quantum integration test#461

Draft
cursor[bot] wants to merge 2 commits into
mainfrom
cursor/repository-automation-efc4
Draft

fix(tests): use whitelisted smoke job in quantum integration test#461
cursor[bot] wants to merge 2 commits into
mainfrom
cursor/repository-automation-efc4

Conversation

@cursor

@cursor cursor Bot commented Jun 20, 2026

Copy link
Copy Markdown

Summary

Hourly repo health automation failed on pre_commit_check because tests/test_quantum_integration.py used the non-whitelisted job name another. The quantum integration bridge only accepts baseline, benchmark, nightly, and smoke, so the test never reached status lookup logic.

Changes

  • Updated test_run_autorun_job_reads_status_for_list_and_dict_shapes to use the whitelisted smoke job name in fixtures and assertions.

Verification

  • pytest tests/test_quantum_integration.py -v — 2 passed
  • Full health cycle: PYTHON_BIN=.venv/bin/python scripts/repo_health_automation.py --once --repair-status --refresh-stale-status --run-agents --continue-on-fail — all steps passed (2725 unit tests, integration contract gate, repo agents)

Checklist

  • I added or updated tests for new behavior
  • I updated relevant documentation (README, .github/copilot-instructions.md, instruction files)
  • All unit tests pass locally (python scripts/test_runner.py --unit)
  • No hardcoded secrets or API keys introduced
  • PR title clearly describes the change (not a generic codespace/branch name)
Open in Web View Automation 

Summary by Sourcery

Bug Fixes:

  • Update quantum integration autorun job test to use the supported 'smoke' job name instead of a non-whitelisted job, ensuring the test passes and validates status handling.

The run_autorun_job bridge validates job names against baseline,
benchmark, nightly, and smoke. Update list/dict status lookup test to
use smoke instead of another so pre-commit unit tests pass.

Co-authored-by: Bryan
Update tracked data_out status files from successful hourly repo health
automation run (2725 tests passed, integration gate and agents ok).

Co-authored-by: Bryan
@sourcery-ai

sourcery-ai Bot commented Jun 20, 2026

Copy link
Copy Markdown
Contributor
Reviewer's guide (collapsed on small PRs)

Reviewer's Guide

Adjusts the quantum integration test to use a whitelisted job name ('smoke') so the autorun job status lookup logic is exercised correctly for both list and dict job payload shapes.

File-Level Changes

Change Details Files
Update quantum integration autorun job test fixtures and assertions to use the whitelisted 'smoke' job name instead of the non-whitelisted 'another'.
  • Change parametrized jobs_payload fixtures to use 'smoke' as the job name in both list and dict payload shapes while keeping preset/status the same.
  • Update the call to integration.run_autorun_job to pass 'smoke' as the job name so the bridge accepts and processes the job.
  • Adjust assertions to expect 'smoke' for job_name and status.name while still validating preset and status structure.
tests/test_quantum_integration.py

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@github-actions github-actions Bot added the tests label Jun 20, 2026
@github-actions

Copy link
Copy Markdown

🟡 Coverage — 7b3a876

Metric Value
Total coverage 65.2%
→ vs main 0.0%
Minimum threshold 60%

Updated on every push · 2026-06-20

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant