-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
PR: Try to decrease timeouts when running tests and check general tests robustness and speed (CI/Testing) #22077
Conversation
…f a 403 rate limit error happens
@dalthviz, while you're checking this, you could also decrease the percentage of IPython console tests run in the slow slots on Windows: Lines 70 to 87 in 1fda346
That will make those slots run faster. |
…ra wait to finish closing tour while testing
…ing main widget. Exit debug mode to prevent timeout
Hello @dalthviz! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2024-05-30 21:19:28 UTC |
…plate testing plugin
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @dalthviz for your hard work on this!
There's also a small error when running the teardown of |
Co-authored-by: Carlos Cordoba <ccordoba12@gmail.com>
Co-authored-by: Carlos Cordoba <ccordoba12@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great! Thanks @dalthviz for taking the time to take a look at it.
Description of Changes
Some changes to improve CI tests runs robustness:
know_leak
helps reducing timeouts/number of tests runs attemptsconnect_to_oi
Editor config optiontest_add_history
andtest_save_with_preferred_eol_chars
spyder_kernels.comms.commbase.CommError: The comm is not connected.
on Windows when running IPython Console test by setting matplotlib backend to inline when running test instead of trying to get it viashellwidget.get_matplotlib_backend()
RuntimeError
when testing due to scrollbars timer logictest_updates_appenv
to prevent failing due to possible HTTP 403 errorsAffirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.
I certify the above statement is true and correct: dalthviz