Remove wasm grandpa tests and introduce wasm tendermint tests

cosmos / ibc-go

Inter-Blockchain Communication Protocol (IBC) implementation in Golang.

https://ibc.cosmos.network/

MIT License

553 stars 597 forks source link

Remove wasm grandpa tests and introduce wasm tendermint tests #6835

Open srdtrk opened 4 months ago

srdtrk commented 4 months ago

Summary

Grandpa wasm tests were disabled due them failing after changes in the 08-wasm module. We found it difficult to keep the grandpa contract and related docker images up to date. Instead, we should introduce tests for wasm 07-tendermint from ibc-rs.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged/assigned

srdtrk commented 4 months ago

I've opened an issue and PR to update contract APIs of ibc-rs. https://github.com/cosmos/ibc-rs/issues/1277

womensrights commented 4 days ago

@gjermundgaraba I think we should remove these and just have the ethereum tests once the wasm ethereum light client is in the repo

gjermundgaraba commented 4 days ago

Yes, that makes a lot of sense 🙏