tests: use a temporary directory for snaps tests
Metadata
Current evaluation
Merged PR #1068 made snaps tests use a temporary directory, fixing Launchpad bug #1643658; landed as commit 1d4e5878d.
Resolution: closed resolved
Reason: The test isolation change was merged through PR #1068 and recorded as commit 1d4e5878d. The PR explicitly links Launchpad bug #1643658 as the problem addressed.
Evaluated by: mr-cal-bot
No scores available.
Issue body
LP: [#1643658](https://bugs.launchpad.net/snapcraft/+bug/1643658)
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #1068 made snaps tests use a temporary directory, fixing Launchpad bug #1643658; landed as commit 1d4e5878d. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #1068 changed snaps tests to use a temporary directory, resolving Launchpad #1643658 in commit 1d4e5878d. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #1068 changed snaps tests to use a temporary directory, resolving Launchpad #1643658. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged into master. Updated snap tests to use a temporary directory, resolving Launchpad bug #1643658. Approved by reviewer, passed CI, and maintained full test coverage. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master. Resolves LP #1643658 by routing snap tests to a temporary directory. Maintains 100% diff coverage with no change to overall metrics. Approved and integrated. |
Update history
No update history recorded yet.
Related work
-
Related To:
snapcraft (launchpad)#1643658
(confidence 98%)
Launchpad bug linked from the PR body as the target being fixed.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1180 tests: support snap directory in external tests | snapcraft | merged | Merged PR #1180 added support for a snap directory in external tests, resolving LP bug #1671620 in commit b52d0e743. | |
| #1259 tests: use launchpad as the source of the compressed test snap | snapcraft | merged | Merged PR #1259 via commit 2409c9eec, switching the compressed test-snap source used by tests to Launchpad. | |
| #1643658 snaps tests assume ubuntu user | snapcraft (launchpad) | closed | Fixed by merged PR snapcore/snapcraft#1068 (commit 1d4e5878d), replacing the hardcoded /home/ubuntu test destination with a temporary directory. | |
| #2352 tests: use valid snap names in unit tests | snapcraft | merged | Merged PR #2352 (commit b8c0a4fbf) changed snap names used in unit tests so they pass snap pack --check-skeleton; author merged directly as a minor test-only change. | |
| #4380 ci: download snap to tests dir | snapcraft | merged | Merged PR #4380 as commit 0de1f0e1d, fixing scheduled Spread tests to download the snap into the tests/ directory. | |
| #1542 tests: add integration tests for build snaps | snapcraft | merged | PR #1542 merged as 8c9900f38, adding integration coverage for build snaps; two scenarios remained excluded due to Launchpad bugs #1716054 and #1716055. | |
| #1652 tests: fork skip into snaps_tests | snapcraft | merged | PR #1652 (merged as commit c951a3679) forked a skipped test into snaps_tests to avoid overly broad snapcraft.tests imports that broke autopackage tests. | |
| #3331 spread tests: move package-repositories test snaps into own dir | snapcraft | merged | Merged PR #3331 (commit bfcebf9ec), moving package-repositories test snaps into a dedicated directory for easier navigation. | |
| #990 tests: fix snaps tests in armhf | snapcraft | merged | Merged PR #990 as commit f467e55bc, correcting armhf failures in snap tests across 11 files and closing Launchpad bug #1650458. |