Skip to content

Commit

Permalink
update comment
Browse files Browse the repository at this point in the history
  • Loading branch information
rnbguy committed Apr 30, 2024
1 parent cdd5dcf commit d9f5c9e
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 4 deletions.
7 changes: 4 additions & 3 deletions docs/architecture/adr-009-revamp-testkit.md
Original file line number Diff line number Diff line change
Expand Up @@ -35,8 +35,8 @@ To achieve this, we have broken down the proposal into sub-proposals:

The current framework uses `HashMap` and `HashSet` to store data. This works for
many test scenarios, but it fails to test proof-sensitive scenarios. Because of
this, we don't have any connection, channel handshake, or packet relay tests that
cover the Tendermint light client.
this, we don't have any connection, channel handshake, or packet relay tests
that cover the Tendermint light client.

We generalize
[`MockContext`](https://github.com/cosmos/ibc-rs/blob/v0.51.0/ibc-testkit/src/testapp/ibc/core/types.rs#L103)
Expand Down Expand Up @@ -123,7 +123,8 @@ pub trait TestBlock: Clone + Debug {
fn into_header(self) -> Self::Header;
}

/// TestHeader is a trait that defines the interface for a header produced by a host blockchain.
/// TestHeader is a trait that defines the interface for a header
/// submitted by relayer from the host blockchain.
pub trait TestHeader: Clone + Debug + Into<Any> {
/// The type of consensus state can be extracted from the header.
type ConsensusState: ConsensusState + Into<AnyConsensusState> + From<Self> + Clone + Debug;
Expand Down
3 changes: 2 additions & 1 deletion ibc-testkit/src/hosts/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -138,7 +138,8 @@ pub trait TestBlock: Clone + Debug {
}
}

/// TestHeader is a trait that defines the interface for a header corresponding to a host blockchain.
/// TestHeader is a trait that defines the interface for a header
/// submitted by relayer from the host blockchain.
pub trait TestHeader: Clone + Debug + Into<Any> {
/// The type of consensus state can be extracted from the header.
type ConsensusState: ConsensusState + Into<AnyConsensusState> + From<Self> + Clone + Debug;
Expand Down

0 comments on commit d9f5c9e

Please sign in to comment.