← Back to issue list

tests: update the snap name already registered for store tests

View original Github issue

Metadata

Project
snapcraft
Number
#1797
Type
pull request
State
merged
Author
come-maiz
Labels
—
Created
Updated
Closed

Current evaluation

Updated store tests to use a currently registered, non-revoked snap after the previous test name was revoked; merged as commit 4cf4f33a.

Resolution: closed resolved

Reason: PR #1797 was merged into Snapcraft, resolving the stale store-test scenario. Commit 4cf4f33a changed the test to use an active snap name that the store no longer reports as revoked.

Evaluated by: mr-cal-bot

No scores available.

Issue body

The previous snap name we were using for this test was revoked at some point. The store is now returning a different message for revoked snaps, so we needed to refresh this sceneario with a new name registered by someone else, but not revoked. - [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)? - [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)? - [x] Have you successfully run `./runtests.sh static`? - [x] Have you successfully run `./runtests.sh unit`? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Updated store tests to use a currently registered, non-revoked snap after the previous test name was revoked; merged as commit 4cf4f33a.
stealth/space-bunny-alpha — closed resolved Store tests were updated to use an active, non-revoked snap name after the prior test snap was revoked; PR #1797 merged as commit 4cf4f33a.
stealth/space-bunny-alpha — closed resolved Merged in #1797 via commit 4cf4f33a, updating store tests to use an active snap name after the previous name was revoked.
qwen/qwen3.6-35b-a3b — — Merged test update replacing a revoked snap name with an active one to match current store responses. Transient CI timeouts were retried before final merge.
qwen3.6-35b-a3b-mtp-q6 — — Merged. Updated a revoked snap name in store tests to a new registered name to fix failing assertions. Transient CI failures from LXD crashes and HTTP 504 errors were retried before successful merge.

Update history

No update history recorded yet.

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.
77%
#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.
75%
#1799 tests: update the registered snap fake snapcraft merged Merged #1799, a test-only update to the registered snap fake changing four lines across two files; it received one approval with no unresolved comments.
74%
#2086 tests: update metadata store integration test, no previous push required snapcraft merged Merged PR #2086 in commit 3882853b7, updating the metadata-store integration test for approved snap names whose metadata can be changed before any revision push.
74%
#1736752 Add a store test for registering a name that was revoked snapcraft (launchpad) closed Closed as stale in August 2026 without adding the requested revoked-name store integration test; no resolving Snapcraft PR or commit was identified.
73%
#2351 tests: use valid snap names in unit tests snapcraft closed Closed unmerged; the same test-only change was merged instead via PR #2352 (commit b8c0a4fbf), updating unit-test snap names to pass `snap pack --check-skeleton`.
72%
#1247 Fixing Store integration tests. snapcraft merged Merged in canonical/snapcraft#1247 as 03105a6c4, updating Store integration tests and registration error handling for the new 40-character snap-name limit.
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%
#2352 tests: use valid snap names in unit tests snapcraft merged Merged PR #2352 (commit b8c0a4fbf) changed snap names used in unit tests so they pass snap pack --check-skeleton; author merged directly as a minor test-only change.
72%
#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.
71%