Add integration tests using the snap
Metadata
Current evaluation
Snap-based integration coverage was added through Charmcraft's Spread suite by merged PR canonical/charmcraft#862 (commit 7790c044).
Resolution: closed resolved
Reason: PR canonical/charmcraft#862 was merged as commit 7790c044, adding the Spread-based integration test setup. The closing maintainer comment confirmed it was done as part of the Spread tests.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Related: #665.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Snap-based integration coverage was added through Charmcraft's Spread suite by merged PR canonical/charmcraft#862 (commit 7790c044). | |
| stealth/space-bunny-alpha | — | closed resolved | Snap integration tests were delivered through the Spread suite by merged PR #862 (commit 7790c044). | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Resolved by implementing the requested snap integration tests as part of the spread test suite. | |
| qwen/qwen3.6-35b-a3b | — | — | Integration tests using the snap were implemented as part of the spread tests framework. The request was fulfilled and the issue closed. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Integration tests using the snap were completed as part of the spread test suite. The request was fulfilled and the issue closed. |
Update history
No update history recorded yet.
Related work
-
Likely Fixed By:
canonical/charmcraft#862
(confidence 99%)
Merged Spread bootstrap work delivered the requested snap-based integration tests.
-
Related To:
canonical/charmcraft#665
(confidence 98%)
The issue explicitly references #665, which introduced Spread for Charmcraft integration testing.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #202 Add snapcraft tests | fetch-service | open | Snap package integration tests were added and CI now runs the Spread suite through snapcraft test, although the tracking issue remains open. | |
| #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. | |
| #665 Start using `spread` for integration tests | charmcraft | closed | Adopted spread for integration tests via merged PR canonical/charmcraft#862 (merge commit 7790c044, branch spread-bootstrap). Issue closed by maintainer sergiusens on 2023-03-16 noting the work was done in that PR. | |
| #1590268 snapcraft-parser has no integration tests | snapcraft (launchpad) | closed | Added a user-level happy-path integration test for snapcraft-parser via merged canonical/snapcraft#560 (commit 8f5b30dba); released in Snapcraft 2.12. | |
| #50 test: test the snapcraft actions | craft-actions | merged | Merged PR #50 added Snapcraft action tests adapted from Charmcraft’s suite, with all reported CI checks passing. | |
| #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). | |
| #1541391 Missing integration tests for login/logout/upload | snapcraft (launchpad) | closed | Added integration coverage for Snapcraft's store-facing login, logout, and upload flows in ubuntu-core/snapcraft#302, linked in the issue's final discussion. | |
| #758 Add an integration test for parts with filesets | snapcraft | closed | PR #758 by jocave added an integration test for parts with filesets; it was closed unmerged after sergiusens pointed out canonical/snapcraft#799 already contains the test (merged as 1a282439c). | |
| #1625 tests: use the snapcraft snap for integration tests | snapcraft | merged | Merged PR #1625 updated integration tests to run the installed Snapcraft snap, better matching the user environment. Resolution: commit eebb070e6. | |
| #560 Add snapcraft-parser integration test. | snapcraft | merged | Merged PR #560 added the snapcraft-parser happy-path integration test in commit 8f5b30dba. |