Move Godeps plugin tests to spread
Metadata
Current evaluation
Godeps integration tests were migrated from Python to Spread by merged snapcraft PR #2200, resolving LP #1783833 in commit 84932cc48.
Resolution: closed resolved
Reason: Merged PR canonical/snapcraft#2200 added a Spread suite for the Godeps plugin. Commit 84932cc48 is recorded as the resolution for LP #1783833.
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 Godeps snapd and plugins integration tests as a Spread suite for the Godeps plugin.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Godeps integration tests were migrated from Python to Spread by merged snapcraft PR #2200, resolving LP #1783833 in commit 84932cc48. | |
| stealth/space-bunny-alpha | — | closed resolved | Godeps integration tests were migrated from Python to Spread by merged PR #2200 in commit 84932cc48, resolving LP #1783833. | |
| stealth/space-bunny-alpha | — | closed resolved | Godeps integration tests were migrated to a Spread suite by merged PR canonical/snapcraft#2200 (84932cc48), resolving LP #1783833. | |
| stealth/space-bunny-alpha | — | closed resolved | Godeps plugin integration tests were moved to a Spread suite by merged PR #2200 (commit 84932cc48), resolving LP #1783833. | |
| qwen/qwen3.6-35b-a3b | — | — | Godeps plugin integration tests were migrated to a Spread suite, completing the requested refactor for improved parallelization and cross-system compatibility. | |
| qwen/qwen3.6-35b-a3b | — | — | Godeps plugin integration tests were migrated to a Spread suite to improve parallelization and cross-system compatibility. The task was completed and the issue closed as fixed. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed. The Godeps plugin integration tests were migrated from Python to a Spread suite, improving parallelization and cross-system compatibility as requested. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Closed. The Godeps plugin integration tests were migrated from Python to a Spread suite to improve parallelization and cross-system compatibility. |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Likely Fixed By:
canonical/snapcraft#2200
(confidence 100%)
Merged PR that migrated the Godeps plugin integration tests to Spread.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2200 tests: add spread suite for godeps plugin | snapcraft | merged | Merged #2200 in commit 84932cc48, moving Godeps plugin integration tests to a Spread suite and resolving LP #1783833. | |
| #1779419 Move Go plugin tests to spread | snapcraft (launchpad) | closed | Fixed by merged snapcore/snapcraft#2173 (commit ad7260ced), adding Spread infrastructure and porting the Go plugin integration test. | |
| #3431 godeps spread test: use latest/stable go snap | snapcraft | merged | Merged PR #3431 as commit f21a0132a, updating godeps spread tests to use the latest/stable Go snap instead of Xenial's Go 1.6. | |
| #1783830 Move CMake plugin tests to spread | snapcraft (launchpad) | closed | CMake plugin integration tests were converted to Spread in snapcore/snapcraft#2183, commit 51518dbe6. | |
| #1783837 Move Meson plugin tests to spread | snapcraft (launchpad) | closed | Meson plugin integration tests were moved from Python to a Spread suite by merged canonical/snapcraft#2205 (commit 19c651af4). | |
| #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. | |
| #691 Add godeps plugin. | snapcraft | merged | Godeps plugin merged into snapcraft via commit a073503c0 (PR #691), adding Go godeps dependency support and resolving LP bug #1595981; later enhanced by follow-up PRs. | |
| #2173 tests: create basic integration test spread infrastructure | snapcraft | merged | Merged PR #2173 in commit ad7260ced, adding Spread integration-test infrastructure and porting the Go plugin integration test; it resolved LP #1779419. | |
| #1783825 Move Autotools plugin tests to spread | snapcraft (launchpad) | closed | Autotools and snapd plugin integration tests were consolidated into a Spread suite by merged PR #2182 (commit eadd86729), resolving LP #1783825. | |
| #4249 spread: disable godeps tests | snapcraft | merged | Merged PR #4249 (aa3d5d67f), disabling spread godeps tests after golang/go#57051 broke builds while retaining the feature for a future fix. |