FSCollector: use session._node_location_to_relpath for nodeids

This is relevant for `pytest t/foo.py --rootdir=/tmp`.

Before:
```
../../../../tmp F.sxx
…
t/foo.py:5: ValueError
…
FAILED ../../../../tmp/::test_fail - ValueError
```

This removes `_check_initialpaths_for_relpath` (added via
pytest-dev#2776 to address part of the
issue), but it is apparently bad trying to make them relative to any
given arg, when they are meant to be relative to `rootdir` really.

Author: blueyed

changelog/6701.bugfix.rst

@@ -0,0 +1 @@
+Node ids for paths outside of the rootdir are generated properly, e.g. for ``pytest testing --rootdir=/tmp -vv``.

src/_pytest/nodes.py

@@ -422,12 +422,6 @@ def _prunetraceback(self, excinfo):
             excinfo.traceback = ntraceback.filter()
 
 
-def _check_initialpaths_for_relpath(session, fspath):
-    for initial_path in session._initialpaths:
-        if fspath.common(initial_path) == initial_path:
-            return fspath.relto(initial_path)
-
-
 class FSHookProxy:
     def __init__(
         self, fspath: py.path.local, pm: PytestPluginManager, remove_mods
@@ -444,7 +438,12 @@ def __getattr__(self, name: str):
 
 class FSCollector(Collector):
     def __init__(
-        self, fspath: py.path.local, parent=None, config=None, session=None, nodeid=None
+        self,
+        fspath: py.path.local,
+        parent=None,
+        config=None,
+        session=None,
+        nodeid: Optional[str] = None,
     ) -> None:
         name = fspath.basename
         if parent is not None:
@@ -457,11 +456,8 @@ def __init__(
         session = session or parent.session
 
         if nodeid is None:
-            nodeid = self.fspath.relto(session.config.rootdir)
-
-            if not nodeid:
-                nodeid = _check_initialpaths_for_relpath(session, fspath)
-            if nodeid and os.sep != SEP:
+            nodeid = session._node_location_to_relpath(self.fspath)
+            if os.sep != SEP:
                 nodeid = nodeid.replace(os.sep, SEP)
 
         super().__init__(name, parent, config, session, nodeid=nodeid, fspath=fspath)

testing/acceptance_test.py

@@ -730,8 +730,8 @@ def test_cmdline_python_namespace_package(self, testdir, monkeypatch):
         assert result.ret == 0
         result.stdout.fnmatch_lines(
             [
-                "test_hello.py::test_hello*PASSED*",
-                "test_hello.py::test_other*PASSED*",
+                "../hello/ns_pkg/hello/test_hello.py::test_hello PASSED *",
+                "../hello/ns_pkg/hello/test_hello.py::test_other PASSED *",
                 "ns_pkg/world/test_world.py::test_world*PASSED*",
                 "ns_pkg/world/test_world.py::test_other*PASSED*",
                 "*4 passed in*",

testing/python/fixtures.py

@@ -3690,6 +3690,7 @@ def test_foo(request):
         result = testdir.runpytest()
         result.stdout.fnmatch_lines(
             [
+                "test_foos.py F *",
                 "The requested fixture has no parameter defined for test:",
                 "    test_foos.py::test_foo",
                 "",
@@ -3707,8 +3708,9 @@ def test_foo(request):
         result = testdir.runpytest("--rootdir", rootdir, tests_dir)
         result.stdout.fnmatch_lines(
             [
+                "../tests/test_foos.py F *",
                 "The requested fixture has no parameter defined for test:",
-                "    test_foos.py::test_foo",
+                "    ../tests/test_foos.py::test_foo",
                 "",
                 "Requested fixture 'fix_with_param' defined in:",
                 "{}:4".format(fixfile),

testing/test_conftest.py

@@ -274,7 +274,14 @@ def fixture():
         build.join(f).mksymlinkto(real.join(f))
     build.chdir()
     result = testdir.runpytest("-vs", "app/test_foo.py")
-    result.stdout.fnmatch_lines(["*conftest_loaded*", "PASSED"])
+    result.stdout.fnmatch_lines(
+        [
+            "conftest_loaded",
+            "../real/app/test_foo.py::test1 fixture_used",
+            "PASSED",
+            "*= 1 passed in *",
+        ]
+    )
     assert result.ret == ExitCode.OK
 
 

testing/test_nodes.py

@@ -1,5 +1,3 @@
-import py
-
 import pytest
 from _pytest import nodes
 
@@ -38,23 +36,3 @@ def test():
     )
     with pytest.raises(ValueError, match=".*instance of PytestWarning.*"):
         items[0].warn(UserWarning("some warning"))
-
-
-def test__check_initialpaths_for_relpath():
-    """Ensure that it handles dirs, and does not always use dirname."""
-    cwd = py.path.local()
-
-    class FakeSession:
-        _initialpaths = [cwd]
-
-    assert nodes._check_initialpaths_for_relpath(FakeSession, cwd) == ""
-
-    sub = cwd.join("file")
-
-    class FakeSession:
-        _initialpaths = [cwd]
-
-    assert nodes._check_initialpaths_for_relpath(FakeSession, sub) == "file"
-
-    outside = py.path.local("/outside")
-    assert nodes._check_initialpaths_for_relpath(FakeSession, outside) is None

testing/test_session.py

@@ -1,5 +1,8 @@
+import os
+
 import pytest
 from _pytest.config import ExitCode
+from _pytest.pytester import Testdir
 
 
 class SessionTests:
@@ -334,7 +337,7 @@ def pytest_sessionfinish():
 
 
 @pytest.mark.parametrize("path", ["root", "{relative}/root", "{environment}/root"])
-def test_rootdir_option_arg(testdir, monkeypatch, path):
+def test_rootdir_option_arg(testdir: Testdir, monkeypatch, path: str) -> None:
     monkeypatch.setenv("PY_ROOTDIR_PATH", str(testdir.tmpdir))
     path = path.format(relative=str(testdir.tmpdir), environment="$PY_ROOTDIR_PATH")
 
@@ -344,15 +347,17 @@ def test_rootdir_option_arg(testdir, monkeypatch, path):
         """
         import os
         def test_one():
-            assert 1
-    """
+            assert os.getcwd() == {!r}
+    """.format(
+            os.getcwd()
+        )
     )
 
     result = testdir.runpytest("--rootdir={}".format(path))
     result.stdout.fnmatch_lines(
         [
             "*rootdir: {}/root".format(testdir.tmpdir),
-            "root/test_rootdir_option_arg.py *",
+            "test_rootdir_option_arg.py *",
             "*1 passed*",
         ]
     )