1
Fork 0
mirror of https://github.com/RGBCube/uutils-coreutils synced 2025-07-28 11:37:44 +00:00

Merge pull request #4532 from sylvestre/clippy-pe

Fix some simple clippy warnings
This commit is contained in:
Sylvestre Ledru 2023-03-18 17:31:36 +01:00 committed by GitHub
commit 60bf8e1f88
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 9 additions and 8 deletions

View file

@ -10,7 +10,7 @@ fn test_output_is_random_permutation() {
let input_seq = vec![1, 2, 3, 4, 5, 6, 7, 8, 9, 10]; let input_seq = vec![1, 2, 3, 4, 5, 6, 7, 8, 9, 10];
let input = input_seq let input = input_seq
.iter() .iter()
.map(|x| x.to_string()) .map(ToString::to_string)
.collect::<Vec<String>>() .collect::<Vec<String>>()
.join("\n"); .join("\n");
@ -52,7 +52,7 @@ fn test_echo() {
.args( .args(
&input_seq &input_seq
.iter() .iter()
.map(|x| x.to_string()) .map(ToString::to_string)
.collect::<Vec<String>>(), .collect::<Vec<String>>(),
) )
.succeeds(); .succeeds();
@ -74,7 +74,7 @@ fn test_head_count() {
let input_seq = vec![1, 2, 3, 4, 5, 6, 7, 8, 9, 10]; let input_seq = vec![1, 2, 3, 4, 5, 6, 7, 8, 9, 10];
let input = input_seq let input = input_seq
.iter() .iter()
.map(|x| x.to_string()) .map(ToString::to_string)
.collect::<Vec<String>>() .collect::<Vec<String>>()
.join("\n"); .join("\n");
@ -105,7 +105,7 @@ fn test_repeat() {
let input_seq = vec![1, 2, 3, 4, 5, 6, 7, 8, 9, 10]; let input_seq = vec![1, 2, 3, 4, 5, 6, 7, 8, 9, 10];
let input = input_seq let input = input_seq
.iter() .iter()
.map(|x| x.to_string()) .map(ToString::to_string)
.collect::<Vec<String>>() .collect::<Vec<String>>()
.join("\n"); .join("\n");

View file

@ -2812,7 +2812,7 @@ mod tests {
#[should_panic] #[should_panic]
fn test_cmd_result_stdout_check_when_false_then_panics() { fn test_cmd_result_stdout_check_when_false_then_panics() {
let result = TestScenario::new("echo").ucmd().arg("Hello world").run(); let result = TestScenario::new("echo").ucmd().arg("Hello world").run();
result.stdout_check(|s| s.is_empty()); result.stdout_check(<[u8]>::is_empty);
} }
#[cfg(feature = "echo")] #[cfg(feature = "echo")]
@ -3055,9 +3055,10 @@ mod tests {
// check `child.is_alive()` and `child.delay()` is working // check `child.is_alive()` and `child.delay()` is working
let mut trials = 10; let mut trials = 10;
while child.is_alive() { while child.is_alive() {
if trials <= 0 { assert!(
panic!("Assertion failed: child process is still alive.") trials > 0,
} "Assertion failed: child process is still alive."
);
child.delay(500); child.delay(500);
trials -= 1; trials -= 1;