Skip to content

internal fixes for intermittent test failures#3707

Merged
eukreign merged 8 commits intolbryio:masterfrom
moodyjon:misc_test_fixes2
Nov 21, 2022
Merged

internal fixes for intermittent test failures#3707
eukreign merged 8 commits intolbryio:masterfrom
moodyjon:misc_test_fixes2

Conversation

@moodyjon
Copy link
Copy Markdown
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
Copy Markdown

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