Integration tests for snap command with target arch
Metadata
Current evaluation
Merged. Adds integration tests for the snap command with --target-arch, fixing silent flag ignoring when placed before the snap subcommand and missing help text. Approved by two reviewers, four files changed.
Suggested action: —
No scores available.
Issue body
Raised [in the forum](https://forum.snapcraft.io/t/kernel-cross-compile-problem-missing-snapcraft-target-arch-options/1083/5) the `--target-arch` option seems to be silently ignored depending on where it's specified, specifically if it's specified before the **snap** command.
Additionally the argument never shows up in `--help`.
We're also lacking integration tests for the different methods of invocation.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged. Adds integration tests for the snap command with --target-arch, fixing silent flag ignoring when placed before the snap subcommand and missing help text. Approved by two reviewers, four files changed. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged integration tests for the snap command with the --target-arch flag. The tests verify correct flag parsing across invocation methods, resolving silent ignoring and missing help text issues. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged integration tests for the snap command's --target-arch option. The update verifies flag behavior across invocation methods, resolving silent ignoring when placed before the subcommand and fixing missing help documentation. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2014 integration tests: snap tests shouldn't be arch-specific | snapcraft | merged | Merged a change removing architecture-specific constraints from snap integration tests. Approved by one reviewer with a two-line modification across a single file. | |
| #554 Support running the snaps tests in multiple architectures | snapcraft | merged | Merged to enable snaps tests across multiple architectures. Modified one file with four additions and two deletions. Resolved after maintainer requested a retest. | |
| #1542 tests: add integration tests for build snaps | snapcraft | merged | Merged integration tests for build snaps. Approved by two reviewers and passed CI. Two test scenarios omitted due to known bugs. | |
| #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. | |
| #4596 fix(package): use correct target-arch on snap | snapcraft | merged | Merged following approval. Fixes incorrect target-arch in the Package service, aligns tests with craft-application updates, and updates spread tests. 14 files changed, +61/-50 lines. | |
| #990 tests: fix snaps tests in armhf | snapcraft | merged | Merged into master to fix failing snap tests on the armhf architecture. Approved by a reviewer, the change updated 11 files and resolved CI failures without altering overall code coverage. |