From 6ac93e868858e4322172c25f59ac072d6156ffb6 Mon Sep 17 00:00:00 2001 From: Francesco_Cina Date: Tue, 21 Mar 2023 18:05:26 +0100 Subject: [PATCH] Cargo clippy --- src/env_reader/mod.rs | 2 +- src/lib.rs | 18 +++++++++--------- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/src/env_reader/mod.rs b/src/env_reader/mod.rs index 2a35403..140448e 100644 --- a/src/env_reader/mod.rs +++ b/src/env_reader/mod.rs @@ -41,7 +41,7 @@ mod test { fn should_return_the_default_value_if_env_variable_not_present() { let mut random: i64 = rand::random(); let env_key = random.to_string(); - random = random + 10; + random += 10; assert!(!env_var_exists(&env_key)); assert_eq!(random.to_string(), env_var(&env_key, random.to_string())); diff --git a/src/lib.rs b/src/lib.rs index f3b1b3b..d4730df 100644 --- a/src/lib.rs +++ b/src/lib.rs @@ -138,7 +138,7 @@ pub fn parse_command>( ) -> Result, shell_words::ParseError> { let s = raw_cmd.into(); let command_string = s.trim().to_string(); - if command_string.len() == 0 { + if command_string.is_empty() { return Ok(None); } let argv = shell_words::split(&command_string)?; @@ -281,13 +281,13 @@ mod test { tcp_timeout: &str, command: &str, ) { - env::set_var("WAIT_BEFORE_HOSTS", before.to_string()); - env::set_var("WAIT_AFTER_HOSTS", after.to_string()); - env::set_var("WAIT_HOSTS_TIMEOUT", timeout.to_string()); - env::set_var("WAIT_HOST_CONNECT_TIMEOUT", tcp_timeout.to_string()); - env::set_var("WAIT_HOSTS", hosts.to_string()); - env::set_var("WAIT_SLEEP_INTERVAL", sleep.to_string()); - env::set_var("WAIT_COMMAND", command.to_string()); + env::set_var("WAIT_BEFORE_HOSTS", before); + env::set_var("WAIT_AFTER_HOSTS", after); + env::set_var("WAIT_HOSTS_TIMEOUT", timeout); + env::set_var("WAIT_HOST_CONNECT_TIMEOUT", tcp_timeout); + env::set_var("WAIT_HOSTS", hosts); + env::set_var("WAIT_SLEEP_INTERVAL", sleep); + env::set_var("WAIT_COMMAND", command); } #[test] @@ -297,7 +297,7 @@ mod test { #[test] fn parse_command_returns_none_when_command_is_empty() { - for c in vec!["", " \t\n\r\n"] { + for c in &["", " \t\n\r\n"] { let p = parse_command(c.to_string()).unwrap(); assert!(p.is_none()); }