← 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

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.

Suggested action:

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
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.
qwen3.6-35b-a3b-mtp-q6 Merged after updating the test fixture to use a valid, non-revoked snap name, as the previous name was revoked by the store. Transient CI failures from LXD crashes and HTTP 504 errors were resolved on retry.

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 after approval, updating Store integration tests to match new snap name registration error messages. Builds on PR #1963, modifying string literals across three files. Resolved with zero outstanding comments.
76%
#1247 Fixing Store integration tests. snapcraft merged 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.
75%
#2352 tests: use valid snap names in unit tests snapcraft merged Merged updates to unit tests that replace invalid snap names with valid ones, ensuring compatibility with snap pack --check-skeleton. The author approved and merged the minor change directly without external review.
73%
#1799 tests: update the registered snap fake snapcraft merged Merged following approval by one reviewer. Updated the registered snap fake in tests, changing 2 files with 4 additions and 4 deletions.
72%
#1849 tests: add snap not found tests snapcraft merged Merged tests for missing snap scenarios. Approved by two reviewers and integrated into the codebase despite unresolved review comments and reported integration test timeouts.
72%
#1224 tests: update name registration window limit test. snapcraft merged Merged after updating the name registration window limit test. The store test suite was split to resolve Travis timeouts, and the branch was updated per reviewer feedback. Addresses LP #1676725 and depends on PR #1225.
72%
#631 Fix the store integration tests snapcraft merged Merged a fix for failing store integration tests caused by rate-limiting during rapid snap registrations. Added a wait delay after the second registration to prevent subsequent tests from hitting the same error.
72%
#1963 Fix Store integration tests with updated snap name registration error messages snapcraft closed Closed in favor of PR #1964. The branch updated integration tests for new snap name validation error messages but was superseded by a subsequent pull request.
72%
#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.
71%