search

keep-network / local-setup

Allow to easily set up tBTC and Keep locally for development and test purposes.
MIT License
8 stars 16 forks source link

E2E tests on local env are failing #128

Open michalinacienciala opened 2 years ago

michalinacienciala commented 2 years ago

Since Jan 28 local E2E tests are failing on the stage of deployment of coverage-pools contracts, in 05_deploy_asset_pool.ts: script:

deploying "UnderwriterToken" (tx: 0x10c888a01b95fdfd41de58dd052b92c4706ded74ceffc31f3ac432d0665a1ce6)...: deployed at 0xF3Bd1410e0E228c7A351E04b52243f9c783038a6 with 3682297 gas
An unexpected error occurred:

Error: ERROR processing /home/runner/work/local-setup/local-setup/coverage-pools/deploy/05_deploy_asset_pool.ts:
Error: cannot estimate gas; transaction may fail or may require manual gas limit (error={"name":"ProviderError","code":-32000,"_isProviderError":true}, method="estimateGas", transaction={"from":"0x3d373D872B7BA29d92Ed47CAA8605b4DD6Ec84eF","data":"0x60e0604052621[...]4c4598850","accessList":null}, code=UNPREDICTABLE_GAS_LIMIT, version=providers/5.4.5)
    at Logger.makeError (/home/runner/work/local-setup/local-setup/coverage-pools/node_modules/@ethersproject/logger/src.ts/index.ts:213:28)

More logs: https://github.com/keep-network/local-setup/runs/4974408180?check_suite_focus=true.

I suspect that the failure may be related to the updated dependencies to @keep-network/tbtc and @threshold-network/solidity-contracts in coverage-pools (that's the only change apart of https://github.com/keep-network/coverage-pools/pull/206/ that got merged to coverage-pools on Jan 27th).

michalinacienciala commented 2 years ago

After closer examination I think failure is related to changes made in https://github.com/keep-network/coverage-pools/pull/206. I created a temporary branch in local-setup which checks out coverage-pools at the commit 31d3af2039adc8a556e2ed125b28af4a7a16a371 and I've run e2e tast on that - they're failing with the UNPREDICTABLE_GAS_LIMIT error. That commit is the last commit of PR206. Changes from PR207 were not present at that point in the commit history.