save

Author: taikoonwang

crates/cli/src/opts/dependency.rs

@@ -160,7 +160,7 @@ mod tests {
 
     #[test]
     fn parses_dependencies() {
-        for (input, expected_path, expected_tag, expected_alias) in [
+        for (input, expected_path, expected_tag, expected_alias) in &[
             ("gakonst/lootloose", "https://github.com/gakonst/lootloose", None, None),
             ("github.com/gakonst/lootloose", "https://github.com/gakonst/lootloose", None, None),
             (
@@ -242,8 +242,7 @@ mod tests {
                 Some("v1"),
                 Some("loot"),
             ),
-        ]
-        .iter() {
+        ] {
             let dep = Dependency::from_str(input).unwrap();
             assert_eq!(dep.url, Some(expected_path.to_string()));
             assert_eq!(dep.tag, expected_tag.map(ToString::to_string));
@@ -269,15 +268,14 @@ mod tests {
 
     #[test]
     fn parses_contract_info() {
-        for (input, expected_path, expected_name) in [
+        for (input, expected_path, expected_name) in &[
             (
                 "src/contracts/Contracts.sol:Contract",
                 Some("src/contracts/Contracts.sol"),
                 "Contract",
             ),
             ("Contract", None, "Contract"),
-        ]
-        .iter() {
+        ] {
             let contract = ContractInfo::from_str(input).unwrap();
             assert_eq!(contract.path, expected_path.map(ToString::to_string));
             assert_eq!(contract.name, expected_name.to_string());
@@ -286,7 +284,7 @@ mod tests {
 
     #[test]
     fn contract_info_should_reject_without_name() {
-        for input in ["src/contracts/", "src/contracts/Contracts.sol"].iter() {
+        for input in &["src/contracts/", "src/contracts/Contracts.sol"] {
             let contract = ContractInfo::from_str(input);
             assert!(contract.is_err())
         }

crates/forge/src/cmd/clone.rs

@@ -645,7 +645,7 @@ mod tests {
         stripped_creation_code: &str,
     ) {
         compiled.compiled_contracts_by_compiler_version().iter().for_each(|(_, contracts)| {
-            for (name, contract) in contracts.iter() {
+            for (name, contract) in contracts {
                 if name == contract_name {
                     let compiled_creation_code =
                         contract.bin_ref().expect("creation code not found");