Move Plainbox plugin tests to spread
Metadata
Current evaluation
Plainbox integration tests were moved to Spread in merged PR #2317 (commit d2b3aca84); PR #2321 applied the same suite to the legacy branch.
Resolution: closed resolved
Reason: PR #2317 was merged as d2b3aca84, adding Spread coverage for the Plainbox plugin and snapd integration. PR #2321 merged the corresponding legacy-branch change.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Currently the snapcraft CLI's integration tests are all written in Python, which makes parallelization difficult and generally makes them harder to write than a spread suite would be. They also don't nicely run across a number of systems like a spread suite.
Rewrite the combined Plainbox snapd and plugins integration tests as a Spread suite for the Plainbox plugin.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Plainbox integration tests were moved to Spread in merged PR #2317 (commit d2b3aca84); PR #2321 applied the same suite to the legacy branch. | |
| stealth/space-bunny-alpha | — | closed resolved | Plainbox integration tests were rewritten as a Spread suite in merged PR #2317 (commit d2b3aca84); PR #2321 merged the same change into legacy. | |
| stealth/space-bunny-alpha | — | closed resolved | Plainbox integration tests moved from Python to Spread by merged canonical/snapcraft#2317 (d2b3aca84), with checkbox-ng pinned for the test snap. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed. The Plainbox plugin integration tests were migrated to a Spread suite, replacing the Python suite to enable parallelization and cross-system execution. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without implementation or linked pull request, indicating the task was abandoned or superseded by later test infrastructure updates. | |
| qwen/qwen3.6-35b-a3b | — | — | Implemented migration of Plainbox plugin integration tests from Python to a Spread suite for improved parallelization and cross-system execution. The request was completed and the issue closed. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Closed without implementation, likely abandoned or superseded. The proposed migration of Plainbox plugin integration tests to a Spread suite was not completed. |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Likely Fixed By:
canonical/snapcraft#2317
(confidence 100%)
Merged the Plainbox plugin Spread suite as d2b3aca84.
-
Likely Fixed By:
canonical/snapcraft#2321
(confidence 99%)
Merged the corresponding Plainbox Spread suite change into the legacy branch.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2321 tests: add spread suite for plainbox plugin (#2317) | snapcraft | merged | Merged PR #2321 via commit d2b3aca84, adding Spread coverage for Plainbox plugin and snapd integration, pinning checkbox-ng, and removing a Plainbox-specific test. | |
| #2317 tests: add spread suite for plainbox plugin | snapcraft | merged | Merged PR #2317 as d2b3aca84, adding a Spread suite for the plainbox plugin and pinning checkbox-ng in its test snap. | |
| #1889 tests: remove plainbox part from plugin tests | snapcraft | merged | Merged #1889 (commit df9f7f795), removing the obsolete plainbox part from plugin tests after Plainbox was folded into Checkbox, fixing integration-test failures. | |
| #1642 tests: move the plainbox test to the integration suite | snapcraft | merged | PR #1642 merged as commit 53fe32786, moving the Plainbox test to the integration suite and removing obsolete year prefixes from namespace expectations. | |
| #2799 Plainbox test updates | snapcraft | merged | Merged PR #2799 updated Plainbox tests across three files (+2/-2), completing the approved test-maintenance change. | |
| #2199 tests: add spread suite for copy plugin | snapcraft | merged | Merged PR #2199 (f62504d) moved copy-plugin integration tests into a Spread suite, resolving LP #1783831. | |
| #1783831 Move Copy plugin tests to spread | snapcraft (launchpad) | closed | Copy plugin integration tests were rewritten as a Spread suite by merged canonical/snapcraft#2199, resolving the issue in commit 41724d561. | |
| #1608 tests: fix the duplicate plainbox test scenarios | snapcraft | merged | Merged PR #1608 via commit abd58f28b, fixing duplicate plainbox test scenarios with a focused one-file test update (+5/-2). | |
| #2863 spread tests: use source-depth: 1 for plainbox tests | snapcraft | merged | Merged PR #2863 set source-depth: 1 for plainbox spread tests in commit 7bfdfbde6. | |
| #3432 spread tests: remove legacy plugin tests | snapcraft | merged | PR #3432 (commit 950fa2508, f2c9caa04) merged: deleted 665 lines of legacy plugin spread tests across 29 files, removing outdated plugin coverage without affecting code coverage (90.83%). |