ibc-rs
ibc-rs copied to clipboard
cosmos
Rust implementation of the Inter-Blockchain Communication (IBC) protocol.
## Improvement Summary Following PR #1167, to make it even simpler to create CosmWasm contracts for implemented light clients, we can write some macros to automate adding the `Context` to...
## Improvement Summary [`mock::ConsensusState` from `ibc_proto`](https://docs.rs/ibc-proto/latest/ibc_proto/ibc/mock/struct.ConsensusState.html) doesn't have any commitment root. ## Proposal To avoid any unexpected assumptions, we should also remove commitment_root from [`MockConsensuState` in `ibc-testkit`](https://docs.rs/ibc-testkit/latest/ibc_testkit/testapp/ibc/clients/mock/consensus_state/struct.MockConsensusState.html). The [`ConsensusState::root`](https://docs.rs/ibc-testkit/latest/ibc_testkit/testapp/ibc/clients/mock/consensus_state/struct.MockConsensusState.html#method.root) implementation...
Closes: #398 ## Description - [x] Packet timeout. - [ ] Packet timeout on channel close. The latest branch from `basecoin-rs` now has non-membership proofs for its IAVL tree. This...
## Summary There is a need to cover Tendermint proof verifications with proper tests after issue #396. Mock Chain passes the proof verifications and we are not able to catch...
## Feature Summary ```sh cargo check --package ibc --target wasm32-unknown-unknown --features=std ``` should work. But I get now ```sh error: the wasm*-unknown-unknown targets are not supported by default, you may...
Follow-up to #255. Our `Error` enums have too many variants. We should drastically condense them following the ideas described [here](https://github.com/cosmos/ibc-rs/pull/255#issuecomment-1325805412). ```[tasklist] ### Tasks - [ ] Remove all host error...
## Feature Summary We introduced `ToProto` trait in #995. It did streamline a lot of code, but we can streamline even more. ## Proposal 1. Extend the `ToProto` trait with...
## Summary Cover different successful/failure scenarios of `verify_conn_delay_passed()` with tests ## Proposal Upon changes in PR #498, earlier `verify_delay_passed()` tests were not applicable anymore, and appropriate tests for two conditions...
[This path](https://github.com/cosmos/ibc-rs/blob/73556be62f90979a62328a15fabb90f96105d4a3/crates/ibc/src/core/context/timeout.rs#L71-L89) for example (and analogous paths for acks and `RecvPacket`), test that event is emitted properly, and no state change is applied.
Our unit tests make extensive use of [default values](https://github.com/cosmos/ibc-rs/blob/0f3e03f84859d13cf842ec718eed3a2d8817a4d4/crates/ibc/src/core/ics04_channel/handler/timeout.rs#L398), including [in "util" functions](https://github.com/cosmos/ibc-rs/blob/0f3e03f84859d13cf842ec718eed3a2d8817a4d4/crates/ibc/src/core/ics04_channel/packet.rs#L313) implicitly used, which creates hard to understand relationships, and ultimately makes it difficult to understand why a...