PR: Try to decrease timeouts when running tests and check general tests robustness and speed (CI/Testing)

[dalthviz]

Description of Changes

Some changes to improve CI tests runs robustness:

• Check if closing the mainwindow instance for tests marked as know_leak helps reducing timeouts/number of tests runs attempts
• Add missing logic to close editorwindows.
• Remove usage of connect_to_oi Editor config option
• Add order mark to a couple of test that were failing consistently: test_add_history and test_save_with_preferred_eol_chars
• Comment out style application over plots and sidebardialog fixtures to decrease chance of segfault over fast tests
• Prevent 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 via shellwidget.get_matplotlib_backend()
• Prevent plots RuntimeError when testing due to scrollbars timer logic
• Add logic to test_updates_appenv to prevent failing due to possible HTTP 403 errors
• Change IPython Console tests percentage being run
• Quit IPdb in some test before finishing them

Affirmation

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

[ccordoba12]

@dalthviz, while you're checking this, you could also decrease the percentage of IPython console tests run in the slow slots on Windows:

spyder/conftest.py

Lines 70 to 87 in 1fda346

	# Break test suite in CIs according to the following criteria:
	# * Mark all main window tests, and a percentage of the IPython console
	# ones, as slow.
	# * All other tests will be considered as fast.
	# This provides a more balanced partitioning of our test suite (in terms of
	# necessary time to run it) between the slow and fast slots we have on CIs.
	slow_items = []
	if os.environ.get('CI'):
	slow_items = [
	item for item in items if 'test_mainwindow' in item.nodeid
	]
	ipyconsole_items = [
	item for item in items if 'test_ipythonconsole' in item.nodeid
	]
	if os.name == 'nt':
	percentage = 0.5

That will make those slots run faster.

[pep8speaks]

Hello @dalthviz! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

• In the file spyder/plugins/updatemanager/tests/test_update_manager.py:

Line 67:80: E501 line too long (81 > 79 characters)

Comment last updated at 2024-05-30 21:19:28 UTC

[ccordoba12]

Thanks @dalthviz for your hard work on this!

[ccordoba12]

There's also a small error when running the teardown of test_outline_in_maximized_editor on the Windows slot tests, but that should be easy to fix.

[ccordoba12]

This is great! Thanks @dalthviz for taking the time to take a look at it.