tests: remove core24 snap workarounds (#4761)
Metadata
Current evaluation
Merged into main after two approvals and passing CI. The change cherry-picks test updates to remove core24 snap workarounds, resolving CI failures by deleting 22 lines of legacy code in a single file.
Suggested action: —
No scores available.
Issue body
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `tox run -m lint`?
- [x] Have you successfully run `tox run -e test-py310`? (supported versions: `py39`, `py310`, `py311`, `py312`)
-----
Cherry-pick #4761 onto `main` so CI can pass
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged into main after two approvals and passing CI. The change cherry-picks test updates to remove core24 snap workarounds, resolving CI failures by deleting 22 lines of legacy code in a single file. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Cherry-picked onto main to remove outdated core24 snap workarounds from tests, resolving CI failures. Merged as #4761. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged PR #4761 into main to remove core24 snap workarounds from tests, resolving CI failures. The change was cherry-picked to restore passing builds. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #4761 tests: remove core24 snap workarounds | snapcraft | merged | Merged after two approvals and passing CI. Removed core24 snap workarounds from spread tests, deleting 22 lines. Branch retargeted to hotfix/8.2 for release. | |
| #2412 feat(snap): migrate the snap to core24 | charmcraft | merged | Merged after two approvals. Migrates snap packaging to core24 across four files. Known test failures were deemed non-blocking. CI passed on most platforms despite Ubuntu 20.04 and 22.04 failures. | |
| #767 build(snap): update snap to core24 | rockcraft | merged | Merged the snap build update to core24, resolving issue #750. Approved by two reviewers, the change modified seven files. Changelog updates are handled separately during pre-release. | |
| #6009 test: remove core20 spread tests | snapcraft | merged | Merged removal of core20 spread tests and migration of related cases to core22 and core24. This update unblocks core20 deprecation for Snapcraft 9 and improves modern test coverage. | |
| #5209 tests: enable core24 plugin tests | snapcraft | merged | Merged after four reviewers approved. The changes enable core24 plugin tests across 132 files, adding 2,795 lines. CI checks passed and the code was successfully integrated into the main branch. | |
| #1819 build(snap): update snap to core24 | charmcraft | closed | Snap core24 build updates were closed without merging. No reviews or CI checks were completed, and the changes were abandoned. | |
| #4594 fix(lifecycle): install 'core24' snap | snapcraft | merged | Merged a lifecycle fix to install the core24 snap, ensuring proper package cutoff functionality based on the project base. Approved by two reviewers, passed CI, and modified 9 files (+97/-9 lines). | |
| #4186 snap: migrate to core22 | snapcraft | merged | Migrated the snap build environment to Ubuntu core22. Approved by two reviewers and passing CI, the change was merged into the main branch, increasing overall code coverage by 4.91%. | |
| #4734 feat: drop `build-base: devel` requirement for core24 | snapcraft | merged | Merged after two approvals and passing CI. Removes the build-base: devel requirement for core24 snaps, modifying 67 files with a net reduction of 80 lines. | |
| #4557 feat!: fail if we know the project is core24 | snapcraft | merged | Merged a breaking change that fails early for core24 projects. Added core24 spread tests with expected failures and updated CI workflows. Approved by two reviewers and passed all checks. |