Qiskit / qiskit / 25374000268

88%

Build Type

push

github

Committed by web-flow

Commit Message

Enable clippy on rust tests too (#16128)

Building off of the recently merged #16018 which ran cargo clippy with
--all-targets to fix some issues flagged in our test code, this commit
enables the --all-targets flag in CI. This will ensure our tests are
running linting too. The --all-targets flag is often paired with the
--all-features flag to ensure clippy is checking all code paths, however
for Qiskit we can't use the --all-features flag because the mimalloc
feature flags on qiskit-pyext and qiskit-cext are mutually exclusive so
we can't run clippy on the entire workspace with every feature enabled.
If we wanted this coverage we would have to run clippy per library with
the --all-features flag, but this doesn't make that change because it's
potentially slower than it's worth since we don't have very many
features. This commit does fix the underlying issues running clippy on
all-features catches (until the pyext build which fails during
compilation).

Coverage Stats

26 of 26 new or added lines in 2 files covered. (100.0%)

5 existing lines in 1 file now uncovered.

106720 of 121826 relevant lines covered (87.6%)

968421.51 hits per line