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.

Sign up for GitHub

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

Jump to bottom

Another fix for the e2e tests #1837

Merged
merged 4 commits into from
Jan 18, 2024
Merged

Another fix for the e2e tests #1837

merged 4 commits into from
Jan 18, 2024

Commits on Jan 18, 2024

  1. fix(tests-e2e): Delete another irrelevant test

    @luckysori
    luckysori committed Jan 18, 2024
    Configuration menu
    Copy the full SHA
    3280cf1 View commit details
    Browse the repository at this point in the history

  2. chore(tests-e2e): Delete dead code

    @luckysori
    luckysori committed Jan 18, 2024
    Configuration menu
    Copy the full SHA
    0efb791 View commit details
    Browse the repository at this point in the history

  3. fix(tests-e2e): Refresh wallet periodically after initial funding

    @luckysori
    luckysori committed Jan 18, 2024
    Configuration menu
    Copy the full SHA
    d929916 View commit details
    Browse the repository at this point in the history

  4. fix(coordinator): Allow requests with up to 50MB bodies 

    By default this is set to 2MB, according to the docs[1].

We need larger bodies than before (even in the e2e test!), because we
added lots of CET adaptor signatures to the database recently[2].

I also played around with nginx's `client_max_body_size`, but that's
actually irrelevant since we don't use nginx as a reverse proxy for
the coordinator in the e2e tests.

Related to #1719.

[1]: https://docs.rs/axum/latest/axum/extract/struct.DefaultBodyLimit.html#method.disable.
[2]: #1817.
    @luckysori
    luckysori committed Jan 18, 2024
    Configuration menu
    Copy the full SHA
    2dccdd8 View commit details
    Browse the repository at this point in the history