Integration tests for snap command with target arch
Metadata
Current evaluation
PR #1378 merged as 18a767537, fixing --target-arch parsing before the snap subcommand and adding integration tests for invocation forms.
Resolution: closed resolved
Reason: The PR was merged as commit 18a767537, which corrected hidden-option parsing and added integration coverage for --target-arch invocation.
Evaluated by: mr-cal-bot
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 |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | PR #1378 merged as 18a767537, fixing --target-arch parsing before the snap subcommand and adding integration tests for invocation forms. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged as commit 18a767537, fixing silent ignoring of --target-arch before the snap subcommand and adding integration tests for invocation forms. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in #1378 via commit 18a767537, fixing --target-arch parsing before the snap subcommand and adding integration tests for supported invocation forms. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged as commit 18a767537, fixing silent ignoring of --target-arch before the snap subcommand and adding integration tests for snap command invocation. | |
| 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. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/snapcraft#1353
(confidence 85%)
Earlier merged work added default options for the implicit snap command.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2014 integration tests: snap tests shouldn't be arch-specific | snapcraft | merged | Merged #2014 in commit fd176f3bc, removing architecture-specific constraints from snap integration tests so they run across architectures. | |
| #1542 tests: add integration tests for build snaps | snapcraft | merged | PR #1542 merged as 8c9900f38, adding integration coverage for build snaps; two scenarios remained excluded due to Launchpad bugs #1716054 and #1716055. | |
| #4596 fix(package): use correct target-arch on snap | snapcraft | merged | Merged PR #4596 corrected Package service target architecture for snap builds and updated lifecycle, unit, and spread tests; landed as c68b3621c. | |
| #1595 tests: fix the skip of snapd integration tests in armhf | snapcraft | merged | Merged #1595 (8db14753d), fixing armhf snapd integration-test skips by checking self.deb_arch only after the parent setUp initialized it. | |
| #990 tests: fix snaps tests in armhf | snapcraft | merged | Merged PR #990 as commit f467e55bc, correcting armhf failures in snap tests across 11 files and closing Launchpad bug #1650458. | |
| #554 Support running the snaps tests in multiple architectures | snapcraft | merged | Merged PR #554 by come-maiz added support for running the snaps tests in multiple architectures (LP #1589779), landing as commit baed96749 with a +4/-2 CI config change. |