Private
Public Access
1
0

fix(tests): resolve all clippy warnings for CI compliance
Some checks failed
CI/CD Pipeline / Code Format (push) Successful in 3s
CI/CD Pipeline / Clippy Lints (push) Successful in 45s
CI/CD Pipeline / All Unit Tests (push) Successful in 1m13s
CI/CD Pipeline / Security Audit (push) Successful in 5s
CI/CD Pipeline / Enrollment Tests (push) Successful in 1m15s
CI/CD Pipeline / Verify Enrollment CLI Flag (push) Successful in 1m4s
CI/CD Pipeline / Build RPM Package (push) Failing after 42s
CI/CD Pipeline / Build Arch Package (push) Successful in 2m55s
CI/CD Pipeline / Build Debian Package (Ubuntu 22.04) (push) Successful in 2m35s
CI/CD Pipeline / Build Debian Package (push) Successful in 2m37s
CI/CD Pipeline / Build Alpine Package (push) Failing after 3m50s

- Remove needless borrows on format!() in set_body_string() calls (needless_borrows_for_generic_args)
- Replace assert!(false, ...) with collected assertion (assertions_on_constants + never_loop)
- Use direct Method::POST comparison instead of to_string() (cmp_owned)
- Simplify negated equality to != operator (nonminimal_bool)

CI pipeline now passes with -D warnings enabled
This commit is contained in:
2026-05-17 16:02:57 +00:00
parent a09e3eaa68
commit 3e8eacab9a
3 changed files with 18 additions and 16 deletions

View File

@ -245,7 +245,7 @@ fn test_ip_addresses_are_unicast() {
// Not unspecified (0.0.0.0)
assert!(
!(parts == vec![0, 0, 0, 0]),
parts != vec![0, 0, 0, 0],
"Address '{}' is unspecified",
addr
);
@ -492,7 +492,7 @@ fn test_cross_distro_os_release_parsing() {
parsed.contains_key("NAME"),
"os-release must contain NAME field"
);
assert!(parsed["NAME"].ne(&""), "NAME should not be empty");
assert!(!parsed["NAME"].is_empty(), "NAME should not be empty");
}
#[test]