test(spread): test building base and snapd snaps
Metadata
Current evaluation
Merged as fb8c81e8f, adding manual Spread coverage that builds current base snaps and the snapd snap against mainline Snapcraft.
Resolution: closed resolved
Reason: PR #5845 was merged in commit fb8c81e8f. Issue #5843 was identified as an independent cause of test failures, with its fix handled separately.
Evaluated by: mr-cal-bot
No scores available.
Issue body
This adds spread tests that build the current base snaps and the snapd snap, ensuring that these snaps continue building with mainline snapcraft.
The tests are currently failing due to https://github.com/canonical/snapcraft/issues/5843, which has its own fix in the works, but enabling this is a distinct workset.
- [x] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `make lint`?
- [x] Have you successfully run `make test`?
---
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged as fb8c81e8f, adding manual Spread coverage that builds current base snaps and the snapd snap against mainline Snapcraft. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #5845 in commit fb8c81e8f, adding Spread coverage that builds current base snaps and snapd with mainline Snapcraft. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #5845 in commit fb8c81e8f, adding Spread coverage for building base and snapd snaps. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged spread tests for building base and snapd snaps. Despite initial CI failures linked to snapcraft issue #5843, the changes were approved and integrated to maintain build compatibility with mainline snapcraft. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Introduces spread tests for building base and snapd snaps. Tests are enabled despite initial failures from snapcraft issue #5843, creating a distinct workset for continuous validation. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/snapcraft#5843
(confidence 98%)
The PR states that the new tests were failing because of #5843, which required a separate fix.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #5841 Include "build snapd snap" test in test suite | snapcraft | closed | Added spread coverage for building snapd and base snaps in merged PR #5845 (commit fb8c81e8f). | |
| #943 test(spread): test on more bases | craft-application | merged | Merged PR #943: added an external Python runtime for testcraft and expanded Spread tests across multiple snap bases to detect base-specific bugs earlier. | |
| #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. | |
| #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. | |
| #386 tests: update testing to use craft-test | fetch-service | merged | Merged #386 as commit 323b0e1, replacing direct Spread calls with snapcraft test and updating the test base to core24. | |
| #3212 tests: expand spread coverage for stage-snaps | snapcraft | closed | Spread-test PR was closed unmerged in favor of snapcraft#3211, merged as 0870f1738 to fix stage-snaps handling for v2 plugins. | |
| #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. | |
| #2793 tests: enable SNAPCRAFT_BUILD_INFO for spread | snapcraft | merged | Merged as commit 12370aae1, enabling SNAPCRAFT_BUILD_INFO across Spread tests to exercise manifest generation and updating nil-basic for the resulting manifest. | |
| #2987 spread tests: set appropriate default base in snapcraft.yamls | snapcraft | merged | PR #2987 merged (commit feb2672d2): spread tests now rewrite the base in snapcraft.yaml to the suite's target base instead of hardcoding core18, with a clear_base() helper for legacy tests. |