test: fix typo in spread extension
Metadata
Current evaluation
PR #415 was closed after approval was retracted; its typo fix was incorporated into merged PR #417 via commit 8363f15.
Resolution: closed superseded
Reason: The requested spread-test typo fix was superseded by PR #417, which corrected the extension and terminate-services tests and was merged as commit 8363f15.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Otherwise unattended upgrade is not stopped during the `snapcraft test` and can cause tests instabilities/noise.
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed superseded | PR #415 was closed after approval was retracted; its typo fix was incorporated into merged PR #417 via commit 8363f15. | |
| stealth/space-bunny-alpha | — | closed superseded | Closed after approval was retracted; its test typo fix was folded into merged PR #417, which corrected extension and terminate-services tests. | |
| qwen/qwen3.6-35b-a3b | — | — | Superseded by PR #417. A reviewer requested verifying service load state before checking if enabled, so the spread extension typo fix was implemented in the referenced PR instead. |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Superseded By:
canonical/fetch-service#417
(confidence 100%)
Merged follow-up PR containing the fix, committed as 8363f15.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #417 test: fix typos in extension and terminate services | fetch-service | merged | Merged PR #417 corrected typos in the spread extension and terminate-services test code, landing as commit 8363f15. | |
| #3662 tests: update spread url | snapcraft | merged | Merged PR #3662 updated Spread tarball URLs in two test files via commit 507959d70. | |
| #291 Refactor spread testing to use spread extensions | fetch-service | closed | Fixed by merged PR #299 (commit 44d9906), which refactored spread tests to generate spread.yaml and use the new spread extension framework. | |
| #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. | |
| #2400 tests: fix maven spread test | snapcraft | merged | Merged PR #2400 as commit 155306683, correcting the Maven spread test with a two-line change. | |
| #934 fix(templates): typo in spread .extension | rockcraft | merged | Merged PR #934 fixed the spread template’s `.extension` typo in commit d411f5db, resolving the report in canonical/rocks-template#13. | |
| #926 test: address a couple of non-flaky spread failures | rockcraft | merged | Merged PR #926 updated three spread-test configuration files to address persistent failures; all CI checks passed and two reviewers approved. | |
| #1366 tests(spread): make release test less flaky | charmcraft | merged | Merged #1366 at commit 05b18a03, reducing flakiness in the Spread release test and resolving #1362. | |
| #534 test(spread): workaround for flaky test | fetch-service | merged | Merged PR #534 (commit 4f89bb7) worked around #533 by skipping the flaky snap names-query assertion while retaining the other spread-test checks. | |
| #2762 tests/spread: fix reactive spread tests on older channels | charmcraft | closed | Closed unmerged after 86 days; no resolving commit or replacement PR was identified, and the Conventional Commits policy check remained failing. |