wormchain: integration tests for wormchain-ibc-receiver #4155
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
kakucodes
requested review from
nik-suri,
johnsaigle and
mdulin2
as code owners
kakucodes
changed the title
joelsmith-2019
previously approved these changes
Nov 21, 2024
joelsmith-2019
previously approved these changes
Dec 10, 2024
nik-suri
previously approved these changes
Dec 10, 2024
kakucodes
force-pushed
the
feature/integration-test-ibc-receiver
branch
from
8d470fe to
a1cb369
Compare
kakucodes
requested review from
djb15,
SEJeff,
pleasew8t,
bruce-riley,
panoel,
claudijd,
kev1n-peters,
a5-pickle,
kcsongor,
nonergodic and
gator-boi
as code owners
kakucodes
dismissed stale reviews from nik-suri and joelsmith-2019
via
df9db20
kakucodes
force-pushed
the
feature/integration-test-ibc-receiver
branch
from
a1cb369 to
df9db20
Compare
nik-suri
approved these changes
Dec 10, 2024
joelsmith-2019
approved these changes
Dec 11, 2024
nik-suri
force-pushed
the
feature/integration-test-ibc-receiver
branch
from
df9db20 to
fad9810
Compare
evan-gray
approved these changes
Dec 12, 2024
kakucodes
pushed a commit
that referenced
this pull request
Dec 18, 2024
cosmwasm(cw_wormhole): integration tests- initial implementation chore: adding additional vaa parsing tests cosmwasm: added cw_wormhole integration tests and some code cleanup cosmwasm: reverted contract changes Update 0012_ibc_generic_messaging.md with permissionless channel creation information (#4167) The `wormhole-ibc-receiver` contract contains an allowlist of Wormhole chain ID to channel mappings (other side of Cosmos chain). The allowlist is not used when receiving a packet because we don't want to handle the case of a token transfer failure on the other side. Instead, we just emit the message and ignore it in the IBC watcher. I am just updating the documentation to reflect this. node: governor token list update tilt: aptos multi-platform support tilt: sui multi-platform support tilt: node multi-platform support tilt: solana-test-validator multi-platform support tilt: smaller sui image Add coin gecko pro API usage for the governor (#4025) * Add coin gecko pro API usage for the governor * Add in missing parameter for node test * Fix missing parameter in publicrpcserver_test.go * Add in NIT fixes * Change CLI description * Reorder error message so that the important part is not truncated in the logs * Remove network test from unit test. Plan on creating a Github action cron action for this instead * Remove unnecessary '&' from URL path * Add in new parameters for gov from rebase * Fix regression on query creation * Add coin gecko pro API usage for the governor * Add in NIT fixes * Remove network test from unit test. Plan on creating a Github action cron action for this instead --------- Co-authored-by: Maxwell Dulin <[email protected]> Co-authored-by: Maxwell Dulin <[email protected]> node: add amazon kms and benchmark signers (#4168) * add kms and benchmark signers * include context during guardiansigner calls * add error counters and address review comment * address pr comments * address pr comments --------- Co-authored-by: pleasew8t <[email protected]> wormchain: add ibc_receiver_test and ict workflow (#4155) Update CODEOWNERS (#4151) sdk: worldchain mainnet SR entry. (#4188) cosmwasm: refactored the to guardianAddress logic wormchain: lower ict setup params to improve stability
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.
Add Integration Tests for IBC Contracts and CI Workflow
Overview
This PR implements comprehensive integration tests for
wormchain-ibc-receiverandwormhole-ibccontracts using Interchain test framework, providing a realistic testing environment for IBC functionality. Additionally, it establishes automated CI testing through a new GitHub workflow.Key Changes
Technical Details
Testing
Tests cover:
Dependencies
Impact
These tests significantly improve our confidence in the IBC contracts by: