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

internal fixes for intermittent test failures #3707

Merged
merged 8 commits into from
Nov 21, 2022

Conversation

moodyjon
Copy link
Contributor

This completes pretty much all the work I planned to do on lbry-sdk test reliability. It should unblock more python3.9 work.

Fixes #3665
Fixes #3671

Bumping hub gets us the benefit of these fixes:
lbryio/hub#98
lbryio/hub#101
lbryio/hub#104

I also revisited #3658 because I encountered a new zlib.error "invalid window size".

I've started seeing failures like this when running tests. Something changed in the environment I think.
https://github.com/moodyjon/lbry-sdk/actions/runs/3447835647/jobs/5754426649
Successful run from 2mo ago with python 3.7.14 (failure was with 3.7.15):
https://github.com/moodyjon/lbry-sdk/actions/runs/3108575245/jobs/5038289310
Upgrade to pyinstaller==4.6 was suggested here:
pyinstaller/pyinstaller#6325

@coveralls
Copy link

coveralls commented Nov 11, 2022

Coverage Status

Coverage remained the same at 56.648% when pulling 6ec9ede on moodyjon:misc_test_fixes2 into 8c6c7b6 on lbryio:master.

@eukreign eukreign merged commit 001819d into lbryio:master Nov 21, 2022
@eukreign eukreign changed the title Miscellaneous fixes for intermittent test failures 2 internal fixes for intermittent test failures Jan 23, 2023
@eukreign eukreign added type: improvement Existing (or partially existing) functionality needs to be changed area: tests labels Jan 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: tests type: improvement Existing (or partially existing) functionality needs to be changed
Projects
None yet
3 participants