tests: add spread suite for plainbox plugin (#2317)
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
Move the plugin and snapd integration tests for the plainbox plugin into a
Spread suite.
Also pin the checkbox-ng package in the tests checkbox snap.
Also removed a test that looks more like a plainbox tests and not a
snapcraft one.
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. 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged changes adding a Spread test suite for plainbox plugin integration tests. The update pins the checkbox-ng package in the test snap and removes a misplaced test. Resolves Launchpad bug #1783839. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged changes reorganizing plainbox plugin integration tests into a Spread suite, pinning the checkbox-ng package, and removing a misplaced test. Successfully integrated into the main branch. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #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. | |
| #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. | |
| #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. | |
| #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%. | |
| #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. | |
| #2793 tests: enable SNAPCRAFT_BUILD_INFO for spread | snapcraft | merged | Merged following approval and passing CI. Enabled SNAPCRAFT_BUILD_INFO for spread tests to cover manifest changes. Included a fix for the nil-basic spread test. Minimal diff across three files. | |
| #2182 tests: add spread suite for autotools plugin | snapcraft | merged | Merged to master, consolidating Autotools plugin and snapd tests into a single Spread suite to resolve Launchpad bug #1783825. The commit modifies six files, reduces code by 16 lines, passed CI checks, and received reviewer approval prior to integration. | |
| #1272 tests: initial setup for the snapcraft snap tests with spread | snapcraft | merged | Merged to master following single reviewer approval. Introduced initial spread test infrastructure for snapcraft across four files (+39 lines). Code coverage remained stable at 95.33%. | |
| #2203 tests: add spread suite for kbuild plugin | snapcraft | merged | Merged following approval and passing CI. Resolves LP: #1783835 by migrating Kbuild plugin integration tests to a Spread suite across six files, increasing overall coverage by 0.02%. |