Fixing 'sign-build' integration tests.
Metadata
Current evaluation
Merged fix for sign-build integration tests. Updated the test to build, register, and sign a snap, and extended the fake server for account-info. Approved by one reviewer; merged with 57 additions and 17 deletions across three files.
Suggested action: —
No scores available.
Issue body
Even for building the local assertion, the command actually checks if the
account logged in has upload permission on the targeted snap (resolving
name to snap-id via account-info).
The test now builds an random snap, register the name in the store and
only then signs the snap. The fake server was extended to record registered
names and return them in the account-info endpoint.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged fix for sign-build integration tests. Updated the test to build, register, and sign a snap, and extended the fake server for account-info. Approved by one reviewer; merged with 57 additions and 17 deletions across three files. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged fix for 'sign-build' integration tests. Updated the test to register the snap name before signing and extended the fake server to return registered names via the account-info endpoint, resolving permission check failures. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged fix for sign-build integration tests. Updated the test workflow to build a snap, register its name, and sign it, while extending the fake server to return registered names via the account-info endpoint, resolving upload permission checks. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #831 'sign-build' implementation. | snapcraft | merged | Merged the snapcraft sign-build command for generating, signing, and pushing assertions. Supersedes PR 795. Lint errors were fixed and design concerns regarding future release integration were resolved with reviewer approval. | |
| #5826 fix: parse a snap file for the sign-build command | snapcraft | merged | Merged fix aligning legacy and modern Craft CLI argument parsing for the sign-build command, resolving mismatch errors. Also removed redundant --local help text. Addresses #5618. Approved and passed CI. |