test: remove artifact location workaround
Metadata
Current evaluation
Merged as commit bdfc9b8, removing the CI test artifact-location workaround after Snapcraft fixed artifact discovery; resolves #391.
Resolution: closed resolved
Reason: PR #392 was merged as commit bdfc9b8. It removed the obsolete artifact-location workaround from the CI test workflow and explicitly fixed #391.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Artifact location for tests is fixed in snapcraft and the workaround
can be removed from the CI test workflow.
Fixes #391
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged as commit bdfc9b8, removing the CI test artifact-location workaround after Snapcraft fixed artifact discovery; resolves #391. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #392 as commit bdfc9b8, removing the CI test artifact-location workaround after Snapcraft fixed artifact discovery; fixes #391. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged following approval and passing CI. Removed a CI test workflow workaround for artifact locations, as snapcraft resolved the underlying issue. Closes #391. |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Related To:
fetch-service#391
(confidence 100%)
Closed issue requesting removal of the artifact-location workaround after the Snapcraft fix.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #391 Remove test location workaround | fetch-service | closed | Removed the temporary test artifact-location workaround after craft-application fixed test output handling; merged in fetch-service#392 as bdfc9b8. | |
| #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. | |
| #4662 fix(test): do not create 2 snapcraft.yaml and confuse location | snapcraft | merged | Merged PR #4662 as commit a50b0617a, fixing test fixture paths so tests do not create duplicate root and /snap manifests that cause ambiguous hook lookup. | |
| #1849 tests: add snap not found tests | snapcraft | merged | Merged missing-snap test coverage; snapcraft PR #1849 landed as commit 54cf2bd9e. |