Check that the estimation of the node is bigger than 0 before returning

This commit is contained in:
Agustin Pane 2021-02-02 12:39:54 -03:00
parent 2f6113d117
commit c1b48f9508
1 changed files with 5 additions and 3 deletions

View File

@ -182,15 +182,17 @@ const calculateMinimumGasForTransaction = async (
const amountOfGasToTryTx = txGasEstimation + dataGasEstimation + additionalGas const amountOfGasToTryTx = txGasEstimation + dataGasEstimation + additionalGas
console.info(`Estimating transaction creation with gas amount: ${amountOfGasToTryTx}`) console.info(`Estimating transaction creation with gas amount: ${amountOfGasToTryTx}`)
try { try {
await getGasEstimationTxResponse({ const estimation = await getGasEstimationTxResponse({
to: safeAddress, to: safeAddress,
from: safeAddress, from: safeAddress,
data: estimateData, data: estimateData,
gasPrice: 0, gasPrice: 0,
gas: amountOfGasToTryTx, gas: amountOfGasToTryTx,
}) })
if (estimation > 0) {
console.info(`Gas estimation successfully finished with gas amount: ${amountOfGasToTryTx}`) console.info(`Gas estimation successfully finished with gas amount: ${amountOfGasToTryTx}`)
return amountOfGasToTryTx return amountOfGasToTryTx
}
} catch (error) { } catch (error) {
console.log(`Error trying to estimate gas with amount: ${amountOfGasToTryTx}`) console.log(`Error trying to estimate gas with amount: ${amountOfGasToTryTx}`)
} }