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

Remove retry loop from make_application. #2144

Merged
merged 1 commit into from
Jun 18, 2024
Merged

Conversation

afck
Copy link
Contributor

@afck afck commented Jun 17, 2024

This should make the end-to-end tests less flaky and slightly faster by making sure cross-chain messages have been processed when the test observes the result.

Motivation

Suppose an application is registered on chain A:

  • If chain B requests that application from chain A, we have to wait for
    • chain A to process the request, and
    • chain B to process the response…
  • If chain A sends an application message to chain B, we have to wait for
    • chain B to process the message…

…before we can expect the application to be registered on chain B.

In the tests, we currently just retry in a loop until this has happened.

Proposal

Call process_inbox in the right places, and remove the retry loop.

Test Plan

All affected tests run in CI.

Links

@afck afck force-pushed the make-application branch 2 times, most recently from 1c4faa0 to 97964d2 Compare June 18, 2024 11:03
@afck afck marked this pull request as ready for review June 18, 2024 12:17
@afck afck requested review from ma2bd and MathieuDutSik June 18, 2024 12:17
@afck afck merged commit 57b1072 into linera-io:main Jun 18, 2024
4 checks passed
@afck afck deleted the make-application branch June 18, 2024 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants