test: add tox environments for snapping and running spread tests
Metadata
Current evaluation
Approved and closed. Added tox environments to streamline snap and spread test execution. A reviewer suggested updating the README to document the new workflow.
Suggested action: —
No scores available.
Issue body
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
This was convenience thing for me, as now I can run `tox run -m test-spread` to build the snap and run the spread tests.
@tigarmo lmk what you think. If you like this I think I want to incorporate it into starbase too
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Approved and closed. Added tox environments to streamline snap and spread test execution. A reviewer suggested updating the README to document the new workflow. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Closed after positive review; reviewer recommended updating the README to document the new tox environments for snap building and spread testing. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Closed without merging. Added tox environments for snap and spread testing. A reviewer requested a README update to document the new workflow, but the changes were never applied and the PR was closed. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1484 fix(spread): run tox in lifecycle spread test | charmcraft | merged | Merged. Resolves #1471 by configuring tox execution within the lifecycle spread test. Approved by two reviewers with eight lines added. | |
| #1423 build: use tox for tests | charmcraft | merged | Merged after dual approval. Replaced existing test configuration with tox, shifting Python version management to the tool while keeping the CI matrix OS-focused. Streamlined testing setup and reduced configuration lines. | |
| #4036 tests: ensure coverage is set per tox env | snapcraft | merged | Merged a single-file change (+3/-2 lines) to set test coverage per tox environment. Approved by a reviewer, passed CI, and integrated without changing coverage metrics. | |
| #576 build: fix tox test environments | craft-parts | merged | Merged a one-file change fixing tox test environments in the build configuration. Approved by two reviewers with zero unresolved comments. The update adjusted three lines to resolve environment setup issues. | |
| #3026 spread tests: add core20 and cleanup systems | snapcraft | merged | Merged. Added core20 support to spread tests, cleaned up system selection logic to exclusively install snaps, and introduced a dedicated environment for unit tests. A reported test failure was promptly fixed by the author before approval and merge. | |
| #32 tools: improve coverage support in the tox test environments | starbase | merged | Merged improvements to coverage support in tox test environments. The change modifies two files and was approved by one reviewer prior to integration. | |
| #2745 tests: update spread tests to account for content snaps | snapcraft | merged | Merged updates to spread tests for gnome and kde extensions to handle content snaps and use gtk. Removed unnecessary snap cleanup logic per reviewer feedback, relying on restore-each for environment purging. Added a NOMATCH helper. Approved and merged. |