← Back to issue list

fix(TestingService): error better when we can't find spread

View original Github issue

Metadata

Project
craft-application
Number
#701
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #701 as bb2bda9c, improving the TestingService error when spread is missing from PATH; fixes #697.

Resolution: closed resolved

Reason: PR #701 was merged as commit bb2bda9c and closed issue #697. It now reports a clearer error when the spread executable cannot be found.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Fixes #697 - [x] Have you followed the guidelines for contributing? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `make lint && make test`? - [ ] Have you added an entry to the changelog (`docs/reference/changelog.rst`)? ---

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #701 as bb2bda9c, improving the TestingService error when spread is missing from PATH; fixes #697.
stealth/space-bunny-alpha — closed resolved Merged PR #701 as bb2bda9c, fixing #697 by replacing the missing-spread FileNotFoundError with a clearer TestingService error.
stealth/space-bunny-alpha — closed resolved Merged PR #701 as bb2bda9c, improving TestingService errors when spread is missing from $PATH and fixing issue #697.
qwen/qwen3.6-35b-a3b — — Merged improvement to TestingService error handling when spreads are missing, resolving #697. Approved by two reviewers and passed CI checks.

Update history

No update history recorded yet.

Related work

Related issues

Issue Project State Summary Similarity
#712 fix(TestingService): provide a better error if spread.yaml is missing craft-application merged Merged as commit f72dc340, PR #712 fixed #703 by giving TestingService a clearer error when spread.yaml is missing.
88%
#703 Bad error message if 'spread.yaml' is missing craft-application closed Fixed by merged PR #712 (commit f72dc340), which improved TestingService's error when spread.yaml is missing.
80%
#1083 ci: fix spread test missing PATH charmcraft merged Merged PR #1083 as commit 13c96b83, fixing the missing PATH in Spread tests; configure and remove tests then passed.
74%
#1029 fix(testing): error early if spread.yaml is invalid. craft-application merged PR #1029 merged (commit 93c27ec5): TestingService now reads and validates spread.yaml before starting the lifecycle, erroring early on invalid files. Changelog entry added under 6.3.0.
74%
#613 Execute spread testing craft-application closed Spread testing delivered: TestingService and an opt-in 'test' command merged via PR #694 (commit 4910b2d8), building on #565 and #592. Maintainer mr-cal closed the issue citing that PR.
72%
#6350 test: fix spread tests snapcraft merged Merged PR #6350 as a18c7f485, fixing all non-store Spread tests and OSV scanning, including pinning Black 26.3.1 to avoid patchelf failures.
71%
#415 test: fix typo in spread extension fetch-service closed PR #415 was closed after approval was retracted; its typo fix was incorporated into merged PR #417 via commit 8363f15.
71%
#6183 test(spread): debug spread failure snapcraft closed Debug-only spread panic investigation was closed unmerged; follow-up #6198 reproduced a dangling-symlink failure but was also closed without delivering a fix.
71%
#723 fix(testing): more intelligent test handling craft-application merged Merged PR #723 via commit 6b319b77, running Spread tests as root and strictly validating only top-level spread.yaml keys.
71%
#717 fix(test): minor test fixes craft-application merged Merged PR #717 (commit 9f02b30f) switched spread tests to streaming output and adopted Spread's new reroot key, fixing #704 and #715.
70%