tests: add snap not found tests
Metadata
Current evaluation
Merged missing-snap test coverage; snapcraft PR #1849 landed as commit 54cf2bd9e.
Resolution: closed resolved
Reason: PR #1849 was approved and merged, adding tests for missing-snap scenarios. Repository history records the merge as commit 54cf2bd9e.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)?
- [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [x] Have you successfully run `./runtests.sh static`?
- [x] Have you successfully run `./runtests.sh unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged missing-snap test coverage; snapcraft PR #1849 landed as commit 54cf2bd9e. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #1849 as commit 54cf2bd9e, adding coverage for missing-snap scenarios. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged snapcore/snapcraft#1849 in commit 54cf2bd9e, adding coverage for missing-snap scenarios. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged tests for missing snap scenarios. Approved by two reviewers and integrated into the codebase despite unresolved review comments and reported integration test timeouts. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Added integration tests for the snap not found scenario. Changes were accepted despite a contributor note regarding integration test timeouts. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1625 tests: use the snapcraft snap for integration tests | snapcraft | merged | Merged PR #1625 updated integration tests to run the installed Snapcraft snap, better matching the user environment. Resolution: commit eebb070e6. | |
| #4755 tests: fix devel base snap test | snapcraft | merged | Merged #4755, fixing the devel-base integration test to verify Snapcraft can create the base snap and that the resulting devel base is usable. | |
| #1411 tests: build the snapcraft snap in travis tests | snapcraft | merged | Merged #1411 via commit 92c1e43b1, adding Travis coverage that builds the Snapcraft snap after LXD test infrastructure landed in #1415. | |
| #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. | |
| #3609 tests: move legacy snapcraft tests (CRAFT-728) | snapcraft | merged | Merged PR #3609 in commit a71aeff54, moving snapcraft_legacy unit tests from tests/unit to tests/legacy to match the package rename. | |
| #46 tests: add fake snapd fixture | craft-parts | merged | Merged PR #46 as 64fdf574 (feature commit 9db65074), adding a 212-line fake snapd fixture for testing snap package handling in craft-parts.packages. | |
| #1877 tests: move test files out of the snapcraft dir | snapcraft | closed | Closed unmerged after conflicts; the test reorganization was redone and merged via snapcraft#1948 at commit 55ad11ccc, moving tests out of the snapcraft package. | |
| #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. | |
| #2351 tests: use valid snap names in unit tests | snapcraft | closed | Closed unmerged; the same test-only change was merged instead via PR #2352 (commit b8c0a4fbf), updating unit-test snap names to pass `snap pack --check-skeleton`. | |
| #1838 tests: fix broken rust test snap | snapcraft | merged | Merged in snapcraft#1838 as 0fe7def33, removing an unnecessary unversioned log dependency that broke the rust-with-revisions test snap. |