tests: classic confinement spread tests for ant and maven
Metadata
Current evaluation
Merged after approval. Added spread tests for classic confinement covering ant and maven. Despite a Codecov warning about missing base commit coverage, the changes (+113/-9 lines across 7 files) were successfully integrated.
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 `./runtests.sh static`?
- [x] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged after approval. Added spread tests for classic confinement covering ant and maven. Despite a Codecov warning about missing base commit coverage, the changes (+113/-9 lines across 7 files) were successfully integrated. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into main branch. Adds spread tests for classic confinement of ant and maven. Codecov reported missing base commit coverage, but the maintainer-approved changes were successfully integrated. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master. Adds classic confinement spread tests for ant and maven. Maintainer sergiusens submitted the changes, which passed static and unit checks. A Codecov report noted missing base commit coverage, but the PR proceeded to merge. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1287 tests: minor cleanups on the spread tests | snapcraft | merged | Merged minor spread test cleanups into master. Modified four files with 16 additions and 6 deletions. Approved by one reviewer with zero unresolved comments, maintaining 95.27% coverage upon integration. | |
| #2211 tests: add spread suite for ament plugin | snapcraft | merged | Merged PR moves the Ament plugin snapd integration test to a Spread suite, resolving LP #1783827. Approved by one reviewer, passed CI, and updated 18 files with a minor coverage decrease. | |
| #2400 tests: fix maven spread test | snapcraft | merged | Merged a +2/-0 change to fix the maven spread test. Authored by maintainer sergiusens, the update was integrated into the main branch without additional reviews or CI blockers. | |
| #2183 tests: add spread suite for cmake plugin | snapcraft | merged | Merged into master, resolving LP #1783830 by migrating CMake plugin integration tests to a Spread suite. Approved, passed CI, and maintained 91.23% code coverage across 17 modified files. | |
| #3242 tests: migrate legacy classic patchelf tests to spread | snapcraft | merged | Merged after reviewer approval and passing CI. The change migrated legacy classic patchelf tests to the spread framework, updating 13 files with a net reduction of 186 lines. | |
| #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. | |
| #3662 tests: update spread url | snapcraft | merged | Merged update to spread tarball URLs in test files. Approved by a reviewer, passed CI checks, modified two files with minimal changes, and maintained existing code coverage. | |
| #3432 spread tests: remove legacy plugin tests | snapcraft | merged | Merged removal of legacy plugin tests from the spread test suite. Deleted 665 lines across 29 files. Approved by reviewer, passed CI, and maintained coverage. Unrelated test failures were addressed in a separate PR. |