Remove test location workaround
Metadata
Current evaluation
Removed the temporary test artifact-location workaround after craft-application fixed test output handling; merged in fetch-service#392 as bdfc9b8.
Resolution: closed resolved
Reason: PR #392 was merged as commit bdfc9b8 and removed the workaround introduced by #386. It followed the upstream fix for craft-application#752, delivered by #758 (f411bf42).
Evaluated by: mr-cal-bot
No scores available.
Issue body
### What needs to get done
Once canonical/craft-application#752 is published, remove workarounds introduced in #386
### Why it needs to get done
To keep the codebase clean.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Removed the temporary test artifact-location workaround after craft-application fixed test output handling; merged in fetch-service#392 as bdfc9b8. | |
| stealth/space-bunny-alpha | — | closed resolved | Removed the test artifact-location workaround in canonical/fetch-service#392, commit bdfc9b8, after the upstream fix landed in craft-application. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed as completed after upstream craft-application#752 was published. The test location workaround from #386 was removed to clean up the codebase. |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Likely Fixed By:
canonical/fetch-service#392
(confidence 100%)
Merged PR that removed the artifact-location workaround and resolved #391.
-
Caused By:
canonical/fetch-service#386
(confidence 100%)
Introduced the workaround that #391 requested be removed after the upstream fix.
-
Related To:
canonical/craft-application#752
(confidence 100%)
Upstream issue fixing test output-location behavior, explicitly cited by #391.
-
Likely Fixed By:
canonical/craft-application#758
(confidence 98%)
Fixed #752 in commit f411bf42, enabling removal of the fetch-service workaround.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #5529 fix: remove test artifact workaround | snapcraft | merged | Merged PR #5529 removed the test-artifact lookup workaround after craft-application 3.5, landing as commit 8fbe76827. | |
| #758 fix(testing): don't change the output artifact location | craft-application | merged | Merged in canonical/craft-application#758 via commit f411bf42, fixing #752 by keeping test-command artifacts in the standard output location instead of a spread subdirectory. | |
| #392 test: remove artifact location workaround | fetch-service | merged | Merged as commit bdfc9b8, removing the CI test artifact-location workaround after Snapcraft fixed artifact discovery; resolves #391. | |
| #752 Test command should not modify output location | craft-application | closed | Fixed in #758 (commit f411bf42): test no longer redirects pack output to spread/, leaving the artifact in the project root for craft-test. |