Include "build snapd snap" test in test suite
Metadata
Current evaluation
Added spread coverage for building snapd and base snaps in merged PR #5845 (commit fb8c81e8f).
Resolution: closed resolved
Reason: PR #5845 added spread tests that build snapd and base snaps, directly addressing the requested test coverage. The change landed in commit fb8c81e8f.
Evaluated by: mr-cal-bot
No scores available.
Issue body
### What needs to get done
We should have a test in our test suite (probably a manual spread test) that builds snapd
### Why it needs to get done
Some recent bugs only got caught when pushing to candidate.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Added spread coverage for building snapd and base snaps in merged PR #5845 (commit fb8c81e8f). | |
| stealth/space-bunny-alpha | — | closed resolved | Fixed by merged PR #5845 (commit fb8c81e8f), adding spread tests that build snapd and base snaps. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without implementation or linked resolution. The requested build test was not added, indicating the issue was abandoned or superseded by alternative testing measures. | |
| qwen/qwen3.6-35b-a3b | — | — | Abandoned. The request to add a manual spread test for building snapd was closed without implementation after six days, with no comments or linked pull requests. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without implementation. The requested spread test for building snapd was not added to the suite and the issue was abandoned. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Closed without implementation. The proposed manual spread test for building snapd snap was abandoned. No pull requests or fixes were created to address the gap. |
Update history
No update history recorded yet.
Related work
-
Likely Fixed By:
canonical/snapcraft#5845
(confidence 100%)
Merged PR whose commit fb8c81e8f added the snapd and base-snap build tests.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #5845 test(spread): test building base and snapd snaps | snapcraft | merged | Merged as fb8c81e8f, adding manual Spread coverage that builds current base snaps and the snapd snap against mainline Snapcraft. | |
| #521 test(spread): add an improved snap inspector test | fetch-service | merged | Merged in PR #521 (commit 06cd210), adding a more realistic snapd setup to the spread snap inspector test and exposing additional snapd network requests. | |
| #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. | |
| #3312 spread tests: introduce electron-builder test | snapcraft | merged | Merged PR #3312 in commit 04f642ba9, adding an electron-builder useTemplate: false spread test with snapcraft.yaml and meta/snap.yaml comparisons. | |
| #3806 spread: add build packages to package repository tests | snapcraft | merged | Merged in #3806 via commit e77c3130d, adding Spread coverage for installing build packages from snapcraft.yaml package repositories in destructive and build-instance modes. | |
| #432 ci: speed up spread testing workflow | fetch-service | merged | Merged PR #432 (commit 9eec284), removing snap-test's dependency on snap-build so spread tests start sooner while snap-build still produces downloadable artifacts. | |
| #2183 tests: add spread suite for cmake plugin | snapcraft | merged | Merged PR #2183 as commit 51518dbe6, moving CMake plugin and snapd integration tests into a Spread suite and resolving LP #1783830. | |
| #1849 tests: add snap not found tests | snapcraft | merged | Merged missing-snap test coverage; snapcraft PR #1849 landed as commit 54cf2bd9e. | |
| #1272 tests: initial setup for the snapcraft snap tests with spread | snapcraft | merged | Merged PR #1272 as commit 545344442, adding initial Spread-based test infrastructure for the snapcraft snap. | |
| #2182 tests: add spread suite for autotools plugin | snapcraft | merged | PR #2182 merged to master as commit eadd86729, consolidating the Autotools plugin and snapd tests into a single Spread suite and resolving LP bug #1783825. |