-
Notifications
You must be signed in to change notification settings - Fork 326
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
Test clear_packet
behavior
#1881
Conversation
clear_packet
behavior
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sweet 🥇
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Soares, this is fantastic work!
(I only had a chance to review it more closely now.)
)?; | ||
|
||
// Wallet on chain B should only receive the second IBC transfer | ||
chains.node_b.chain_driver().assert_eventual_wallet_amount( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wow, this is pretty neat being able to capture this advanced case in such a simple way.
* Cleaned up clear_packets logic from RelayPath. * changelog * Test on ordered channel relaying behavior * Test now shows correct behavior * Change test names to target clear_packet test * Add tools/ directory to CI trigger * Add tools/ directory to integration-test CI trigger Co-authored-by: Adi Seredinschi <adi@informal.systems>
This adds integration test for #1879.
Description
PR author checklist:
unclog
.docs/
).Reviewer checklist:
Files changed
in the GitHub PR explorer.