tests: support snap directory in external tests
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
LP: [#1671620](https://bugs.launchpad.net/snapcraft/+bug/1671620)
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Updates external tests to support the snap directory. Temporarily held due to a conflicting snap directory in etcd breaking docker tests, then confirmed readiness and merged. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged after addressing a false positive snap directory in etcd that broke docker tests. The change enables external tests to properly handle snap directories, resolving Launchpad bug #1671620. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1068 tests: use a temporary directory for snaps tests | snapcraft | merged | 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. | |
| #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. | |
| #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. |