fix(TestingService): provide a better error if spread.yaml is missing
Metadata
Current evaluation
Merged as commit f72dc340, PR #712 fixed #703 by giving TestingService a clearer error when spread.yaml is missing.
Resolution: closed resolved
Reason: PR #712 was approved and merged into the project. Its commit f72dc340 implements the missing-spread.yaml error handling and closes issue #703.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Fixes #703
- [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`?
- [x] 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 as commit f72dc340, PR #712 fixed #703 by giving TestingService a clearer error when spread.yaml is missing. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #712 (f72dc340), improving TestingService's error when spread.yaml is missing and fixing #703. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged update that improves error messaging when spread.yaml is missing in the TestingService. Approved by two reviewers, passed CI checks, and includes a changelog entry. Resolves #703. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/craft-application#703
(confidence 100%)
The PR explicitly fixes this missing spread.yaml error-message issue.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #701 fix(TestingService): error better when we can't find spread | craft-application | merged | Merged PR #701 as bb2bda9c, improving the TestingService error when spread is missing from PATH; fixes #697. | |
| #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. | |
| #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. |