test: update craftctl test
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
Updates the spread test to validate the fix for https://github.com/canonical/snapcraft-rocks/issues/33
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Updated spread test to validate the fix for canonical/snapcraft-rocks#33, verifying craftctl behavior and confirming the related issue resolution. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #386 tests: update testing to use craft-test | fetch-service | merged | 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). | |
| #1091 docs: revamp craftctl how-to | craft-parts | merged | Merged documentation update revamping the craftctl how-to guide using upstream Snapcraft content. Approved by two reviewers, CI checks passed after fixes, and changelog updated for a minor release prior to merge. | |
| #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. | |
| #6048 docs: correct 'craftctl' to 'snapctl' in config doc | snapcraft | merged | Merged a documentation fix correcting craftctl to snapctl in the config guide. Approved by two reviewers with passing CI checks. The update modifies one file. | |
| #50 test: test the snapcraft actions | craft-actions | merged | Merged adding tests for snapcraft actions, mirroring charmcraft tests. Approved by two reviewers with all CI checks passing. Updates two files with 89 additions and 1 deletion. | |
| #717 fix(test): minor test fixes | craft-application | merged | Merged minor test fixes replacing spread test pausing with a stream approach and adopting the new reroot key. Resolves issues #704 and #715, plus CRAFT-4399 and CRAFT-4411. Approved by two reviewers with all CI checks passing. | |
| #173 tests: check before handling rockcraft instances | rockcraft | closed | Merged after two approvals and passing CI. Updates spread test cleanup to conditionally stop and delete rockcraft lxc instances, preventing errors when tests do not create them. (+5/-2 lines) | |
| #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. | |
| #5529 fix: remove test artifact workaround | snapcraft | merged | Merged a fix that removes the snapcraft test artifact location workaround, as craft-application 3.5 now handles this natively. Approved by two reviewers, passed all CI checks, and modified five files. | |
| #1098 build: update craft-providers | rockcraft | merged | Merged update to the craft-providers dependency to resolve recurring snapd watch errors in spread tests. Approved by two reviewers and passed CI checks. The change modifies three files with minor adjustments. |