spread tests: use source-depth: 1 for plainbox tests
Metadata
Current evaluation
Merged after approval and CI validation. Updated spread tests to use source-depth: 1 for plainbox tests, modifying two files with three added lines.
Suggested action: —
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/)?
- [ ] Have you successfully run `./runtests.sh static`?
- [ ] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged after approval and CI validation. Updated spread tests to use source-depth: 1 for plainbox tests, modifying two files with three added lines. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a configuration update to spread tests, setting source-depth: 1 for plainbox tests to streamline execution. The change was authored by sergiusens and approved without comments. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged pull request updating spread tests to use source-depth: 1 for plainbox tests. Integrated into the main branch by maintainer sergiusens without additional comments. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2583 plainbox spread tests: use https for the git source entries | snapcraft | merged | Merged a one-line change updating plainbox spread tests to use HTTPS for git source entries. Approved by two reviewers and passed CI checks. | |
| #2317 tests: add spread suite for plainbox plugin | snapcraft | merged | Merged after approval and passing CI checks. Introduces a Spread test suite for plainbox plugin integration tests and pins the checkbox-ng package in the test environment. | |
| #3020 spread tests: default base for local plugin tests | snapcraft | merged | Merged updates to spread tests establishing a default base for local plugin tests. The change follows a prior commit pattern, adding two lines across two files. Approved by one reviewer and passed CI checks. | |
| #2321 tests: add spread suite for plainbox plugin (#2317) | snapcraft | merged | Merged after approval and passing CI. The pull request adds a Spread test suite for the plainbox plugin, pins the checkbox-ng package, and removes a misplaced test from the snapcraft test suite. | |
| #62 spread: add a "big" spread test | rockcraft | merged | Merged a reusable spread test framework to house multiple checks without repeated setup. The initial test validates bug #44 regarding directory permissions. Approved by one reviewer and passed CI checks. | |
| #2379 test(spread): per-base destructive mode tests | charmcraft | merged | Merged following approval by two reviewers. Added per-base destructive mode tests for spread and removed a redundant uv test. Changes were integrated despite mixed CI results. | |
| #2216 spread tests: keep sources local | snapcraft | merged | Merged PR #2216 to replace remote FTP sources in spread tests with in-tree tar archives, preventing test failures from server downtime. Approved by one reviewer, passed CI checks, and slightly increased code coverage. | |
| #13 test(spread): add more spread tests | snapcraft-rocks | merged | Successfully merged spread tests covering architectures, chisel slices, craftctl, package-repositories, and patchelf. Approved by two reviewers across 15 files (+206/-5 lines). | |
| #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. | |
| #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. |