tests: add spread suite for scons plugin
Metadata
Current evaluation
Merged PR #2208 as commit a0d0297fc: added a Spread suite for the SCons plugin and moved its plugin/snapd integration tests there, resolving LP bug #1783843 (+32/-76, 9 files).
Resolution: closed resolved
Reason: The pull request itself was merged into master as commit a0d0297fc ("tests: add spread suite for scons plugin (#2208)"), which the commit history for tests/spread/plugins/scons confirms. It delivered the stated outcome of porting the SCons plugin and snapd integration tests into a Spread suite, resolving LP #1783843.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] 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)?
- [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [x] Have you successfully run `./runtests.sh static`?
- [x] Have you successfully run `./runtests.sh unit`?
-----
This PR resolves [LP: #1783843](https://bugs.launchpad.net/snapcraft/+bug/1783843) by moving the plugin and snapd integration tests for the SCons plugin into a Spread suite.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #2208 as commit a0d0297fc: added a Spread suite for the SCons plugin and moved its plugin/snapd integration tests there, resolving LP bug #1783843 (+32/-76, 9 files). | |
| stealth/space-bunny-alpha | — | closed resolved | Merged as a0d0297fc: added a Spread suite for the SCons plugin and moved its plugin/snapd integration tests out of nose, resolving LP bug #1783843. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #2208 as a0d0297fc, adding a Spread suite for the SCons plugin and moving its integration tests, resolving LP #1783843. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged changes resolving Launchpad bug #1783843 by migrating SCons plugin integration tests to a Spread suite. Approved by one reviewer, passed CI, and maintained 91.18% coverage. Net reduction of 44 lines across 9 files. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master. Relocates SCons plugin integration tests to a Spread suite, resolving LP #1783843. Code coverage remains unchanged at 91.18%. |
Update history
No update history recorded yet.
Related work
-
Blocked By:
snapcraft#2173
(confidence 80%)
Earlier merged PR #2173 (commit ad7260ced) created the basic integration-test Spread infrastructure that this SCons suite builds on, addressing LP #1779419.
-
Related To:
snapcraft#3432
(confidence 55%)
Later merged PR #3432 deleted the legacy plugin spread tests, continuing the migration started by this PR.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2183 tests: add spread suite for cmake plugin | snapcraft | merged | Merged PR #2183 as commit 51518dbe6, moving CMake plugin and snapd integration tests into a Spread suite and resolving LP #1783830. | |
| #2182 tests: add spread suite for autotools plugin | snapcraft | merged | PR #2182 merged to master as commit eadd86729, consolidating the Autotools plugin and snapd tests into a single Spread suite and resolving LP bug #1783825. | |
| #1783843 Move Scons plugin tests to spread | snapcraft (launchpad) | closed | SCons integration tests were migrated to a Spread suite by snapcore/snapcraft#2208, merged as commit a0d0297fc. | |
| #2206 tests: add spread suite for catkin plugin | snapcraft | merged | PR #2206 merged as commit 317d4a03a, converting Catkin plugin and snapd integration tests into a Spread suite and resolving LP bug #1783828. | |
| #2205 tests: add spread suite for meson plugin | snapcraft | merged | Merged PR #2205 as commit 19c651af4, moving Meson plugin integration tests from Python into a Spread suite and resolving LP #1783837. | |
| #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. | |
| #2212 tests: add spread suite for ruby plugin | snapcraft | merged | Merged PR snapcore/snapcraft#2212 (commit b7729970a) adds a Spread suite for the Ruby plugin, replacing the Python/snapd integration test; closes LP #1783840. | |
| #2203 tests: add spread suite for kbuild plugin | snapcraft | merged | Merged into snapcraft master as 67d94bd4 (squashed commit f5d1b1741), adding tests/spread/plugins/kbuild and moving Kbuild plugin integration tests out of the Python unit suite; closes LP bug #1783835. | |
| #2210 tests: add spread suite for waf plugin | snapcraft | merged | Merged PR #2210 moved Waf plugin integration tests into a new Spread suite, resolving LP #1783846 in commit 5498d0aba. | |
| #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. |