Start using `spread` for integration tests
Metadata
Current evaluation
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.
Resolution: closed resolved
Reason: The spread integration-test infrastructure was implemented and merged in PR #862 (merge commit 7790c044, branch canonical/spread-bootstrap). The issue was closed after the maintainer commented that the work was done in that PR, and the repo now contains extensive spread tests (tests/spread/**, spread.yaml, spread GitHub workflows).
Evaluated by: mr-cal-bot
No scores available.
Issue body
After the initial infrastructure is added/setup, we can start actually adding integration tests.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | 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. | |
| stealth/space-bunny-alpha | — | closed resolved | Closed as done by facundobatista, pointing at merged PR #862 (commit 80f1ec31 'tests: bootstrap spread', merge 7790c044), which added the initial spread setup under tests/spread. | |
| stealth/space-bunny-alpha | — | closed resolved | Spread integration-test support was completed in merged PR canonical/charmcraft#862, via commit 80f1ec31 and merge 7790c044. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Completed and merged via pull request #862, which implemented the spread infrastructure and added integration tests. The issue was closed after the changes were successfully integrated. | |
| qwen/qwen3.6-35b-a3b | — | — | Resolved via pull request #862. The spread integration test infrastructure was implemented, fulfilling the request to adopt spread for testing. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Integration tests using spread were implemented and merged in pull request #862, resolving the request to adopt spread for testing. |
Update history
No update history recorded yet.
Related work
-
Likely Fixed By:
canonical/charmcraft#862
(confidence 97%)
PR referenced by the closing maintainer comment as the place the spread work was completed; its merge commit 7790c044 appears in charmcraft history.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1555 refactor(tests/spread): stop use init in most spread tests | charmcraft | closed | Closed unmerged and replaced by PR #1556, merged as commit 4682b32b, which updated Spread tests to use explicit charmcraft.yaml files instead of charmcraft init. | |
| #2173 tests: create basic integration test spread infrastructure | snapcraft | merged | Merged PR #2173 in commit ad7260ced, adding Spread integration-test infrastructure and porting the Go plugin integration test; it resolved LP #1779419. | |
| #291 Refactor spread testing to use spread extensions | fetch-service | closed | Fixed by merged PR #299 (commit 44d9906), which refactored spread tests to generate spread.yaml and use the new spread extension framework. | |
| #1324 test(spread): test-deploy charms in large spread tests | charmcraft | merged | Merged PR #1324 via commit a14d6d91, adding test deployment of charms and bundles to large Spread tests. | |
| #613 Execute spread testing | craft-application | closed | Spread testing delivered: TestingService and an opt-in 'test' command merged via PR #694 (commit 4910b2d8), building on #565 and #592. Maintainer mr-cal closed the issue citing that PR. | |
| #1556 refactor(tests/spread): test with explicit charmcraft.yaml files | charmcraft | merged | Merged PR #1556 in commit 4682b32b, replacing prepare-step manipulation of charmcraft.yaml with explicit spread-test configuration files. | |
| #2276 spread: move legacy wiki tests to spread | snapcraft | merged | Merged PR #2276 as commit 806b0b0db, moving all non-fake-server legacy wiki tests from the general integration suite to Spread. | |
| #16 tests: add spread support | rockcraft | merged | Merged PR #16 added Spread support for CRAFT-865 via commit 3aeba2b3 (merge commit 5bf2680a). | |
| #1962 Spread test the documentation examples | charmcraft | open | Documentation examples are now covered by dedicated Spread tests and CI through merged PR #2126; the still-open issue should be closed. | |
| #706 Add integration tests using the snap | charmcraft | closed | Snap-based integration coverage was added through Charmcraft's Spread suite by merged PR canonical/charmcraft#862 (commit 7790c044). |