fix: estimate_gas interface with drip unit gas price. Resolves #36

Conflux-Chain · Aug 27, 2024 · a3da4a3 · a3da4a3
1 parent 9ecb791
commit a3da4a3
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/conflux_web3/client.py b/conflux_web3/client.py
@@ -196,6 +196,10 @@ def estimate_gas_and_collateral_munger(
     ) -> Tuple[TxParam, Optional[EpochNumberParam]]:
         if "from" not in transaction and self.default_account:
             transaction = assoc(transaction, "from", self.default_account)
+        if 'value' in transaction:
+            transaction['value'] = to_int_if_drip_units(transaction['value'])
+        if 'gasPrice' in transaction:
+            transaction['gasPrice'] = to_int_if_drip_units(transaction['gasPrice'])
 
         if block_identifier is None:
             params = (transaction, self._default_block)

diff --git a/tests/transaction/test_base_tx.py b/tests/transaction/test_base_tx.py
@@ -35,7 +35,8 @@ def test_basetx_estimate(w3: Web3, address: Base32Address):
     tx = {
         "from": address,
         "to": w3.account.create().address,
-        "value": 100
+        "value": 100,
+        "gasPrice": w3.cfx.gas_price
     }
 
     estimate = w3.cfx.estimate_gas_and_collateral(tx, w3.cfx.epoch_number-5)