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

Integration test for IBC connections #1249

Closed
Tracked by #1256
evan-forbes opened this issue Jan 16, 2023 · 0 comments · Fixed by #1474
Closed
Tracked by #1256

Integration test for IBC connections #1249

evan-forbes opened this issue Jan 16, 2023 · 0 comments · Fixed by #1474
Assignees
@cmwaters
Labels
testing items that are strictly related to adding or extending test coverage

Comments

@evan-forbes
Copy link
Member

evan-forbes commented Jan 16, 2023 •
edited by rootulp
Loading

Ideally, we would be able to test that IBC still works as intended with a few commonly connected chains, such as osmosis and the cosmos hub. Not only could we test functionality such as the token filter introduced in #1219, but we could also have more confidence that we don't accidentally break compatability. We might be able to do this with ibctest after each docker release.
@evan-forbes evan-forbes added the testing items that are strictly related to adding or extending test coverage label Jan 16, 2023
@evan-forbes evan-forbes added this to the Mainnet milestone Jan 16, 2023
This was referenced Jan 18, 2023
Closed
Closed
@cmwaters cmwaters self-assigned this Mar 13, 2023
@cmwaters cmwaters mentioned this issue Mar 13, 2023
Merged
cmwaters added a commit that referenced this issue Mar 15, 2023
@cmwaters
test: add integration test for the ibc tokenfilter (#1474)
953860c 
Closes: #1249

Adds a test suite under `testing/tokenfilter` that does the
following:
- ensures that native celestia tokens can be sent out from celestia and
can return
- ensures that other chains can't send tokens to celestia chains and
that their balance is unaffected.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cmwaters cmwaters

Labels
testing items that are strictly related to adding or extending test coverage
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

test: add integration test for the ibc tokenfilter celestiaorg/celestia-app
2 participants
@cmwaters @evan-forbes