← Back to issue list

ci: move staging store tests to spread

View original Github issue

Metadata

Project
snapcraft
Number
#3090
Type
pull request
State
merged
Author
sergiusens
Labels
—
Created
Updated
Closed

Current evaluation

Merged into master in commit 3a037fb11, moving staging store tests from the legacy suite to Spread and removing 1,434 lines across 27 files.

Resolution: closed resolved

Reason: PR #3090 was approved with no unresolved comments and merged into master. Commit 3a037fb11 records the migration of staging store tests to Spread.

Evaluated by: mr-cal-bot

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
stealth/space-bunny-alpha — closed resolved Merged into master in commit 3a037fb11, moving staging store tests from the legacy suite to Spread and removing 1,434 lines across 27 files.
stealth/space-bunny-alpha — closed resolved Merged into master: staging store tests moved to Spread, removing 1,434 lines across 27 files. PR #3090 had 100% diff coverage.
qwen/qwen3.6-35b-a3b — — 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.
qwen3.6-35b-a3b-mtp-q6 — — Merged into master, moving staging store tests to the spread framework. The change achieved 100% diff coverage with negligible impact on overall project coverage. CI pipeline updated accordingly.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/snapcraft#3089 (confidence 90%)

    An earlier closed PR with the same title and scope, superseded by the merged #3090 change.

Related issues

Issue Project State Summary Similarity
#3089 ci: move staging store tests to spread snapcraft closed Superseded by merged PR #3090, which moved staging store tests to Spread in commit 3a037fb11.
91%
#1290 test(spread): clean up store tests charmcraft merged Merged in commit 34ab4bdd, cleaning up store spread tests by deleting 28 lines across four files; the remaining library-test failure was fixed separately in #1292.
80%
#5954 tests: fix store tests snapcraft merged PR #5954 merged as commit 0be1e384d: store spread tests switched to new staging credentials/key test-staging-key-2 and to wheel-based automated tests (as in #5885).
77%
#3561 spread tests: store metrics against staging (CRAFT-391) snapcraft merged Merged PR #3561 via commit 152e99a, routing Spread test metrics to staging for CRAFT-391.
75%
#2276 spread: move legacy wiki tests to spread snapcraft merged Merged PR #2276 as commit 806b0b0db, moving all non-fake-server legacy wiki tests from the general integration suite to Spread.
75%
#1287 tests: minor cleanups on the spread tests snapcraft merged Merged into master as commit 1cf0f8c9f "tests: minor cleanups on the spread tests (#1287)" — a test-only tidy-up touching 4 spread test files (+16/-6), reviewed and approved by fgimenez, with coverage unchanged at 95.27%.
74%
#1460 fix(tests): make store spread tests less flaky charmcraft merged Merged PR #1460 as fd5bd6ba, making store spread tests less timing-sensitive and fixing flaky upload/release checks in #1429.
72%
#909 (qa) Spread tests that exercise all store related operations (CRAFT-1080) charmcraft merged Merged #909 in e5ac156c, adding Spread coverage for all Charmcraft store operations under CRAFT-1080 (8 files, +332/-4).
72%
#5371 test(spread): update store tests snapcraft merged Merged #5371 (commits 3067b4bac/6979dd634, CRAFT-4419): adjusted snapcraft store spread tests and CI secrets after re-provisioning, pointing metrics tests at the testing snap, base64-encoding credentials, and using new staging keys.
72%
#2301 tests: move most tests to spread and reorder travis.yaml snapcraft merged Merged PR #2301 moved most Python tests into spread, retained the store test, and reordered Travis jobs to improve test speed (commit 88005edf4).
71%