Fix Store integration tests with updated snap name registration error messages (take 2)
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
This starts from @maxiberta's #1963 and fixes a couple more strings.
Here goes nothing.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged updates to Store integration tests aligning with revised snap name registration error messages. Builds on #1963, correcting additional string references to restore test compatibility. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged after updating Store integration tests to match revised snap name registration error messages. Corrected additional expected strings, building on PR #1963. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #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. | |
| #1797 tests: update the snap name already registered for store tests | snapcraft | merged | 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. | |
| #2056 Fix formatting of some store errors | snapcraft | merged | Merged after PR #2050 landed first. Fixed SnapcraftError formatting that crashed __str__ on pre-formatted strings with braces. Author rebased and added tests for StoreReviewError and StoreMetadataError fixes. Approved and passed CI. | |
| #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. | |
| #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. | |
| #595 Fix the store update test to register a unique name | snapcraft | merged | Merged a fix for the store update test by registering a unique name. The author confirmed the change was ready despite unrelated ROS CI failures, and the pull request was successfully merged. |