feat: Add cosmos message poller #325
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@agouin Thanks for the quick review! But apologies, this was still in draft. I'm trying to add another poller function which will leverage this generic refactor. That's great all tests pass though. |
DavidNix
force-pushed
the
nix/feat/cosmos-poll-for-message
branch
from
f6d3fd9 to
80f03ae
Compare
agouin
reviewed
Oct 17, 2022
| @@ -0,0 +1,6 @@ | |||
| package cosmos_test | |||
|
|
|||
| const ( | |||
agouin
approved these changes
Oct 17, 2022
|
Addresses cosmos/relayer#853 |
jtieri
pushed a commit
that referenced
this pull request
Jan 6, 2023
jtieri
pushed a commit
that referenced
this pull request
Jan 6, 2023
jtieri
added a commit
that referenced
this pull request
Jan 9, 2023
* fix: Unable to build ibctest binary (#326) * feat: Add cosmos message poller (#325) * fix: change import paths from v6 to v3 * Make configutil public (#336) * Make configutil public * Rename test package to chainutil. Make util packages subpackages of util package * Consolidate into single testutil package * fix: formatting issue in doc * fix: change import path from test to testutil * fix: go mod tidy * packet-forward-middleware packet memo, retry on timeout, and atomic forwards (#306) * Make examples its own module * Make cosmos specific module and run in CI in separate task * Update e2e test for memo refactor * Update test to chain-level params * Use gaia with pfm with configurable timeout and retries * Update SendIBCTransfer uses * fix interchain test * Add GetClients method to Relayer and helper for getting transfer channel between chains * Update packet forward test for multi-hop, add multi-hop refund test * Update tests for atomic forwards * Wait for a block after ack before checking balances * reduce wait to 1 block * Add multi-hop flow with refund through chain with native denom. Add assertions for escrow accounts * Remove stale comment * handle feedback * Add TransferOptions * fix: remove extra argument to NewMsgTransfer call * build: go mod tidy * chore: update gaia image version for packet forward tests * fix: fallback to old version of the interchain accounts demo binary Co-authored-by: David Nix <DavidNix@users.noreply.github.com> Co-authored-by: Andrew Gouin <andrew@gouin.io>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds a new poll function for cosmos chains. The ability to poll for any arbitrary cosmos message.
I will use this in the relayer codebase to help with a test regarding keeping light clients up to date.