integration tests: consolidate lxd/multipass fixtures for reuse
Metadata
Current evaluation
Merged PR #44 in commit ee0f643, consolidating reusable LXD and Multipass integration-test fixtures and including a documentation fix.
Resolution: closed resolved
Reason: PR #44 was approved and merged into the main branch. Commit ee0f643 records the fixture consolidation and related documentation correction.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Signed-off-by: Chris Patterson <chris.patterson@canonical.com>
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #44 in commit ee0f643, consolidating reusable LXD and Multipass integration-test fixtures and including a documentation fix. | |
| qwen/qwen3.6-35b-a3b | — | closed resolved | Merged PR consolidating LXD and Multipass test fixtures for reuse across integration tests, alongside a documentation fix. |
Update history
| Date | Change |
|---|---|
| created |
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #207 tests: split lxd integration tests | craft-providers | merged | Merged #207 to split LXD integration tests, resolving review feedback from #205; landed as commit 3eba5ca. | |
| #998 ci: enable and stabilize multipass tests in QA | craft-providers | open | Maintainer CI PR enabling multipass integration tests in qa.yaml on a large self-hosted runner (LXD disabled), wiring setup-tests to install-multipass, waiting on multipassd with diagnostics, and fixing bytes-vs-string and cache-mount test assertions. All listed checks pass; 5 review comments unresolved. | |
| #28 multipass tests: cleanup after launch tests | craft-providers | merged | Merged PR #28 to ensure temporary Multipass instances are cleaned up after launch tests, adding explicit teardown handling in the multipass test fixture. | |
| #32 lxc: add project helpers and integration integration tests | craft-providers | merged | Merged PR #32 added LXC project helpers and integration tests, landing as merge commit e392301 with implementation commit 9b72743. | |
| #2983 tests: add LXD spread test | snapcraft | merged | Merged PR #2983 as 4b8522a14, adding spread coverage that builds the LXD package with LXD and destructive/host providers; installation checks were deferred. | |
| #178 tests: refactor lxd launcher tests | craft-providers | merged | Merged PR #178 in commit 43146e2 refactored LXD launcher tests to distinguish regular instances from base instances, enabling the test coverage needed by #177. | |
| #3326 lxd unit tests: simplify command checking pattern | snapcraft | merged | Merged PR #3326 (9c9381695) simplified LXD unit-test command checks, added a shared command prefix, removed environment-argument mocks, and cut 309 net lines. | |
| #712 fix: use Multipass-compatible instance names | craft-providers | merged | Merged PR #712 as commit e52ea2d, normalizing Multipass instance names with LXD-compatible logic to fix #226 and unblock charmcraft#2058. | |
| #1225 tests: improve multipass spread backend | charmcraft | merged | Merged as 6d68140c, adding concurrent Multipass Spread workers with LXD-style naming; defaults use 2 workers for Ubuntu 22.04 and 1 for other series. |