tests: remove snapcraft:core
Metadata
Current evaluation
Merged PR #123 (commit 0cffc84, merged in ba78ce4) deleted the obsolete `snapcraft:core` integration test entry and added `snapcraft:devel`, since Multipass removed the core image.
Resolution: closed resolved
Reason: The PR itself was merged into the default branch (merge commit ba78ce4, 'Merge pull request #123 from canonical/multipass-testing') with the single-file change '+0/-9'. Multipass dropped support for the `core` image, so the test referencing it was removed; remaining tests now use core22/core24.
Evaluated by: mr-cal-bot
No scores available.
Issue body
core has been removed by multipass, craft-providers does not need to
support core so remove the test for it.
- [x] 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 #123 (commit 0cffc84, merged in ba78ce4) deleted the obsolete `snapcraft:core` integration test entry and added `snapcraft:devel`, since Multipass removed the core image. | |
| qwen/qwen3.6-35b-a3b | — | closed resolved | Merged PR removing the `snapcraft:core` test from craft-providers, as Multipass removed `core` support. |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Related To:
canonical/craft-providers#300
(confidence 45%)
Later commit 511996d removed core16 tests and defaulted to core22, continuing the removal of legacy core images.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2353 tests: remove dependency on snapcraft for integration tests | snapcraft | merged | PR #2353 by sergiusens merged as commit 6a7c860c4, dropping snapcraft imports from integration tests (27 files, +956/-733) so the suite no longer depends on snapcraft and tests stay independent, alongside the bases/multipass cleanup. | |
| #4755 tests: fix devel base snap test | snapcraft | merged | Merged #4755, fixing the devel-base integration test to verify Snapcraft can create the base snap and that the resulting devel base is usable. | |
| #1849 tests: add snap not found tests | snapcraft | merged | Merged missing-snap test coverage; snapcraft PR #1849 landed as commit 54cf2bd9e. | |
| #2293 build providers: use the new snapcraft: remote for multipass | snapcraft | merged | Merged PR #2293 as commit 66060e839, switching Multipass builds to the snapcraft: remote and delegating unsupported-base errors to Multipass. | |
| #6009 test: remove core20 spread tests | snapcraft | merged | Squash-merged PR #6009 as commit d0f1b4394, deleting core20 spread tests and migrating applicable ones to core22/core24 (+20 net tests). Unblocked core20 lifecycle removal for Snapcraft 9 (#5675). | |
| #133 Added a test to check that the snapcraft files are ignored. | snapcraft | closed | Superseded by merged PR canonical/snapcraft#137, which added the Snapcraft-file exclusion regression test and cleaned up base-plugin tests. | |
| #3609 tests: move legacy snapcraft tests (CRAFT-728) | snapcraft | merged | Merged PR #3609 in commit a71aeff54, moving snapcraft_legacy unit tests from tests/unit to tests/legacy to match the package rename. | |
| #1813979 snapcraft cleanbuild wants to install multipass but it's already installed | snapcraft (launchpad) | closed | Deb-packaged Snapcraft 2.43.1 failed while Multipass worked in 3.0.1. Legacy cleanbuild was later removed because builds are clean by default. | |
| #2258 build providers: snapcraft images for multipass | snapcraft | merged | Merged PR #2258 as commit 5ffefbf96, adding Snapcraft build images for Multipass and a Darwin launch workaround for LP #1791821. |