RPC-based Gas Estimation for Moonbeam-based Networks

[albertov19]

Motivation

Moonbeam introduced MBIP-5 in which gas estimation deviated from Ethereum. The reasoning is that now storage changes are reflected now on gas estimation. This is to prevent spamming the network with junk data. This is a problem on a lot of cheap-to-transact networks

Solution

Solution was simple, just add Moonbeam-based networks to the has_different_gas_calc function (like with Arbitrum) here

If this function returns true then gas estimation is done using the actual provider URL and eth_estimateGas.

[mattsse]

the outlined solution makes sense, though I think you forgot to include it here :)

[mattsse]

please restore other formatting issues like trailing ;

[mattsse]

ah nvm, I overlooked it hehe

[Evalir]

looks good, pending mattsse's nit

[albertov19]

@mattsse @Evalir thanks! I think it should be good. Everything was added by the cargo fmt command, I just reverted the ;

[albertov19]

Ah apologies I did not understand you wanted me to revert the | change as well - This should be ready to go right?

I tested this implementation in a local Foundry Forge instance and it worked btw.

[mattsse]

lgtm