Fixing Store integration tests.
Metadata
Current evaluation
Merged in canonical/snapcraft#1247 as 03105a6c4, updating Store integration tests and registration error handling for the new 40-character snap-name limit.
Resolution: closed resolved
Reason: The pull request was merged as commit 03105a6c4. It adapted Store integration tests and registration error handling to the Store's 40-character snap-name limit.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Coping with recently added snap-name size limit (40-chars) in Store.
Pending more specific error response from the store. See LP: #1681545
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged in canonical/snapcraft#1247 as 03105a6c4, updating Store integration tests and registration error handling for the new 40-character snap-name limit. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #1247 as commit 03105a6c4, updating Store integration tests and registration error handling for the new 40-character snap-name limit. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged updates to Store integration tests accommodating a new 40-character snap-name size limit. Test prefixes and configurations were adjusted to pass CI, with further error handling improvements planned. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to fix Store integration tests for a new 40-character snap-name limit. The contributor updated test prefixes to snapcrafttest per reviewer feedback, resolving failures caused by the store's updated constraints. |
Update history
No update history recorded yet.
Related work
-
Related To:
Launchpad#1681545
(confidence 95%)
Store-side tracking item cited while awaiting a more specific error response.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1964 Fix Store integration tests with updated snap name registration error messages (take 2) | snapcraft | merged | Merged in canonical/snapcraft#1964 as commit 44ecb321a, updating Store integration tests for new snap-name registration errors; it supersedes #1963. | |
| #1963 Fix Store integration tests with updated snap name registration error messages | snapcraft | closed | PR #1963 (maxiberta) updated Store integration tests for new snap-name validation errors but was closed unmerged by the author on 2018-02-28 in favour of #1964, which merged as commit 44ecb321a. | |
| #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. | |
| #839 Fixing 'sign-build' integration tests. | snapcraft | merged | Merged in canonical/snapcraft#839 as commit b67a60d58, fixing sign-build integration tests by registering snap names and exposing them through the fake account-info endpoint. | |
| #585 Improve the store errors returning exceptions | snapcraft | merged | Merged in canonical/snapcraft#585 via commit ddbbdbd06, refactoring store error handling to raise exceptions and addressing LP #1594636. | |
| #1676725 staging integration tests for registration are failing | snapcraft (launchpad) | closed | Resolved by updating store registration-window tests for the 10-names-per-10-minutes rule in snapcraft PR #1224 (commit c76e66be5). | |
| #1797 tests: update the snap name already registered for store tests | snapcraft | merged | Updated store tests to use a currently registered, non-revoked snap after the previous test name was revoked; merged as commit 4cf4f33a. | |
| #1600078 snapcraft upload store integration tests is failing because it needs a manual review | snapcraft (launchpad) | closed | Store upload integration test failed with StoreReviewError (manual review). Fixed in canonical/snapcraft#639 (commit ab1b1f8f2) by using a test snap eligible for automatic publishing. | |
| #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. | |
| #1595319 upload integration test is not using a unique name | snapcraft (launchpad) | closed | Fixed by canonical/snapcraft#595 ("Fix the store update test to register a unique name"), commit d5daf8c9c, merged June 2016 and released in snapcraft 2.12 via xenial SRU; elopio set verification-done, then closed. |