← Back to issue list

tests: enable core24 plugin tests

View original Github issue

Metadata

Project
snapcraft
Number
#5209
Type
pull request
State
merged
Author
sergiusens
Labels
Created
Updated
Closed

Current evaluation

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.

Suggested action:

No scores available.

Issue body

- [ ] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `tox run -m lint`? - [ ] Have you successfully run `tox run -e test-py310`? (supported versions: `py39`, `py310`, `py311`, `py312`) -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b 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.
qwen3.6-35b-a3b-mtp-q6 Merged the changes enabling core24 plugin tests. Reviewers noted unrelated CI failures from separate issues, and the code was successfully integrated into the main branch.
qwen3.6-35b-a3b-mtp-q6 Merged changes enabling core24 plugin tests. Identified test failures were attributed to unrelated issues and timeouts, allowing the update to proceed without blocking.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#5977 test: re-enable core24 python test snapcraft merged Merged the re-enablement of the core24 python test after it began passing again. No code changes were required. Resolves #5207, approved by two reviewers, and passed CI checks before merge.
77%
#6202 test(spread): create core26 spread test suite snapcraft merged Merged the core26 spread test suite, adding CI jobs and generalizing tests for core22, 24, and 26 across 419 files. Approved by two reviewers and passed most CI checks prior to integration.
76%
#4056 spread: core24 testing infra snapcraft merged Merged after receiving two approvals and passing CI checks. Implements core24 testing infrastructure for spread, changing two files with a net addition of 16 lines. Code coverage remained unchanged.
76%
#4763 tests: remove core24 snap workarounds (#4761) snapcraft merged 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.
75%
#4600 fix(tests): core24 classic linter test expectations snapcraft merged Merged after approval and passing CI. The change updates core24 classic linter test expectations. A maintainer noted a follow-up PR was needed for unit tests, and a reviewer later flagged a separate core22 spread failure.
75%
#4582 test: add core24 component spread tests snapcraft merged Merged after approval and passing CI. Adds core24 component spread tests by flattening and copying existing test files. Coverage increased slightly, and changes were landed on the feature/components branch.
74%
#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.
74%
#4606 feat(app): support extensions in core24 snapcraft merged Added extension support to core24. Merged after approval by two reviewers, passing all CI checks with 100% coverage on modified lines. The change updated nine files, adding 183 lines and removing 30.
74%
#4595 tests(spread): remove core24 ros2 extension linter tests snapcraft merged Merged removal of core24 ros2 extension linter tests, as no ROS 24.04 or extension exists. Approved by two reviewers, passed CI, and maintained coverage. Deleted 101 lines across five files.
74%
#4690 feat(extensions): support core24 in the gnome extensions snapcraft merged Merged to add core24 support to GNOME extensions. Approved by four reviewers, passed all CI checks, and maintained 100% test coverage on modified lines across five files.
73%