test: add tox environments for snapping and running spread tests
Metadata
Current evaluation
Approved but never merged; the tox convenience change was closed unmerged after prolonged inactivity, with README updates and four review comments unresolved.
Resolution: closed stale
Reason: canonical/rockcraft#215 was closed without merging after more than three years of inactivity. The README request from 2023-03-24 and four unresolved review comments remained unfinished.
Evaluated by: mr-cal-bot
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 |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed stale | Approved but never merged; the tox convenience change was closed unmerged after prolonged inactivity, with README updates and four review comments unresolved. | |
| stealth/space-bunny-alpha | — | closed stale | The tox test-spread convenience change was approved but closed without a merge after prolonged inactivity; the reviewer’s README update request remained unresolved. | |
| 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. |
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 #1484, which adds tox execution to the lifecycle spread test and fixes #1471. | |
| #312 Enable tox in rtd | rockcraft | closed | Closed unmerged after more than a year without reviews, CI, or comments; the proposed Read the Docs tox integration was not accepted. | |
| #1553 Investigate tox linting issues not being picked up in spread tests. | charmcraft | closed | Closed with no recorded fix; tox lint failures were not wired into the full-lifecycle spread test. | |
| #4197 WIP: tox CI updates | snapcraft | closed | Closed unmerged after prolonged inactivity; the WIP tox CI update received no reviews, comments, or CI results. | |
| #616 build: use tox for unit tests in CI | craft-parts | merged | Merged PR #616 switched unit-test CI to tox; it was approved by two reviewers and landed as commit bd6c687f. | |
| #2416 fix(test): fix lifecycle spread tests for tox-uv | charmcraft | merged | Merged PR #2416 as 4fbe5e18, restoring tox-uv setup omitted during init-template updates so lifecycle spread tests pass on Ubuntu 24.04. | |
| #4036 tests: ensure coverage is set per tox env | snapcraft | merged | Merged #4036 in commit ef29278e2, setting test coverage independently for each tox environment. |