Merge pull request #8461 from francoisfreitag/flake8-linkcheck

Make tests/ a Python package

tests/roots/test-ext-doctest-skipif/conf.py

@@ -6,7 +6,7 @@ source_suffix = '.txt'
 exclude_patterns = ['_build']
 
 doctest_global_setup = '''
-from test_ext_doctest import record
+from tests.test_ext_doctest import record
 
 record('doctest_global_setup', 'body', True)
 '''

tests/roots/test-ext-doctest/doctest.txt

@@ -139,7 +139,7 @@ Special directives
 
 .. testcleanup:: *
 
-   import test_ext_doctest
+   from tests import test_ext_doctest
    test_ext_doctest.cleanup_call()
 
 non-ASCII result

tests/test_build_latex.py

@@ -16,7 +16,6 @@ from shutil import copyfile
 from subprocess import PIPE, CalledProcessError
 
 import pytest
-from test_build_html import ENV_WARNINGS
 
 from sphinx.builders.latex import default_latex_documents
 from sphinx.config import Config
@@ -26,6 +25,8 @@ from sphinx.util import docutils
 from sphinx.util.osutil import cd, ensuredir
 from sphinx.writers.latex import LaTeXTranslator
 
+from .test_build_html import ENV_WARNINGS
+
 LATEX_ENGINES = ['pdflatex', 'lualatex', 'xelatex']
 DOCCLASSES = ['howto', 'manual']
 STYLEFILES = ['article.cls', 'fancyhdr.sty', 'titlesec.sty', 'amsmath.sty',

tests/test_build_linkcheck.py

@@ -15,7 +15,7 @@ import textwrap
 import pytest
 import requests
 
-from utils import CERT_FILE, http_server, https_server, modify_env
+from .utils import CERT_FILE, http_server, https_server, modify_env
 
 
 @pytest.mark.sphinx('linkcheck', testroot='linkcheck', freshenv=True)

tests/test_build_texinfo.py

@@ -15,7 +15,6 @@ from subprocess import PIPE, CalledProcessError
 from unittest.mock import Mock
 
 import pytest
-from test_build_html import ENV_WARNINGS
 
 from sphinx.builders.texinfo import default_texinfo_documents
 from sphinx.config import Config
@@ -23,6 +22,8 @@ from sphinx.testing.util import strip_escseq
 from sphinx.util.docutils import new_document
 from sphinx.writers.texinfo import TexinfoTranslator
 
+from .test_build_html import ENV_WARNINGS
+
 TEXINFO_WARNINGS = ENV_WARNINGS + """\
 %(root)s/index.rst:\\d+: WARNING: unknown option: &option
 %(root)s/index.rst:\\d+: WARNING: citation not found: missing

tests/test_ext_autodoc_autoattribute.py

@@ -12,7 +12,8 @@
 import sys
 
 import pytest
-from test_ext_autodoc import do_autodoc
+
+from .test_ext_autodoc import do_autodoc
 
 
 @pytest.mark.sphinx('html', testroot='ext-autodoc')

tests/test_ext_autodoc_autoclass.py

@@ -12,7 +12,8 @@
 import sys
 
 import pytest
-from test_ext_autodoc import do_autodoc
+
+from .test_ext_autodoc import do_autodoc
 
 
 @pytest.mark.sphinx('html', testroot='ext-autodoc')

tests/test_ext_autodoc_autodata.py

@@ -12,7 +12,8 @@
 import sys
 
 import pytest
-from test_ext_autodoc import do_autodoc
+
+from .test_ext_autodoc import do_autodoc
 
 
 @pytest.mark.sphinx('html', testroot='ext-autodoc')

tests/test_ext_autodoc_autofunction.py

@@ -12,7 +12,8 @@
 import sys
 
 import pytest
-from test_ext_autodoc import do_autodoc
+
+from .test_ext_autodoc import do_autodoc
 
 
 @pytest.mark.sphinx('html', testroot='ext-autodoc')

tests/test_ext_autodoc_configs.py

@@ -12,10 +12,11 @@ import platform
 import sys
 
 import pytest
-from test_ext_autodoc import do_autodoc
 
 from sphinx.testing import restructuredtext
 
+from .test_ext_autodoc import do_autodoc
+
 IS_PYPY = platform.python_implementation() == 'PyPy'
 
 

tests/test_ext_autodoc_events.py

@@ -9,10 +9,11 @@
 """
 
 import pytest
-from test_ext_autodoc import do_autodoc
 
 from sphinx.ext.autodoc import between, cut_lines
 
+from .test_ext_autodoc import do_autodoc
+
 
 @pytest.mark.sphinx('html', testroot='ext-autodoc')
 def test_process_docstring(app):

tests/test_ext_autodoc_private_members.py

@@ -9,7 +9,8 @@
 """
 
 import pytest
-from test_ext_autodoc import do_autodoc
+
+from .test_ext_autodoc import do_autodoc
 
 
 @pytest.mark.sphinx('html', testroot='ext-autodoc')

tests/test_ext_intersphinx.py

@@ -15,14 +15,15 @@ from unittest import mock
 
 import pytest
 from docutils import nodes
-from test_util_inventory import inventory_v2, inventory_v2_not_having_version
 
 from sphinx import addnodes
 from sphinx.ext.intersphinx import (INVENTORY_FILENAME, _get_safe_url, _strip_basic_auth,
                                     fetch_inventory, inspect_main, load_mappings,
                                     missing_reference, normalize_intersphinx_mapping)
 from sphinx.ext.intersphinx import setup as intersphinx_setup
-from utils import http_server
+
+from .test_util_inventory import inventory_v2, inventory_v2_not_having_version
+from .utils import http_server
 
 
 def fake_node(domain, type, target, content, **attrs):

tests/test_util_inspect.py

@@ -129,8 +129,8 @@ def test_signature_partialmethod():
 
 
 def test_signature_annotations():
-    from typing_test_data import (Node, f0, f1, f2, f3, f4, f5, f6, f7, f8, f9, f10, f11, f12,
+    from .typing_test_data import (Node, f0, f1, f2, f3, f4, f5, f6, f7, f8, f9, f10, f11, f12,
-                                  f13, f14, f15, f16, f17, f18, f19, f20, f21)
+                                   f13, f14, f15, f16, f17, f18, f19, f20, f21)
 
     # Class annotations
     sig = inspect.signature(f0)
@@ -226,10 +226,10 @@ def test_signature_annotations():
     if (3, 5, 0) <= sys.version_info < (3, 5, 3):
         assert stringify_signature(sig) == '(self) -> List[Node]'
     else:
-        assert stringify_signature(sig) == '(self) -> List[typing_test_data.Node]'
+        assert stringify_signature(sig) == '(self) -> List[tests.typing_test_data.Node]'
 
     sig = inspect.signature(Node.__init__)
-    assert stringify_signature(sig) == '(self, parent: Optional[typing_test_data.Node]) -> None'
+    assert stringify_signature(sig) == '(self, parent: Optional[tests.typing_test_data.Node]) -> None'
 
     # show_annotation is False
     sig = inspect.signature(f7)

tests/test_util_typing.py

@@ -53,7 +53,7 @@ def test_restify_type_hints_containers():
     assert restify(Tuple[str, str, str]) == ":class:`Tuple`\\ [:class:`str`, :class:`str`, :class:`str`]"
     assert restify(Tuple[str, ...]) == ":class:`Tuple`\\ [:class:`str`, ...]"
     assert restify(List[Dict[str, Tuple]]) == ":class:`List`\\ [:class:`Dict`\\ [:class:`str`, :class:`Tuple`]]"
-    assert restify(MyList[Tuple[int, int]]) == ":class:`test_util_typing.MyList`\\ [:class:`Tuple`\\ [:class:`int`, :class:`int`]]"
+    assert restify(MyList[Tuple[int, int]]) == ":class:`tests.test_util_typing.MyList`\\ [:class:`Tuple`\\ [:class:`int`, :class:`int`]]"
     assert restify(Generator[None, None, None]) == ":class:`Generator`\\ [:obj:`None`, :obj:`None`, :obj:`None`]"
 
 
@@ -76,10 +76,10 @@ def test_restify_type_hints_Union():
     if sys.version_info >= (3, 7):
         assert restify(Union[int, Integral]) == ":obj:`Union`\\ [:class:`int`, :class:`numbers.Integral`]"
         assert (restify(Union[MyClass1, MyClass2]) ==
-                ":obj:`Union`\\ [:class:`test_util_typing.MyClass1`, :class:`test_util_typing.<MyClass2>`]")
+                ":obj:`Union`\\ [:class:`tests.test_util_typing.MyClass1`, :class:`tests.test_util_typing.<MyClass2>`]")
     else:
         assert restify(Union[int, Integral]) == ":class:`numbers.Integral`"
-        assert restify(Union[MyClass1, MyClass2]) == ":class:`test_util_typing.MyClass1`"
+        assert restify(Union[MyClass1, MyClass2]) == ":class:`tests.test_util_typing.MyClass1`"
 
 
 @pytest.mark.skipif(sys.version_info < (3, 7), reason='python 3.7+ is required.')
@@ -88,15 +88,15 @@ def test_restify_type_hints_typevars():
     T_co = TypeVar('T_co', covariant=True)
     T_contra = TypeVar('T_contra', contravariant=True)
 
-    assert restify(T) == ":obj:`test_util_typing.T`"
+    assert restify(T) == ":obj:`tests.test_util_typing.T`"
-    assert restify(T_co) == ":obj:`test_util_typing.T_co`"
+    assert restify(T_co) == ":obj:`tests.test_util_typing.T_co`"
-    assert restify(T_contra) == ":obj:`test_util_typing.T_contra`"
+    assert restify(T_contra) == ":obj:`tests.test_util_typing.T_contra`"
-    assert restify(List[T]) == ":class:`List`\\ [:obj:`test_util_typing.T`]"
+    assert restify(List[T]) == ":class:`List`\\ [:obj:`tests.test_util_typing.T`]"
 
 
 def test_restify_type_hints_custom_class():
-    assert restify(MyClass1) == ":class:`test_util_typing.MyClass1`"
+    assert restify(MyClass1) == ":class:`tests.test_util_typing.MyClass1`"
-    assert restify(MyClass2) == ":class:`test_util_typing.<MyClass2>`"
+    assert restify(MyClass2) == ":class:`tests.test_util_typing.<MyClass2>`"
 
 
 def test_restify_type_hints_alias():
@@ -107,7 +107,7 @@ def test_restify_type_hints_alias():
 
 
 def test_restify_broken_type_hints():
-    assert restify(BrokenType) == ':class:`test_util_typing.BrokenType`'
+    assert restify(BrokenType) == ':class:`tests.test_util_typing.BrokenType`'
 
 
 def test_stringify():
@@ -127,7 +127,7 @@ def test_stringify_type_hints_containers():
     assert stringify(Tuple[str, str, str]) == "Tuple[str, str, str]"
     assert stringify(Tuple[str, ...]) == "Tuple[str, ...]"
     assert stringify(List[Dict[str, Tuple]]) == "List[Dict[str, Tuple]]"
-    assert stringify(MyList[Tuple[int, int]]) == "test_util_typing.MyList[Tuple[int, int]]"
+    assert stringify(MyList[Tuple[int, int]]) == "tests.test_util_typing.MyList[Tuple[int, int]]"
     assert stringify(Generator[None, None, None]) == "Generator[None, None, None]"
 
 
@@ -164,10 +164,10 @@ def test_stringify_type_hints_Union():
     if sys.version_info >= (3, 7):
         assert stringify(Union[int, Integral]) == "Union[int, numbers.Integral]"
         assert (stringify(Union[MyClass1, MyClass2]) ==
-                "Union[test_util_typing.MyClass1, test_util_typing.<MyClass2>]")
+                "Union[tests.test_util_typing.MyClass1, tests.test_util_typing.<MyClass2>]")
     else:
         assert stringify(Union[int, Integral]) == "numbers.Integral"
-        assert stringify(Union[MyClass1, MyClass2]) == "test_util_typing.MyClass1"
+        assert stringify(Union[MyClass1, MyClass2]) == "tests.test_util_typing.MyClass1"
 
 
 def test_stringify_type_hints_typevars():
@@ -182,8 +182,8 @@ def test_stringify_type_hints_typevars():
 
 
 def test_stringify_type_hints_custom_class():
-    assert stringify(MyClass1) == "test_util_typing.MyClass1"
+    assert stringify(MyClass1) == "tests.test_util_typing.MyClass1"
-    assert stringify(MyClass2) == "test_util_typing.<MyClass2>"
+    assert stringify(MyClass2) == "tests.test_util_typing.<MyClass2>"
 
 
 def test_stringify_type_hints_alias():
@@ -194,4 +194,4 @@ def test_stringify_type_hints_alias():
 
 
 def test_stringify_broken_type_hints():
-    assert stringify(BrokenType) == 'test_util_typing.BrokenType'
+    assert stringify(BrokenType) == 'tests.test_util_typing.BrokenType'