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

pytest: speed up test_opening.py::test_funder_contribution_limits when !DEVELOPER #4583

Merged

Conversation

rustyrussell
Copy link
Contributor

@rustyrussell rustyrussell commented Jun 4, 2021

This test takes 695 seconds, because fundwallet waits for the wallet to
notice the tx, which takes 60 seconds if not DEVELOPER. Do all the waiting
at once, and this speeds the test up to 153 seconds.

Found by CI in #4581

Changelog-None

…n !DEVELOPER

This test takes 695 seconds, because fundwallet waits for the wallet to
notice the tx, which takes 60 seconds if not DEVELOPER.  Do all the waiting
at once, and this speeds the test up to 153 seconds.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
@rustyrussell rustyrussell added this to the v0.10.1 milestone Jun 4, 2021
@cdecker
Copy link
Member

cdecker commented Jun 4, 2021

ACK adbabbf

@rustyrussell rustyrussell merged commit 67c03c0 into ElementsProject:master Jun 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants