-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
[BUG] Test suite randomly freezing since update to 10.15.1 #1734
Comments
That does sounds suspiciously like a deadlock I thought I fixed in 10.15.0. Which I can't reproduce with 10.15.1, locally or with Github Actions. Although deadlocks are often hard to reproduce. The tests you linked to in Rich were running in 10.15.0, which would explain the deadlock there. The LiveError you get is due to nesting Live (or status or progress) instances, which won't work correctly. You could also get this if your test suite calls enter without a corresponding exit. This could be a bug in Rich. Deadlocks are tricky. But since I can't reproduce it, I'd check your test suite first. Makes sure you aren't doing anything in the tests which might cause this. It's possible that 10.15.1 exposed a bug in your code. Otherwise, a full traceback may help me diagnose the issue. |
I did manage to replicate this. Please try v10.15.2 |
I installed v10.15.2 and did several CI runs on a couple different repos and am not seeing the issue any more. Thanks for the fix! |
Did I solve your problem? Consider sponsoring the ongoing work on Rich and Textual. Or buy me a coffee to say thanks. |
Describe the bug
Since upgrading to Rich 10.15.1 I am seeing my test suites randomly get stuck. So far I have only noticed this when running tests and have not seen it when running programs outside of the tests. Here is one example. This issue is also happening in the Rich test suite (example here). The tests stop on the pytest step and never make any progress until GitHub Actions eventually time out.
I have also been able to reproduce this locally so it is not specific to GitHub Actions only. If I stop the tests with
ctrl + c
locally I get the following error (the full stack trace is really long so I'm only posting the last error, if more will help I can post it):As far as I can tell the issue is random. It happens frequently, but I haven't been able to find a way to guarantee it happens.
Platform
Locally I am using Linux, but in GitHub Actions I have see the issue on Linux, Mac, and Windows.
I may ask you to cut and paste the output of the following commands. It may save some time if you do it now.
platform="Linux" WindowsConsoleFeatures(vt=False, truecolor=False)
The text was updated successfully, but these errors were encountered: