Safe-react: [L2] Create testing environments for L2 networks

[katspaugh]

QA team with the help from @tschubotz will create safes and assets on each of the first three L2 networks:

• Polygon
• BSC
• Arbitrum

These safes will be used for the initial testing of these deployments.

Test Data for each network

1. a safe with original network Currency. Take into account that original currency will be sent from the safe to the owners to test txs execution.

2. owners that should be added:
   Liliya's = 0xF2CeA96575d6b10f51d9aF3b10e3e4E5738aa6bd
   Franco's = 0x8aEf2f5c3F17261F6F1C4dA058D022BE92776af8 or 0x0D65139Da4B36a8A39BF1b63e950038D42231b2e

3. ERc20 tokens

4. ERC721 token (NFTs)

5. Contracts that may be used for contract interaction and Transaction Builder

[tschubotz]

https://chainlist.org/ <- Site to add chains to Metamask

[liliya-soroka]

@francovenica , could you check if we need something else for testing?

[tschubotz]

I'll start with this today, @francovenica @liliya-soroka. I'll let you know once I have some thing. In case you have specific needs, please let me know what kind of test data/txs you.

Contracts that may be used for contract interaction and Transaction Builder

Could you elaborate on that?

[tschubotz]

I deployed some test contracts to use

BSC:

• MintableERC20
  
  
  
  • 0x060B54F2001eb23e8688A7E44d159da8Acf56f40
  
  
• MintableERC721
  
  
  
  • 0x55EC23Ae91F5631Ac66C3f11D32a0F67EF9d3670
  
  
• TestDecodingV1
  
  
  
  • 0x5A5922E9ef11157498Fa5325bc39E7a1Ba8c402D
  
  
  • 0x9452fa9F1691d52299F077436559746D2e6Fa1ad
  
  

Polygon

• MintableERC20
  
  
  
  • 0x060B54F2001eb23e8688A7E44d159da8Acf56f40
  
  
• MintableERC721
  
  
  
  • 0x80a658D43277e0cF63D4E6A7D69fCc19fd34E073
  
  
  • 0x55EC23Ae91F5631Ac66C3f11D32a0F67EF9d3670
  
  
• TestDecodingV1
  
  
  
  • 0x5A5922E9ef11157498Fa5325bc39E7a1Ba8c402D
  
  
  • 0x9452fa9F1691d52299F077436559746D2e6Fa1ad
  
  

Let me know if more is needed.

I commited the code I used to deploy them at: https://gitlab.gnosisdev.com/wallets/gnosis-safe-qa/-/tree/master/test_contracts
So they can be deployed to Arbitrum and other networks easily in the future.

I'll take a look at writing a script to automate filling a Safe with example txs and assets next.

[liliya-soroka]

@tschubotz ,will we have known addresses on these networks?

[tschubotz]

My ERC20 token contract was non-standard, just updated the addresses ..

@tschubotz ,will we have known addresses on these networks?

The feature will exist, yes. I don't think we have set any up yet though. I'll add this to my list!

[tschubotz]

With Richard's safe-tasks tool it's quite easy to write a script that triggers some Safe txs in order to auto-populate test data.

Once you provide me with a list of assets and/or transactions you would like to see @liliya-soroka @francovenica , I'm ready to go :)
Feel free to also just link me with a Safe on Rinkeby/mainnet/xdai that you normally use, then I can just re-create its txs.

[tschubotz]

The feature will exist, yes. I don't think we have set any up yet though. I'll add this to my list!

@liliya-soroka I just added the Gnosis Safe 1.3.0 contracts as "known contracts" in the admin for polygon, BSC, Arbitrum incl. logo.

List of addresses here: https://docs.google.com/spreadsheets/d/1JVvwTfp3WNQ1CXMx8xVQ2n7Aj-u0KDiXxGHNW5OjHb4/edit#gid=459807097

[katspaugh]

I've created a safe on Arbitrum, with Liliya and me as owners:

It's not usable yet because the backend doesn't fully work with Arbitrum yet.

@francovenica please let us know your wallet address.

[dasanra]

@tschubotz ERC721 mintable for BSC address is wrong. It's the same address you shared for ERC20 :smile:

I tested it and it mints the ERC20

[francovenica]

@tschubotz Here my mainnet accounts. I have 2, feel free to add just one if you want:
0x8aEf2f5c3F17261F6F1C4dA058D022BE92776af8
0x0D65139Da4B36a8A39BF1b63e950038D42231b2e

I'll update this ticket description

[francovenica]

@francovenica , could you check if we need something else for testing?

Besides what the ERC20/721 can mint, we will need actual mainnet tokens to pay for the tx.

Also are there apps that could work or not? I assume tx builder and wallet connect should work same as always.

[dasanra]

@francovenica TX-Builder and WalletConnect will need an update in order to work on BSC. Depending on time, and state of the SDK this could be done for the release or later. Also there are 2 apps on the list that have a BSC version but they seem they are not detecting BSC network correctly so we will have to remove them from the default list first and add them later.

[tschubotz]

@tschubotz ERC721 mintable for BSC address is wrong. It's the same address you shared for ERC20 😄

I tested it and it mints the ERC20

My bad, fixed now! :) https://bscscan.com/address/0x55EC23Ae91F5631Ac66C3f11D32a0F67EF9d3670#code

[tschubotz]

I just sent some BNB and Matic for testing to
0xF2CeA96575d6b10f51d9aF3b10e3e4E5738aa6bd (liliya)
0x8aEf2f5c3F17261F6F1C4dA058D022BE92776af8 (Franco)

[katspaugh]

Here's a Polygon safe with Liliya, Franco and me as owners:

[liliya-soroka]

The same test safe on a few networks

https://rinkeby.etherscan.io/address/0x208F8672491e86021c2e7a5cc55238a85Ff43482
https://polygonscan.com/address/0x208F8672491e86021c2e7a5cc55238a85Ff43482