[EPIC] Include a relayer implementation in e2e tests

cosmos / solidity-ibc-eureka

This is a work-in-progress solidity implementation of IBC Eureka.

MIT License

31 stars 3 forks source link

[EPIC] Include a relayer implementation in e2e tests #81

Closed srdtrk closed 1 day ago

srdtrk commented 3 weeks ago

Right now, all e2e tests pass using manual relaying. It is worth looking into a path for relayer integration. Since union's voyager relayer already has similar functionalities, it might be worth developing plugins for it.

[ ] Use voyager to update client (on eth)
[ ] Use voyager to relay timeouts (on eth)
[ ] Use voyager to relay recv packet (on eth)
[ ] Use voyager to relay ack packet (on eth)

srdtrk commented 2 weeks ago

Work on this is taking place in voyager-eureka-modules at the moment. If it reaches a satisfactory status, then it can be merged into this repo.

While trying to implement the relayer, I noticed a missing feature, that is multicalls. Which is the ability to include multiple contract calls in a single transaction. This was added in #87 to this repo. Doing this also opened up another major gas optimization using transient store, which is being merged in #91.

srdtrk commented 1 day ago

This issue is closed in favor of https://github.com/cosmos/solidity-ibc-eureka/issues/121