Open davidmurdoch opened 2 years ago
@anishnaik I've found the issue. callGasLimit
is working in eth_estimateGas
, but allowUnlimitedContractSize
is not.
estimateGas
makes a copy of the VM before running the transaction in order to estimate it. The allowUnlimitedContractSize
property is not copied, as you can see here:
There is currently no work around but I hope to get a fix in for this very soon.
Hi @davidmurdoch this makes sense! Appreciate your help.
@davidmurdoch any updates on this?
Can you try ganache@alpha
and let me know if that fixes it for you??
Hey @davidmurdoch apologies for the delayed response. ganache@alpha
worked with the above codebase :)
Is there a timeline for an official release?
There is no timeline; we are working now to improve stability.
Reporduction:
Clone https://github.com/anishnaik/ganache-bug and checkout the
bug
branch.The testing script is called
deploy.js
.Ganache@7.3.2 Terminal 1
Terminal 2
This should fail with the following error:
You can try to set a
gasLimit
for thedeploy()
call and if it is large enough, the test will pass.ganache-cli@6.12.2 Terminal 1
Terminal 2
This test should pass
This could also be a dependency issue in the hardhat environment?
Environment MacBook Pro w/ Apple M1 Max Hardhat v2.9.9 Ganache@7.3.2 ganache-cli@6.12.2
Originally posted by @anishnaik in https://github.com/trufflesuite/ganache/issues/1645#issuecomment-1171642018