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

ICS20-V2 Path Forwarding E2E Tests #6578

Closed
5 of 8 tasks
chatton opened this issue Jun 12, 2024 · 4 comments
Closed
5 of 8 tasks

ICS20-V2 Path Forwarding E2E Tests #6578

chatton opened this issue Jun 12, 2024 · 4 comments

Comments

@chatton
Copy link
Contributor

chatton commented Jun 12, 2024

Summary

Tracking issue for ICS20-V2 Path Forwarding E2E tests

Tests:


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged/assigned
@bznein
Copy link
Contributor

bznein commented Jul 3, 2024

I'll start by taking the following one:

Given 3 chains with ics20-2 channels between A and B, and between B and C, perform a successful transfer of tokens from A to C through B.

@crodriguezvega
Copy link
Contributor

Just a note to remember to update the unreleased compatibility tests when we have all the e2e tests ready.

@bznein
Copy link
Contributor

bznein commented Jul 10, 2024

Given 3 chains with ics20-2 channels between A and B, and between B and C, perform a transfer of tokens from A to C through B, where the hop between B and C will time out. [Nikolas]

I discussed with @chatton on this one and it doesn't seem like we have a way to cause a timeout on e2e tests. We could arrange something like creating a relayer that only watches A->B, wait for timeout, and then update it to watch B to C, but that would be ugly and not possible to do cleanly right now. Right now we do have an integration timeout test (TestOnTimeoutPacketForwarding) which might be enough. @crodriguezvega what do you think?

@crodriguezvega
Copy link
Contributor

All tests have been completed 🥳 Shout-out to @bznein for taking care of all these tests!

@github-project-automation github-project-automation bot moved this from In review 👀 to Done 🥳 in ibc-go Jul 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

3 participants