← Back to issue list

test: rust plugin unit tests - remove system requirements

View original Github issue

Metadata

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

Current evaluation

Merged after three approvals. The update removes the local Rust installation requirement for running rust plugin unit tests. Initial CI code-style failures were fixed prior to successful merge.

Suggested action:

No scores available.

Issue body

- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? ----- This ensures that you don't have to have rust on your system in order to run the rust plugin unit tests

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged after three approvals. The update removes the local Rust installation requirement for running rust plugin unit tests. Initial CI code-style failures were fixed prior to successful merge.
qwen3.6-35b-a3b-mtp-q6 Merged changes removing the system Rust dependency for running rust plugin unit tests. The update passed CI after addressing a code-style check failure.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2431 rust plugin: update to use latest rustup command options snapcraft merged Merged update to the Rust plugin replacing deprecated rustup command options with current equivalents. Approved by a reviewer, passed CI, modified five files, and maintained 94.5% test coverage.
74%
#3376 tests: add missing mock for rust unit tests snapcraft merged Merged a fix for failing cross-compilation unit tests when rustup is present. The change mocks os.path.exists() to return False during the pull step, preventing test failures. Approved by maintainer and passed all CI checks.
71%
#2434 rust plugin: refactor to use the latest rustup snapcraft merged Merged Rust plugin refactor for latest rustup. Removed deprecated flags, added command qualifiers, established shared rustup/cargo directory, and standardized config. Approved and passed CI.
70%