chore: Update after review

Author: kulikthebird

packages/std/src/binary.rs

@@ -76,7 +76,7 @@ impl Binary {
     }
 
     pub fn to_hex(&self) -> String {
-        hex::encode(self.0.clone())
+        hex::encode(&self.0)
     }
 }
 

packages/std/src/testing/mock.rs

@@ -1467,7 +1467,10 @@ mod tests {
                 },
                 transaction: Some(TransactionInfo::new(
                     3,
-                    Binary::from_hex("E5469DACEC17CEF8A260FD37675ED87E7FB6A2B5AD95193C51308006C7E494B3").unwrap(),
+                    Binary::from_hex(
+                        "E5469DACEC17CEF8A260FD37675ED87E7FB6A2B5AD95193C51308006C7E494B3"
+                    )
+                    .unwrap(),
                 )),
                 contract: ContractInfo {
                     address: Addr::unchecked(MOCK_CONTRACT_ADDR)

packages/std/src/utils.rs

@@ -1,8 +1,8 @@
 /// Implements a hidden constructor for non-exhaustive structures that need
 /// to be built in libraries like cw-multi-test.
 macro_rules! impl_hidden_constructor {
-    ( $response:ty, $( $field: ident : $t: ty),* ) => {
-        impl $response {
+    ( $type:ty, $( $field: ident : $t: ty),* ) => {
+        impl $type {
             /// Constructor for testing frameworks such as cw-multi-test.
             /// This is required because the type is #[non_exhaustive].
             /// As a contract developer you should not need this constructor since

packages/vm/src/testing/mock.rs

@@ -282,7 +282,8 @@ pub fn mock_env() -> Env {
         },
         transaction: Some(TransactionInfo::new(
             3,
-            Binary::from_hex("E5469DACEC17CEF8A260FD37675ED87E7FB6A2B5AD95193C51308006C7E494B3").unwrap(),
+            Binary::from_hex("E5469DACEC17CEF8A260FD37675ED87E7FB6A2B5AD95193C51308006C7E494B3")
+                .unwrap(),
         )),
         contract: ContractInfo {
             address: Addr::unchecked(contract_addr),