tests: add spread suite for plainbox plugin
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
Move the plugin and snapd integration tests for the CMake plugin into a
Spread suite.
Also pin the checkbox-ng package in the tests checkbox snap.
LP: #1783839
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 |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into main. Moves CMake plugin integration tests to a Spread suite and pins the checkbox-ng package in the test snap, addressing LP #1783839. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into main. Moved CMake plugin integration tests to a Spread suite and pinned the checkbox-ng package. Commit message was amended per maintainer feedback before final merge. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #2863 spread tests: use source-depth: 1 for plainbox tests | snapcraft | merged | Merged after approval and CI validation. Updated spread tests to use source-depth: 1 for plainbox tests, modifying two files with three added lines. | |
| #1889 tests: remove plainbox part from plugin tests | snapcraft | merged | Merged changes removing the plainbox component from plugin tests. This resolves integration test failures caused by the plainbox library being integrated into the checkbox application, rendering the separate plainbox part obsolete for snaps. | |
| #2199 tests: add spread suite for copy plugin | snapcraft | merged | Merged into master following approval and passing CI. Moves copy plugin integration tests to a spread suite, resolving bug LP: #1783831. Modified 10 files, reduced code by 41 lines, and increased coverage by 0.02%. | |
| #2173 tests: create basic integration test spread infrastructure | snapcraft | merged | Merged changes establishing basic integration test infrastructure using Spread. Resolves LP: #1779419 by implementing a reusable test suite, initially proven with the Go plugin test. Approved after iterative revisions. | |
| #1642 tests: move the plainbox test to the integration suite | snapcraft | merged | Merged changes to move the plainbox test to the integration suite. Updated expected namespace results to remove outdated year prefixes, fixing a test failure caused by upstream plainbox changes. Approved and passed CI. | |
| #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. | |
| #2201 tests: add spread suite for nil plugin | snapcraft | merged | Merged to add a Spread test suite for Nil plugin integration tests, resolving Launchpad bug #1783838. Approved by reviewers, passed CI, and updated 8 files. | |
| #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. | |
| #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. |