← Back to issue list

tests: add spread suite for ament plugin

View original Github issue

Metadata

Project
snapcraft
Number
#2211
Type
pull request
State
merged
Author
kyrofa
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #2211 as commit bdaa8ea5, replacing the Ament plugin snapd integration test with a Spread suite and resolving LP #1783827.

Resolution: closed resolved

Reason: The approved PR merged into master as commit bdaa8ea52, moving Ament plugin integration coverage from snapd tests into a Spread suite.

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: #1783827](https://bugs.launchpad.net/snapcraft/+bug/1783827) by moving the snapd integration test for the Ament plugin into a Spread suite.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #2211 as commit bdaa8ea5, replacing the Ament plugin snapd integration test with a Spread suite and resolving LP #1783827.
stealth/space-bunny-alpha — closed resolved Merged #2211 in commit bdaa8ea5, replacing the Ament plugin snapd integration test with a Spread suite and resolving LP #1783827.
qwen/qwen3.6-35b-a3b — — 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.
qwen3.6-35b-a3b-mtp-q6 — — Merged into master, resolving LP #1783827 by migrating the Ament plugin snapd integration test to a Spread suite. The change resulted in a negligible 0.02% decrease in overall code coverage.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1783827 Move Ament plugin tests to spread snapcraft (launchpad) closed Added a Spread suite for the Ament plugin in snapcore/snapcraft#2211 (commit bdaa8ea52), completing the requested test migration.
86%
#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.
85%
#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.
84%
#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.
84%
#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.
83%
#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.
82%
#2208 tests: add spread suite for scons plugin snapcraft merged 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).
82%
#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.
82%
#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.
80%
#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.
79%