tests: fix maven spread test
Metadata
Current evaluation
Merged PR #2400 as commit 155306683, correcting the Maven spread test with a two-line change.
Resolution: closed resolved
Reason: PR snapcore/snapcraft#2400 was merged into master, resolving the faulty Maven spread test in commit 155306683.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)?
- [ ] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [ ] Have you successfully run `./runtests.sh static`?
- [ ] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #2400 as commit 155306683, correcting the Maven spread test with a two-line change. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #2400 as commit 155306683, correcting the Maven spread test with a two-line change. | |
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged by maintainer sergiusens to fix the maven spread test. Integrated into main branch, resolving the test failure despite a Codecov report noting missing base coverage data. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #738 test: add maven plugin test | rockcraft | closed | Closed unmerged and superseded by #752 (b7ea7434), which added the Maven spread-test path while fixing Java symlink behavior. | |
| #3662 tests: update spread url | snapcraft | merged | Merged PR #3662 updated Spread tarball URLs in two test files via commit 507959d70. | |
| #3663 tests: update spread url | snapcraft | merged | Merged via commit bcea80568, updating two Spread test tarball URLs (+2/-2 lines). | |
| #415 test: fix typo in spread extension | fetch-service | closed | PR #415 was closed after approval was retracted; its typo fix was incorporated into merged PR #417 via commit 8363f15. | |
| #2533 tests: classic confinement spread tests for ant and maven | snapcraft | merged | Merged PR #2533 in commit 029e2d0c6, adding spread tests for classic-confinement Ant and Maven snaps. | |
| #1366 tests(spread): make release test less flaky | charmcraft | merged | Merged #1366 at commit 05b18a03, reducing flakiness in the Spread release test and resolving #1362. | |
| #1287 tests: minor cleanups on the spread tests | snapcraft | merged | Merged into master as commit 1cf0f8c9f "tests: minor cleanups on the spread tests (#1287)" — a test-only tidy-up touching 4 spread test files (+16/-6), reviewed and approved by fgimenez, with coverage unchanged at 95.27%. | |
| #5075 tests: fix validation sets spread test | snapcraft | merged | Merged PR #5075 (commit a22b218ab) fixed #5053 by making the validation-sets spread test use hello-world and check presence instead of a fixed revision. | |
| #2211 tests: add spread suite for ament plugin | snapcraft | merged | Merged PR #2211 as commit bdaa8ea5, replacing the Ament plugin snapd integration test with a Spread suite and resolving LP #1783827. | |
| #1499 test(spread): replace lunar test with mantic | charmcraft | merged | Merged PR #1499 replaced the Lunar spread test with Mantic in one line (commit 5ee61696). |