diff --git a/AUTHORS b/AUTHORS index a64c95acb..ff47f9d7c 100644 --- a/AUTHORS +++ b/AUTHORS @@ -56,7 +56,6 @@ Charnjit SiNGH (CCSJ) Chris Lamb Christian Boelsen Christian Fetzer -Christian Neumüller Christian Theunert Christian Tismer Christopher Gilling diff --git a/src/_pytest/config/__init__.py b/src/_pytest/config/__init__.py index 3a0eca546..b861563e9 100644 --- a/src/_pytest/config/__init__.py +++ b/src/_pytest/config/__init__.py @@ -30,7 +30,6 @@ from _pytest._code import filter_traceback from _pytest.compat import importlib_metadata from _pytest.outcomes import fail from _pytest.outcomes import Skipped -from _pytest.pathlib import unique_path from _pytest.warning_types import PytestConfigWarning hookimpl = HookimplMarker("pytest") @@ -367,7 +366,7 @@ class PytestPluginManager(PluginManager): """ current = py.path.local() self._confcutdir = ( - unique_path(current.join(namespace.confcutdir, abs=True)) + current.join(namespace.confcutdir, abs=True) if namespace.confcutdir else None ) @@ -406,18 +405,19 @@ class PytestPluginManager(PluginManager): else: directory = path - directory = unique_path(directory) - # XXX these days we may rather want to use config.rootdir # and allow users to opt into looking into the rootdir parent # directories instead of requiring to specify confcutdir clist = [] - for parent in directory.parts(): + for parent in directory.realpath().parts(): if self._confcutdir and self._confcutdir.relto(parent): continue conftestpath = parent.join("conftest.py") if conftestpath.isfile(): - mod = self._importconftest(conftestpath) + # Use realpath to avoid loading the same conftest twice + # with build systems that create build directories containing + # symlinks to actual files. + mod = self._importconftest(conftestpath.realpath()) clist.append(mod) self._dirpath2confmods[directory] = clist return clist @@ -432,10 +432,6 @@ class PytestPluginManager(PluginManager): raise KeyError(name) def _importconftest(self, conftestpath): - # Use realpath to avoid loading the same conftest twice - # with build systems that create build directories containing - # symlinks to actual files. - conftestpath = unique_path(conftestpath) try: return self._conftestpath2mod[conftestpath] except KeyError: diff --git a/src/_pytest/pathlib.py b/src/_pytest/pathlib.py index 0403b6947..19f9c062f 100644 --- a/src/_pytest/pathlib.py +++ b/src/_pytest/pathlib.py @@ -11,7 +11,6 @@ from functools import partial from os.path import expanduser from os.path import expandvars from os.path import isabs -from os.path import normcase from os.path import sep from posixpath import sep as posix_sep @@ -335,12 +334,3 @@ def fnmatch_ex(pattern, path): def parts(s): parts = s.split(sep) return {sep.join(parts[: i + 1]) or sep for i in range(len(parts))} - - -def unique_path(path): - """Returns a unique path in case-insensitive (but case-preserving) file - systems such as Windows. - - This is needed only for ``py.path.local``; ``pathlib.Path`` handles this - natively with ``resolve()``.""" - return type(path)(normcase(str(path.realpath()))) diff --git a/testing/test_conftest.py b/testing/test_conftest.py index 9888f5457..447416f10 100644 --- a/testing/test_conftest.py +++ b/testing/test_conftest.py @@ -1,4 +1,3 @@ -import os.path import textwrap import py @@ -6,7 +5,6 @@ import py import pytest from _pytest.config import PytestPluginManager from _pytest.main import ExitCode -from _pytest.pathlib import unique_path def ConftestWithSetinitial(path): @@ -143,11 +141,11 @@ def test_conftestcutdir(testdir): # but we can still import a conftest directly conftest._importconftest(conf) values = conftest._getconftestmodules(conf.dirpath()) - assert values[0].__file__.startswith(str(unique_path(conf))) + assert values[0].__file__.startswith(str(conf)) # and all sub paths get updated properly values = conftest._getconftestmodules(p) assert len(values) == 1 - assert values[0].__file__.startswith(str(unique_path(conf))) + assert values[0].__file__.startswith(str(conf)) def test_conftestcutdir_inplace_considered(testdir): @@ -156,7 +154,7 @@ def test_conftestcutdir_inplace_considered(testdir): conftest_setinitial(conftest, [conf.dirpath()], confcutdir=conf.dirpath()) values = conftest._getconftestmodules(conf.dirpath()) assert len(values) == 1 - assert values[0].__file__.startswith(str(unique_path(conf))) + assert values[0].__file__.startswith(str(conf)) @pytest.mark.parametrize("name", "test tests whatever .dotdir".split()) @@ -166,7 +164,7 @@ def test_setinitial_conftest_subdirs(testdir, name): conftest = PytestPluginManager() conftest_setinitial(conftest, [sub.dirpath()], confcutdir=testdir.tmpdir) if name not in ("whatever", ".dotdir"): - assert unique_path(subconftest) in conftest._conftestpath2mod + assert subconftest in conftest._conftestpath2mod assert len(conftest._conftestpath2mod) == 1 else: assert subconftest not in conftest._conftestpath2mod @@ -277,21 +275,6 @@ def test_conftest_symlink_files(testdir): assert result.ret == ExitCode.OK -@pytest.mark.skipif( - os.path.normcase("x") != os.path.normcase("X"), - reason="only relevant for case insensitive file systems", -) -def test_conftest_badcase(testdir): - """Check conftest.py loading when directory casing is wrong.""" - testdir.tmpdir.mkdir("JenkinsRoot").mkdir("test") - source = {"setup.py": "", "test/__init__.py": "", "test/conftest.py": ""} - testdir.makepyfile(**{"JenkinsRoot/%s" % k: v for k, v in source.items()}) - - testdir.tmpdir.join("jenkinsroot/test").chdir() - result = testdir.runpytest() - assert result.ret == ExitCode.NO_TESTS_COLLECTED - - def test_no_conftest(testdir): testdir.makeconftest("assert 0") result = testdir.runpytest("--noconftest")