← Back to issue list

build: use tox for unit tests in CI

View original Github issue

Metadata

Project
craft-parts
Number
#616
Type
pull request
State
merged
Author
lengau
Labels
Created
Updated
Closed

Current evaluation

Merged. CI configuration updated to use tox for unit tests. Approved by two reviewers with passing checks. Modified two files (+66/-10).

Suggested action:

No scores available.

Issue body

- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged. CI configuration updated to use tox for unit tests. Approved by two reviewers with passing checks. Modified two files (+66/-10).
qwen3.6-35b-a3b-mtp-q6 Merged changes to replace the CI unit testing framework with tox, standardizing test execution and simplifying the build pipeline.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#4273 docs: fix tox doc builds snapcraft merged Merged to main after reviewer approval and passing CI. The change fixes tox documentation builds by modifying a single file (+2/-12 lines), with no impact on code coverage.
82%
#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.
81%
#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.
80%
#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.
80%
#4507 ci: update PR template to use tox commands snapcraft merged Merged a two-line update to the pull request template, replacing make and pytest commands with tox. Approved by two reviewers, passed CI checks, and maintained 89.22% test coverage.
78%
#1755 fix(ci): fix tox ensure_version_matches charmcraft merged Merged after approval by two reviewers. Resolved a CI failure in the tox ensure_version_matches check with a single-line adjustment. The fix was successfully integrated into the codebase.
77%
#67 lint: fix ruff linting in tox craft-archives merged Merged after reviewer approval and passing CI. Resolves ruff linting issues in the tox environment by updating three files. Code coverage remains unchanged at 88.07%.
77%
#102 tox: pass CI environment variable starbase merged Merged after approval by two reviewers and passing CI checks. The change upstreams a modification to pass the CI environment variable to tox, adding two lines to one file.
76%
#89 tox/ci: update Python versions tested starbase merged Merged after approval and passing CI checks. Updated tox and CI configuration to test newer Python versions. The change modified two files with a net reduction of one line. Code coverage remained at 100%.
76%
#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.
76%