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

@@ -533,7 +533,10 @@ impl Envs {
             },
             transaction: Some(TransactionInfo::new(
                 3,
-                Binary::from_hex("0102030405060708").unwrap(),
+                Binary::from_hex(
+                    "E5469DACEC17CEF8A260FD37675ED87E7FB6A2B5AD95193C51308006C7E494B3",
+                )
+                .unwrap(),
             )),
             contract: ContractInfo {
                 address: self.contract_address.clone(),
@@ -1467,7 +1470,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/instance.rs

@@ -921,7 +921,7 @@ mod tests {
 
         let report2 = instance.create_gas_report();
         assert_eq!(report2.used_externally, 251);
-        assert_eq!(report2.used_internally, 17953755);
+        assert_eq!(report2.used_internally, 18034325);
         assert_eq!(report2.limit, LIMIT);
         assert_eq!(
             report2.remaining,
@@ -1088,7 +1088,7 @@ mod tests {
             .unwrap();
 
         let init_used = orig_gas - instance.get_gas_left();
-        assert_eq!(init_used, 17954006);
+        assert_eq!(init_used, 18034576);
     }
 
     #[test]
@@ -1113,7 +1113,7 @@ mod tests {
             .unwrap();
 
         let execute_used = gas_before_execute - instance.get_gas_left();
-        assert_eq!(execute_used, 24613486);
+        assert_eq!(execute_used, 24624251);
     }
 
     #[test]
@@ -1156,6 +1156,6 @@ mod tests {
         );
 
         let query_used = gas_before_query - instance.get_gas_left();
-        assert_eq!(query_used, 11119756);
+        assert_eq!(query_used, 11105221);
     }
 }

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),