tox/pytest: Add testpaths and use importlib

tomschr · tomschr · commit f24a1dace517 · 2022-12-20T10:39:39.000+01:00
Switch to the more modern importlib approach as it doesn't require to modify sys.path: https://docs.pytest.org/en/7.2.x/explanation/pythonpath.html
diff --git a/changelog.d/388.trivial.rst b/changelog.d/388.trivial.rst
@@ -0,0 +1,3 @@
+For pytest, switch to the more modern :mod:`importlib` approach
+as it doesn't require to modify :data:`sys.path`:
+https://docs.pytest.org/en/7.2.x/explanation/pythonpath.html
diff --git a/setup.cfg b/setup.cfg
@@ -55,12 +55,12 @@ semver = py.typed
 [tool:pytest]
 norecursedirs = .git build .env/ env/ .pyenv/ .tmp/ .eggs/ venv/
 testpaths = tests docs
-# pythonpath = src
+pythonpath = src tests
 filterwarnings =
     ignore:Function 'semver.*:DeprecationWarning
     # ' <- This apostroph is just to fix syntax highlighting
 addopts =
-    # --import-mode=importlib
+    --import-mode=importlib
     --no-cov-on-fail
     --cov=semver
     --cov-report=term-missing
diff --git a/tests/conftest.py b/tests/conftest.py
@@ -4,8 +4,6 @@
 
 import semver
 
-# sys.path.insert(0, "docs/usage")
-
 from coerce import coerce  # noqa:E402
 from semverwithvprefix import SemVerWithVPrefix  # noqa:E402
 import packaging.version
diff --git a/tox.ini b/tox.ini
@@ -19,6 +19,7 @@ python =
 [testenv]
 description = Run test suite for {basepython}
 allowlist_externals = make
+skip_install = true
 commands = pytest {posargs:}
 deps =
     pytest

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	+For pytest, switch to the more modern :mod:`importlib` approach
	`2`	+as it doesn't require to modify :data:`sys.path`:
	`3`	`+https://docs.pytest.org/en/7.2.x/explanation/pythonpath.html`