← Back to issue list

tests: update testing to use craft-test

View original Github issue

Metadata

Project
fetch-service
Number
#386
Type
pull request
State
merged
Author
cmatsuoka
Labels
—
Created
Updated
Closed

Current evaluation

Merged. Replaced direct spread calls with snapcraft test in workflows and spread.yaml, requiring a core24 base update. Approved by two reviewers and modified 10 files (+56/-125 lines).

Suggested action: —

No scores available.

Issue body

Update spread.yaml and github workflows to use snapcraft test instead of calling spread directly. Updating to base core24 is needed to use snapcraft test. - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged. Replaced direct spread calls with snapcraft test in workflows and spread.yaml, requiring a core24 base update. Approved by two reviewers and modified 10 files (+56/-125 lines).

Update history

Date Change
created

Related issues

Issue Project State Summary Similarity
#102 test: update craftctl test snapcraft-rocks merged Merged after two approvals and passing CI checks. Updates the spread test to validate the fix for snapcraft-rocks#33, modifying one file with seven added lines.
82%
#1272 tests: initial setup for the snapcraft snap tests with spread snapcraft merged Merged to master following single reviewer approval. Introduced initial spread test infrastructure for snapcraft across four files (+39 lines). Code coverage remained stable at 95.33%.
80%
#1325 test(spread): consolidate basic init/build tests charmcraft merged Merged after approval by two reviewers. Consolidates basic init and build spread tests across four files, reducing 80 lines and adding 28. Resolves CRAFT-2127.
79%
#2793 tests: enable SNAPCRAFT_BUILD_INFO for spread snapcraft merged Merged following approval and passing CI. Enabled SNAPCRAFT_BUILD_INFO for spread tests to cover manifest changes. Included a fix for the nil-basic spread test. Minimal diff across three files.
78%
#1386 fix: final spread test fixes for craft-application charmcraft merged Merged after approval by two reviewers. Resolves final spread test issues for craft-application across seven files, removing 127 lines and adding 104. Changes integrated into the main branch.
77%
#13 test(spread): add more spread tests snapcraft-rocks merged Successfully merged spread tests covering architectures, chisel slices, craftctl, package-repositories, and patchelf. Approved by two reviewers across 15 files (+206/-5 lines).
76%
#16 tests: add spread support rockcraft merged Merged after approval and passing CI, adding spread support to tests per CRAFT-865.
76%
#1381 fix: fix most spread tests on craft-application charmcraft merged Merged updates fixing most spread tests for craft-application. Two smoketests remain broken. Approved despite minor linting and unit test failures. Resolves CRAFT-2239.
75%
#1287 fix(tests): fixes bundle spread test charmcraft merged Merged. Resolves issue #1262 and CRAFT-2052 by fixing the bundle spread test. Replaced a failing charm path check with Python-based YAML validation. Approved by two reviewers, passed CI, and successfully executed the large spread test.
75%
#6164 test(spread): add core26 package-cutoff test snapcraft merged Merged. Adds a core26 package-cutoff test as a downstream test for craft-parts PR #1538. Incorporates changes from snapcraft PR #6171 and resolves issues #6146 and #6148. Approved by two reviewers and merged after passing CI checks.
75%