tests: add snap not found tests
Metadata
Current evaluation
Merged tests for missing snap scenarios. Approved by two reviewers and integrated into the codebase despite unresolved review comments and reported integration test timeouts.
Suggested action: —
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 |
|---|---|---|---|---|
| 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into the main branch. Added tests for the snap not found scenario. The changes were accepted despite a contributor note about integration tests timing out. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #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. | |
| #2014 integration tests: snap tests shouldn't be arch-specific | snapcraft | merged | Merged a change removing architecture-specific constraints from snap integration tests. Approved by one reviewer with a two-line modification across a single file. | |
| #150 inspectors/snap: add additional inspector tests | fetch-service | merged | Merged after approval by two reviewers. The pull request adds additional inspector tests for snap. Successfully integrated into the main branch. | |
| #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. | |
| #706 Add integration tests using the snap | charmcraft | closed | Resolved by implementing the requested snap integration tests as part of the spread test suite. | |
| #336 examples_tests: added more tests after the snaps are installed | snapcraft | merged | Merged after adding post-installation tests for seven snap examples. Incorporated reviewer feedback to check service status instead of HTTP calls, resolved regressions, and received final approval. | |
| #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. | |
| #521 test(spread): add an improved snap inspector test | fetch-service | merged | Merged an improved snap inspector test that emulates system setup by stopping snapd, clearing device files, setting a global proxy, and restarting snapd to uncover network requests. Approved by two reviewers, passed all CI checks, and merged. | |
| #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. |