ci: download snap to tests dir
Metadata
Current evaluation
Merged a CI fix that ensures scheduled spread tests download the snap to the tests directory. Approved by two reviewers, passed CI checks, and modified one file.
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] Have you successfully run `make lint`?
- [x] Have you successfully run `pytest tests/unit`?
-----
The scheduled spread tests were not downloading the snap to the `tests/` directory.
I triggered a run and it is working now. See logs here: https://github.com/snapcore/snapcraft/actions/runs/6313456368
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged a CI fix that ensures scheduled spread tests download the snap to the tests directory. Approved by two reviewers, passed CI checks, and modified one file. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into main. Resolved CI spread test failures by ensuring the snap downloads to the tests directory. Verified working via CI logs. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into main to resolve CI spread tests failing to download the snap to the tests directory. Codecov reported a 0.01% coverage decrease upon merge. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #432 ci: speed up spread testing workflow | fetch-service | merged | Merged CI workflow optimization that decouples snap-test from snap-build, reducing pipeline latency. The build job remains for artifact retrieval. Approved by two reviewers, passed all checks, and modified one file. | |
| #1803 ci: correctly run from snap | snapcraft | merged | 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. | |
| #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. | |
| #3476 ci: reduce amount of artifacts to upload for spread | snapcraft | merged | Merged a CI optimization that reduces spread upload artifacts to the tests directory, decreasing payload size. Local execution now requires copying the snap to the test directory. Approved and passed all checks. | |
| #2900 ci: publish the CI built snap to the Snap Store | snapcraft | merged | Merged into master. Adds CI configuration to automatically publish built snaps to the Snap Store. Approved by a maintainer, passed all checks, and introduced 29 lines across two files with 100% diff coverage. | |
| #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. | |
| #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. | |
| #5845 test(spread): test building base and snapd snaps | snapcraft | merged | Merged spread tests for building base and snapd snaps. Despite initial CI failures linked to snapcraft issue #5843, the changes were approved and integrated to maintain build compatibility with mainline snapcraft. | |
| #3091 ci: install the snapd snap when preparing spread systems | snapcraft | merged | Merged CI configuration changes to install the snapd snap during spread system preparation. The update modifies two files with a two-line addition to streamline test environment setup. | |
| #352 ci(snap): publish snap for PRs and feature branches | rockcraft | merged | Merged CI workflow update to publish Snap packages for pull requests and feature branches. Approved by two reviewers, changing one file with 25 additions and one deletion. |