tests: fix status test for staging store
Metadata
Current evaluation
Merged into master after approval and passing CI. The change corrected the staging store status test with a five-line update across one file, resulting in negligible coverage impact.
Suggested action: —
No scores available.
Issue body
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `./runtests.sh static`?
- [ ] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged into master after approval and passing CI. The change corrected the staging store status test with a five-line update across one file, resulting in negligible coverage impact. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master. Fixes the status test for the staging store. Codecov reported a negligible coverage decrease of less than 0.01% upon integration. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master, fixing the staging store status test. Codecov reports a negligible coverage decrease of less than 0.01% upon integration. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #5954 tests: fix store tests | snapcraft | merged | Merged after approval. Updated store tests with new staging credentials and switched to wheels for automated spread tests. CI passed, with unrelated spread failures noted by the author. | |
| #3090 ci: move staging store tests to spread | snapcraft | merged | Merged into master. The change migrated staging store tests to the spread testing framework, removing 1434 lines across 27 files. Approved by two reviewers, passed CI, and achieved 100% diff coverage prior to integration. | |
| #2789 tests: update the push and release tests for staging | snapcraft | merged | Merged after reviewer approval. Updates push and release tests for the staging Snap Store to reflect the store now handling releases automatically instead of displaying a ready message. Minimal one-line change. | |
| #4133 tests: ensure test_status tests are self-contained | snapcraft | merged | Merged into main after review approval and passing CI checks. Resolves #4132 by ensuring test_status tests are self-contained. The single-file change adds +3/-3 lines, achieves 100% diff coverage, and leaves overall coverage unchanged. | |
| #1677365 test_store_status.StatusTestCase.test_status_staging_store is failing in staging | snapcraft (launchpad) | closed | Fixed by updating the test expectations to include the new track column in the output. The issue was closed after the test was corrected. | |
| #3561 spread tests: store metrics against staging (CRAFT-391) | snapcraft | merged | Merged spread tests to store metrics against staging. Approved, passed CI, and added four lines without affecting coverage. | |
| #1460 fix(tests): make store spread tests less flaky | charmcraft | merged | Merged following approval by two reviewers and successful CI checks. The change resolves issue #1429 by adjusting two files to reduce flakiness in store spread tests, with a net diff of +7/-4 lines. | |
| #1027 tests: fix broken unit test in master | snapcraft | merged | Merged a one-line fix resolving a broken unit test in master. Approved by one reviewer with no unresolved comments. Changes passed CI and maintained 96.37% code coverage. Integrated directly into the main branch. | |
| #778 test: don't run staging tests if the API is erroring | craft-application | merged | Merged after approval from two reviewers. The change skips staging tests when the API errors, modifying two lines across one file. CI checks passed except one unrelated snap test, which did not block the merge. | |
| #1538 tests: update the store failure tests | snapcraft | merged | Merged after approval and passing CI. Updates store failure tests by adjusting autopkgtest scripts to report final errors, sets exception exit codes to 2, and clarifies integration test execution against fake servers. |