Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Mock context should accommodate TM light blocks #100

Closed
5 tasks done
adizere opened this issue Nov 9, 2020 · 0 comments
Closed
5 tasks done

Mock context should accommodate TM light blocks #100

adizere opened this issue Nov 9, 2020 · 0 comments
Assignees
@adizere

Comments

@adizere
Copy link
Contributor

adizere commented Nov 9, 2020

Crate

ibc

Summary

Currently, MockContext only permits a history of MockHeaders:

https://github.com/informalsystems/ibc-rs/blob/2858779c185d9c9578e7f8203cd5c455e75dcbc3/modules/src/mock_context.rs#L36

Towards being able to do integration tests against a Tendermint mock chain, we'd like to expand the MockContext struct so that it can accommodate different types of blocks. For instance, using tendermint-testgen, we could generate Tendermint LightBlocks and thus emulate a TM chain history, which could serve as the underlying history representation in MockContext.

Problem Definition

Why do we need this feature? For informalsystems/hermes#158 and ultimately informalsystems/hermes#381

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate milestone (priority) applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@adizere adizere self-assigned this Nov 9, 2020
This was referenced Nov 9, 2020
Closed
Merged
@hu55a1n1 hu55a1n1 transferred this issue from informalsystems/hermes Sep 29, 2022
livelybug pushed a commit to octopus-network/ibc-rs that referenced this issue Oct 14, 2022
@adizere
ADR001 (cosmos#100)
65ee309 
* First draft of ADR 001

* Adding more content

* Fix description for each crate.

* Fix anchor link
shuoer86 pushed a commit to shuoer86/ibc-rs that referenced this issue Nov 4, 2023
@tac0turtle
Merge pull request cosmos#100 from cosmos/dependabot/npm_and_yarn/js/…
4a951ea 
…protobufjs-6.10.3

Bump protobufjs from 6.8.9 to 6.10.3 in /js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@adizere adizere

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@adizere