tests: replace the first batch of demo tests with snapd integration tests
Metadata
Current evaluation
Merged after reviewer consensus. Replaces outdated demo tests with a dedicated snapd integration suite, adding Travis compatibility and streamlining future hook and config validation.
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`?
-----
The mosquitto demo is not showing anything interesting, and it's test is not going to catch any bugs.
Instead of that, I have added a new suite of tests for the integration with snapd, that will execute a python hello world, showing that a python snap built with snapcraft can be executed.
The same with the others. I replaced them with installing and running a hello world.
To this new suite now we can add more interesting things like hooks, and config. With the added benefit that it will run on travis, so we will catch bugs earlier.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged after reviewer consensus. Replaces outdated demo tests with a dedicated snapd integration suite, adding Travis compatibility and streamlining future hook and config validation. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged after replacing demo tests with snapd integration tests. Reviewers approved a dedicated suite for Travis CI. An opencv test was temporarily reverted per maintainer request before final approval and merge. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged after maintainers approved a dedicated integration test suite for better CI execution. Replaces outdated demo tests with snapd integration tests running a Python hello world, with a temporary opencv test reverted. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1596 tests: move ruby demo test to snapd integration suite | snapcraft | merged | Merged after approval and passing CI. Relocated the ruby demo test to the snapd integration suite and corrected a command in the ruby-hello snap. Modified four files with minimal line changes. | |
| #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. | |
| #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. | |
| #2353 tests: remove dependency on snapcraft for integration tests | snapcraft | merged | Merged after reviewer approval and passing CI. The change removes the snapcraft dependency from integration tests to improve test independence, modifying 27 files with 956 additions and 733 deletions. | |
| #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. | |
| #46 tests: add fake snapd fixture | craft-parts | merged | Merged after two approvals and passing CI. Introduces a fake snapd fixture to test snap package handling in craft-parts.packages. Adds 212 lines across two files. Windows compatibility limitations were noted for separate follow-up. | |
| #706 Add integration tests using the snap | charmcraft | closed | Resolved by implementing the requested snap integration tests as part of the spread test suite. |