← Back to issue list

test: add Ruby plugin spread tests

View original Github issue

Metadata

Project
snapcraft
Number
#5976
Type
pull request
State
merged
Author
steinbro
Labels
—
Created
Updated
Closed

Current evaluation

Merged after review and CI approval. Adds Ruby plugin spread tests to resolve #5966. Retargeted to the hotfix/8.14 branch to align with the plugin's introduction.

Suggested action: —

No scores available.

Issue body

Fixes #5966 --- - [ ] I've followed the [contribution guidelines](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md). - [ ] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/). - [ ] I've successfully run `make lint && make test`. - [ ] I've added or updated any relevant documentation. - [ ] I've updated the relevant release notes.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged after review and CI approval. Adds Ruby plugin spread tests to resolve #5966. Retargeted to the hotfix/8.14 branch to align with the plugin's introduction.
qwen3.6-35b-a3b-mtp-q6 — — Merged into hotfix/8.14. Adds Ruby plugin spread tests and resolves a PATH conflict where mruby displaced the system Ruby executable during gem installation. Fixes #5966.
qwen3.6-35b-a3b-mtp-q6 — — Added Ruby plugin spread tests to resolve #5966. Identified a PATH conflict where mruby overwrote the system Ruby, causing gem failures. Tests confirmed for core22 and core24. Retargeted to hotfix/8.14 and merged.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#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.
83%
#1783840 Move Ruby plugin tests to spread snapcraft (launchpad) closed Closed without a linked resolution. The proposal to rewrite Ruby plugin integration tests as a Spread suite was abandoned.
77%
#4954 tests: update spread tests in hotfix/7.5 snapcraft merged Merged into hotfix/7.5 to update spread tests and repair CI. The branch includes six cherry-picks and six new commits, passed all CI and coverage checks, received reviewer approval, and unblocked issues #4950 and #4936.
76%
#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.
74%
#5966 Add ruby spread tests snapcraft open Request to add core22/core24 spread tests for the new craft-parts Ruby plugin. Work is done: merged PR snapcraft#5976 added the ruby-hello spread tests, which are present in the repo's build-and-run-hello suites.
73%
#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.
72%
#2199 tests: add spread suite for copy plugin snapcraft merged Merged into master following approval and passing CI. Moves copy plugin integration tests to a spread suite, resolving bug LP: #1783831. Modified 10 files, reduced code by 41 lines, and increased coverage by 0.02%.
72%
#3432 spread tests: remove legacy plugin tests snapcraft merged Merged removal of legacy plugin tests from the spread test suite. Deleted 665 lines across 29 files. Approved by reviewer, passed CI, and maintained coverage. Unrelated test failures were addressed in a separate PR.
72%
#5534 Add unit and spread tests for the .NET (v2) plugin snapcraft merged Merged after rebase and test fixes. Updates .NET plugin tests by removing outdated spread tests, adding core24 cases to registered plugin tests, and introducing a dotnet-hello spread test. All CI checks passed with two approvals.
72%
#2210 tests: add spread suite for waf plugin snapcraft merged Merged into master. Resolves LP #1783846 by migrating Waf plugin integration tests to a Spread suite. Approved by one reviewer, passed CI, and updated 19 files.
72%