tests: ensure coverage is set per tox env
Metadata
Current evaluation
Merged #4036 in commit ef29278e2, setting test coverage independently for each tox environment.
Resolution: closed resolved
Reason: PR #4036 was merged into main as commit ef29278e2. It updates the test configuration to set coverage separately for each tox environment.
Evaluated by: mr-cal-bot
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/)?
- [ ] Have you successfully run `make lint`?
- [ ] Have you successfully run `pytest tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged #4036 in commit ef29278e2, setting test coverage independently for each tox environment. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in #4036 (commit ef29278e2), updating tests to set coverage separately for each tox environment. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #4036 (commit ef29278e2), configuring test coverage separately for each tox environment. | |
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into main. Updates test configuration to ensure coverage is correctly scoped per tox environment. Codecov verified no coverage impact upon merge. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #32 tools: improve coverage support in the tox test environments | starbase | merged | Merged PR #32 improved coverage handling across the tox test environments. | |
| #1674 fix: make tox tests succeed | charmcraft | merged | Merged in commit b24ccd5b: fixed the tox environment variable and added a workaround for canonical/operator#1212, allowing tox tests to succeed. | |
| #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. | |
| #215 test: add tox environments for snapping and running spread tests | rockcraft | closed | Approved but never merged; the tox convenience change was closed unmerged after prolonged inactivity, with README updates and four review comments unresolved. | |
| #13 tests: Ensure coverage gets per-environment filenames | starbase | merged | Merged PR #13 added test configuration for distinct per-environment coverage filenames; change landed via commit 28bb9d8 and merge commit 7e30bab. | |
| #332 tox: improve UX for testing | craft-providers | merged | Merged PR #332 as commit 38739ea, updating tox configuration to improve the local testing experience. | |
| #1423 build: use tox for tests | charmcraft | merged | Merged #1423 in commit 8b291de3, moving test execution to tox so tox handles Python versions while CI matrices cover operating systems. |