fix: set value before estimating gas (#7829)

foundry-rs · May 1, 2024 · cafc260 · cafc260
1 parent 5885dbc
commit cafc260
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/crates/forge/bin/cmd/create.rs b/crates/forge/bin/cmd/create.rs
@@ -242,17 +242,17 @@ impl CreateArgs {
             provider.get_transaction_count(deployer_address, BlockId::latest()).await
         }?);
 
+        // set tx value if specified
+        if let Some(value) = self.tx.value {
+            deployer.tx.set_value(value);
+        }
+
         deployer.tx.set_gas_limit(if let Some(gas_limit) = self.tx.gas_limit {
             Ok(gas_limit.to())
         } else {
             provider.estimate_gas(&deployer.tx, BlockId::latest()).await
         }?);
 
-        // set tx value if specified
-        if let Some(value) = self.tx.value {
-            deployer.tx.set_value(value);
-        }
-
         if is_legacy {
             let gas_price = if let Some(gas_price) = self.tx.gas_price {
                 gas_price.to()