tests: add spread suite for catkin plugin
Metadata
Current evaluation
Merged migration of Catkin plugin integration tests to a Spread suite, resolving Launchpad bug #1783828. Approved by one reviewer, passed CI, and maintained coverage across 37 files.
Suggested action: —
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: #1783828](https://bugs.launchpad.net/snapcraft/+bug/1783828) by moving the plugin and snapd integration tests for the Catkin plugin into a Spread suite.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged migration of Catkin plugin integration tests to a Spread suite, resolving Launchpad bug #1783828. Approved by one reviewer, passed CI, and maintained coverage across 37 files. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged by maintainer kyrofa, resolving Launchpad bug 1783828. The change migrates Catkin plugin integration tests to a Spread suite with no impact on code coverage. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged by kyrofa, resolving Launchpad bug #1783828 by migrating Catkin plugin integration tests to a Spread suite. Code coverage remained unchanged at 91.18% upon merge to master. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #1783829 Move Catkin Tools plugin tests to spread | snapcraft (launchpad) | closed | Closed and abandoned. The requested migration of Catkin Tools plugin integration tests to a Spread suite was not implemented or merged, with no resolution details provided. | |
| #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%. | |
| #2208 tests: add spread suite for scons plugin | snapcraft | merged | 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. | |
| #1783828 Move Catkin plugin tests to spread | snapcraft (launchpad) | closed | Closed as abandoned. The request to rewrite Catkin plugin integration tests as a Spread suite was never implemented, with no comments or follow-up recorded. | |
| #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. | |
| #2212 tests: add spread suite for ruby plugin | snapcraft | merged | Merged into master. Ruby plugin integration tests were consolidated into a new Spread suite, resolving LP: #1783840. The change modified 13 files, passed CI checks, and received reviewer approval. | |
| #2201 tests: add spread suite for nil plugin | snapcraft | merged | Merged to add a Spread test suite for Nil plugin integration tests, resolving Launchpad bug #1783838. Approved by reviewers, passed CI, and updated 8 files. | |
| #2209 tests: add spread suite for tar-content plugin | snapcraft | merged | Merged to master, resolving Launchpad bug #1783845 by migrating tar-content plugin integration tests to a Spread suite. Approved by one reviewer, passed CI, and increased coverage by less than 0.01%. | |
| #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. |