add revert handling

[skosito]

covering flow:

• GatewayEVM.execute called with address of Hello contract from example-contracts
• worker picks up Called event and forwards call to Hello contract's onCrossChainCall
• onCrossChainCall reverts
• worker calls GatewayEVM.executeRevert using RevertOptions from Called event

is this useful? also it seems localnet in general needs some modifications, for example currently it was broken because zrc20 contract was missing - in this case, onCrossChainCall expects zrc20 to be passed as arg, which zrc20 we would send if localnet doesn't deploy some?

[fadeev]

@skosito I think deployProtocolContracts should deploy an ERC-20 (let's call it USDC) and an associated ZRC-20 USDC. It should also deploy ZRC-20 ETH. If EVM test contract sends native gas token, then ZRC-20 ETH is passed to the onCrossChainCall, if ERC-20 USDC is sent, then ZRC-20 USDC.

This way we mirror the behavior of the real environment allowing a developer to test both native gas and ERC-20 tokens.

[fadeev]

For simplicity, we can just start with ETH & ZRC-20 ETH support. And possibly add ERC-20 later.

[fadeev]

Suggested change

	"TOKEN",
	"ZRC-20 ETH",

[fadeev]

@skosito thanks for this PR! Now a example contract works when called from EVM! 🙌

[skosito]

@skosito I think deployProtocolContracts should deploy an ERC-20 (let's call it USDC) and an associated ZRC-20 USDC. It should also deploy ZRC-20 ETH. If EVM test contract sends native gas token, then ZRC-20 ETH is passed to the onCrossChainCall, if ERC-20 USDC is sent, then ZRC-20 USDC.

This way we mirror the behavior of the real environment allowing a developer to test both native gas and ERC-20 tokens.

sure i will try to update

[skosito]

adding just zrc-eth for now so we handle current flow we have, i think simpler approach is that we can extend with more zrc20s and other functionalities as we add examples, since atm we are not supporting all events anyways, and keeping localnet simple as possible would be nice - we can add issues as we go

i also added same revert handling on zevm side

[fadeev]

Reverts are working now!

[fadeev]

Closed #10 & #11