tests: use a temporary directory for snaps tests
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
LP: [#1643658](https://bugs.launchpad.net/snapcraft/+bug/1643658)
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master. Updated snap tests to use a temporary directory, resolving Launchpad bug #1643658. Maintainer approval received. Code coverage remained stable at 96.33% with 100% diff coverage. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1180 tests: support snap directory in external tests | snapcraft | merged | Merged after approval. Adds support for the snap directory in external tests. Briefly held due to an unrelated etcd directory breaking docker tests, then merged once resolved. | |
| #1259 tests: use launchpad as the source of the compressed test snap | snapcraft | merged | Merged a single-line change to configure tests to download the compressed test snap from Launchpad. The update required no review or CI validation. | |
| #4380 ci: download snap to tests dir | snapcraft | merged | Merged a CI fix that ensures scheduled spread tests download the snap to the tests directory. Approved by two reviewers, passed CI checks, and modified one file. | |
| #1849 tests: add snap not found tests | snapcraft | merged | Merged tests for missing snap scenarios. Approved by two reviewers and integrated into the codebase despite unresolved review comments and reported integration test timeouts. | |
| #1542 tests: add integration tests for build snaps | snapcraft | merged | Merged integration tests for build snaps. Approved by two reviewers and passed CI. Two test scenarios omitted due to known bugs. | |
| #2352 tests: use valid snap names in unit tests | snapcraft | merged | Merged updates to unit tests that replace invalid snap names with valid ones, ensuring compatibility with snap pack --check-skeleton. The author approved and merged the minor change directly without external review. | |
| #4755 tests: fix devel base snap test | snapcraft | merged | Merged a fix for the devel base snap test. The update ensures snapcraft correctly creates and utilizes the devel base. Approved by two reviewers, passed CI checks, and applied with a minimal one-file change. | |
| #2421 tests: remove obsolete snap and external tests | snapcraft | merged | Merged after maintainer approval, removing obsolete snap and external tests. Accepted despite minor coverage drop and unrelated CI failures, though later noted to break documentation. | |
| #1603 tests: add /snap/bin to PATH in autopkgtests | snapcraft | merged | Merged to add /snap/bin to the autopkgtests PATH, fixing binary lookup failures. Approved, passed CI, and depends on PR #1595. | |
| #3944 tests: fix electron spread test | snapcraft | merged | Merged a fix for a failing electron spread test. Updated the expected snap.yaml to include the TMPDIR environment variable, resolving a diff mismatch. |