Görli contracts deployment

[nkuba]

Görli became a recommended test network after Ropsten's deprecation notice (link).

In this PR we update deployment scripts and configuration to support Görli.

The contracts were already deployed to Görli:
@keep-network/tbtc-v2@0.1.1-goerli.0

[michalinacienciala]

Generally, the changes look good to me. But there is some problem with the deployment on goerli - I'm getting:

initializing Bridge as ECDSA wallet owner
An unexpected error occurred:
Error: ERROR processing /home/runner/work/tbtc-v2/tbtc-v2/solidity/deploy/16_initialize_wallet_owner.ts:
TypeError: Cannot read property 'length' of undefined

This doesn't seem to be directly caused by the changes in this PR, it looks more like a problem with threshold-network/keep-core#3061. @nkuba, do you think that merging of threshold-network/keep-core#3096 may help?

[nkuba]

Generally, the changes look good to me. But there is some problem with the deployment on goerli - I'm getting:

initializing Bridge as ECDSA wallet owner
An unexpected error occurred:
Error: ERROR processing /home/runner/work/tbtc-v2/tbtc-v2/solidity/deploy/16_initialize_wallet_owner.ts:
TypeError: Cannot read property 'length' of undefined

This doesn't seem to be directly caused by the changes in this PR, it looks more like a problem with keep-network/keep-core#3061. @nkuba, do you think that merging of keep-network/keep-core#3096 may help?

Interesting... We need to fix it in a separate PR to keep-core.

Actually bfd629c shuould fix it.

[michalinacienciala]

Actually bfd629c should fix it.

Yep, I no longer see that error. There's a different error now though, see my comment keep-network/coverage-pools#215 (comment). Usually such errors were caused by some problems with the config - when deployer account from dependencies did not match the one used currently for deploy. But it doesn't seem like it this time... Any ideas?

[EDIT]: OK. I chatted with Kuba offline and he concluded that the error is related to the fact that for given ecdsa package that we use as dependency, we can call the initializeWalletOwner function only once (and it has already been called when @keep-network/tbtc@1.1.2-goerli.0 package was being prepared). This is because during deployment of tbtc-v2 contracts we're connecting Bridge with ecdsa's WalletRegistry and this connection can be made only once.
So in order to fully test the deployment of tbtc-v2 contracts on goerli we now need to have a new ecdsa package for goerli. We'll be able to publish it once threshold-network/keep-core#3081 gets merged.

[michalinacienciala]

Deployment on goerli verified in job https://github.com/keep-network/tbtc-v2/runs/7744385930?check_suite_focus=true (workflow uses code combined from #387 and #382). Works as expected