test: fix store tests
Metadata
Current evaluation
Store-test fixes were superseded by merged #5954, updating credentials and configuration after secret re-provisioning in commit 0be1e384d.
Resolution: closed superseded
Reason: PR #5370 was superseded by #5954, which merged the store-test credential and configuration updates. The replacement landed as commit 0be1e384d.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [x] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `make lint`?
- [x] Have you successfully run `make test`?
---
All of snapcraft's secrets were re-provisioned from Sergio's credentials to mine and it broke some of the store tests.
This PR adjusts the tests and the secrets.
failing test | fix |
| -------------------------------------------- | ---------------------------------------------------------------------------- |
| tests/spread/store/basic-workflow:candid | use the testing snap for `snapcraft metrics` |
| tests/spread/store/basic-workflow:legacy | use base64 encoding for the credentials |
| tests/spread/store/basic-workflow:ubuntu_one | use the testing snap for `snapcraft metrics` |
| tests/spread/store/confdbs | use a new key registered on the staging store |
| tests/spread/store/validation-sets | use a new key registered on the staging store |
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed superseded | Store-test fixes were superseded by merged #5954, updating credentials and configuration after secret re-provisioning in commit 0be1e384d. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged as #5954 via commit 0be1e384d, updating store test credentials and configuration after secret re-provisioning broke the tests. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged to fix store test failures caused by re-provisioned credentials. Updated test configurations and secrets for staging workflows, including base64 encoding and new key registrations. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to resolve store test failures caused by credential re-provisioning. Updated test suites to use staging store keys, base64 encoded credentials, and testing snaps for metrics commands. |
Update history
No update history recorded yet.
Related work
-
Superseded By:
snapcraft#5954
(confidence 99%)
Merged replacement PR fixing store tests after credentials were re-provisioned.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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). | |
| #1984 tests: update store tests user | snapcraft | merged | Merged PR #1984 (3c313dab1), updating the Store test account, removing two obsolete tests, and revising related test data, documentation, and warnings. | |
| #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. | |
| #631 Fix the store integration tests | snapcraft | merged | Merged PR #631 (commit 351d4aaac) fixed store integration tests by pausing after rapid snap registrations so later tests did not inherit the rate-limit error. | |
| #1538 tests: update the store failure tests | snapcraft | merged | Merged in #1538 via commit 0a9284cee, updating store-failure tests for autopkgtest reporting, exit-code-2 exceptions, and fake-store execution. | |
| #1541 tests: fix the TEST_STORE environment variable | snapcraft | merged | Merged in PR #1541 as commit a36d3e053, fixing TEST_STORE handling so tests no longer skip before the empty value is translated to the fake store. | |
| #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. | |
| #1223 tests: fix resource spread test | charmcraft | merged | Merged #1223 (commit 35993996), fixing the flaky store-resources spread test and advancing #1193; related store fixes landed in #1226 and #1234. | |
| #956 tests: idempotent store installs. | snapcraft | merged | PR #956 by sergiusens added idempotent store-install tests (3 files, +10 lines), approved and merged into master as commit d1ad16636. |