Fixing Store integration tests.
Metadata
Current evaluation
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.
Suggested action: —
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 |
|---|---|---|---|---|
| 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to update Store integration tests for a new 40-character snap-name limit. Contributor replaced u1test prefixes with snapcrafttest per reviewer feedback. Resolves test failures pending LP #1681545 store error handling. |
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. | |
| #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. | |
| #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. | |
| #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. | |
| #687 Enable integration tests to run in the production store | snapcraft | merged | Merged changes enabling integration tests in the production store. Consolidated TestStore, adjusted register intervals to match production settings, and cleaned up code across eight files. Approved for improved clarity and alignment. | |
| #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. | |
| #1538 tests: update the store failure tests | snapcraft | merged | Merged after approval and passing CI. Updates store failure tests by adjusting autopkgtest scripts to report final errors, sets exception exit codes to 2, and clarifies integration test execution against fake servers. |