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.

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

ibc-test-framework: Add support for basecoin-rs #2113

Open
5 tasks
thanethomson opened this issue Apr 21, 2022 · 0 comments
Open
5 tasks

ibc-test-framework: Add support for basecoin-rs #2113

thanethomson opened this issue Apr 21, 2022 · 0 comments
Labels
I: CI/CD Internal: related to CI/CD pipeline I: infrastructure Internal: related to Infrastructure (testing, deployment, etc) O: new-feature Objective: cause to add a new feature or support O: tests Objective: Test more aspect of the relayer

Comments

@thanethomson
Copy link
Contributor

Summary

Follows from #1920.

It seems as though the current ibc-test-framework only has a driver for gaiad. If we'd like to propose to add a driver for basecoin-rs so we can use a single integration testing framework for both ibc-rs and basecoin-rs.

Problem Definition

Our primary concern is ensuring that ibc-rs works with basecoin-rs and vice-versa (that neither breaks the other).

A secondary concern is that we'd like to minimize duplication of effort when it comes to integration testing, so having a single integration testing framework used by both ibc-rs and basecoin-rs in CI would be valuable.

Proposal

We'd like to add a chain driver for basecoin-rs in ibc-test-framework such that we can bridge gaiad and basecoin-rs with Hermes and run integration tests against these two bridged chains.

This issue covers the requisite work in the ibc-rs repository, and the corresponding issue in basecoin-rs is tracked here: informalsystems/basecoin-rs#47

Acceptance Criteria

When the ibc-rs repo contains an integration test in CI that uses ibc-test-framework to run at least one integration test for a setup consisting of both gaiad and basecoin-rs, with these two chains bridged by Hermes.


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate milestone (priority) applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@thanethomson thanethomson added O: new-feature Objective: cause to add a new feature or support O: tests Objective: Test more aspect of the relayer I: infrastructure Internal: related to Infrastructure (testing, deployment, etc) I: CI/CD Internal: related to CI/CD pipeline labels Apr 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I: CI/CD Internal: related to CI/CD pipeline I: infrastructure Internal: related to Infrastructure (testing, deployment, etc) O: new-feature Objective: cause to add a new feature or support O: tests Objective: Test more aspect of the relayer
Projects
None yet
Development

No branches or pull requests

1 participant