ci: correctly run from snap
Metadata
Current evaluation
Merged in #1803 via commit ab0075160, correcting CI to run tests from the snap without masking missing runtime dependencies through bundled development dependencies.
Resolution: closed resolved
Reason: PR #1803 was merged as commit ab0075160, making CI execute tests from the snap and separate development dependencies from the runtime environment so hidden failures are exposed.
Evaluated by: mr-cal-bot
No scores available.
Issue body
We loosly assume tests are using the snap to run, what is worse is that we setup all the development dependencies together with the runtime ones to hide this fact
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
- [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 in #1803 via commit ab0075160, correcting CI to run tests from the snap without masking missing runtime dependencies through bundled development dependencies. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #1803 (commit ab0075160), updating CI to run tests correctly from the snap and separating runtime from development dependencies, exposing hidden failures. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in #1803 via commit ab0075160, making tests run correctly from the snap and exposing hidden failures by removing development dependencies from the runtime environment. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged a CI fix to correctly run tests from snap by separating development and runtime dependencies. Approved by one reviewer, passed CI, and resolved dependency isolation to expose hidden bugs for future fixes. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged CI updates to correctly run tests from within a snap environment. The change stops bundling development dependencies with runtime ones to mask incorrect test assumptions, aiming to expose hidden bugs for future fixes. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2030 ci: build the snap on self-hosted runners | charmcraft | merged | Merged #2030 in commit 4df991fd, moving snap builds to self-hosted amd64 runners so containerized artifacts match downstream test architecture. |