Don't litter /tmp with test artifacts.
Metadata
Current evaluation
Merged as de28a767b: made source/origin download locations mockable so tests could clean up artifacts instead of littering /tmp.
Resolution: closed resolved
Reason: PR #808 was approved and merged into canonical/snapcraft as commit de28a767b, resolving the test-artifact cleanup problem by making download paths mockable.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- Make the download location for source/origins mockable to
make cleaning up after tests easier.
LP:#1624099
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged as de28a767b: made source/origin download locations mockable so tests could clean up artifacts instead of littering /tmp. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in canonical/snapcraft#808 as commit de28a767b, making source/origin download locations mockable so tests can clean up artifacts instead of littering /tmp. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #808 made source/origins download locations mockable so tests can clean up artifacts instead of leaving them in /tmp; reviewer approved the requested changes. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged. Changes make the download location for source/origins mockable to prevent test artifacts from littering /tmp. Reviewers approved the updates, and the pull request was successfully merged. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged changes that make test download locations mockable, preventing test artifacts from cluttering /tmp. Author implemented requested updates and received approval before merge. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1624099 parser - Clean up tmp files from tests. | snapcraft (launchpad) | closed | Tests littered /tmp with downloaded origins; fixed by making source/origin download locations mockable so tests clean up artifacts. Merged in snapcore/snapcraft#808 (commit de28a767b). | |
| #684 Run tests from /tmp | snapcraft | merged | Merged as commit c6db1cd3a, updating the test setup to run tests from /tmp. |