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

Skip sphinx.locale.init_console when running tests #11159

Merged
merged 3 commits into from
Jan 31, 2023

Conversation

jeanas
Copy link
Contributor

@jeanas jeanas commented Jan 29, 2023

Feature or Bugfix

Bugfix

Purpose

Tests for Sphinx's CLIs, like test_apidoc, indirectly init_console to initialize translations for Sphinx's console domain, the one with Sphinx's log messages. This bleeds over subsequent tests by making warnings translated. Fix this by skipping init_console when running Sphinx's test suite.

Fixes #10500

Relates

#10500

@jeanas
Copy link
Contributor Author

jeanas commented Jan 29, 2023

I tested this wrongly. I'll fix it.

@jeanas
Copy link
Contributor Author

jeanas commented Jan 29, 2023

Updated. The ruff failures seem unrelated.

tests/conftest.py Outdated Show resolved Hide resolved
Tests for Sphinx's CLIs, like test_apidoc, indirectly init_console to
initialize translations for Sphinx's console domain, the one with
Sphinx's log messages. This bleeds over subsequent tests by making
warnings translated. Fix this by skipping init_console when running
Sphinx's test suite.

Fixes sphinx-doc#10500
@AA-Turner AA-Turner merged commit ecfd08d into sphinx-doc:master Jan 31, 2023
@jeanas jeanas deleted the test-failures branch January 31, 2023 22:12
@jeanas
Copy link
Contributor Author

jeanas commented Jan 31, 2023

Thanks for merging!

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 3, 2023
@AA-Turner AA-Turner added this to the 6.2.0 milestone Apr 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants