Merge pull request #11244 from bluetech/rootdir-tweaks

Rootdir tweaks

doc/en/how-to/usage.rst

@@ -46,21 +46,28 @@ Use ``""`` instead of ``''`` in expression when running this on Windows
 
 **Run tests by node ids**
 
-Each collected test is assigned a unique ``nodeid`` which consist of the module filename followed
-by specifiers like class names, function names and parameters from parametrization, separated by ``::`` characters.
+Each collected test is assigned a unique ``nodeid`` which consist of the module file path followed
+by specifiers like class names and function names separated by ``::`` characters,
+and parameters from parametrization in ``[...]``.
+You can use the same syntax to match tests relative to the working directory.
 
 To run a specific test within a module:
 
 .. code-block:: bash
 
-    pytest test_mod.py::test_func
+    pytest tests/test_mod.py::test_func
 
+To run all tests in a class:
 
-Another example specifying a test method in the command line:
+.. code-block:: bash
+
+    pytest tests/test_mod.py::TestClass
+
+Specifying a specific test method:
 
 .. code-block:: bash
 
-    pytest test_mod.py::TestClass::test_method
+    pytest tests/test_mod.py::TestClass::test_method
 
 **Run tests by marker expressions**
 

src/_pytest/config/__init__.py

@@ -1170,7 +1170,7 @@ def _initini(self, args: Sequence[str]) -> None:
             ns.inifilename,
             ns.file_or_dir + unknown_args,
             rootdir_cmd_arg=ns.rootdir or None,
-            config=self,
+            invocation_dir=self.invocation_params.dir,
         )
         self._rootpath = rootpath
         self._inipath = inipath
@@ -1243,7 +1243,7 @@ def _decide_args(
         self,
         *,
         args: List[str],
-        pyargs: List[str],
+        pyargs: bool,
         testpaths: List[str],
         invocation_dir: Path,
         rootpath: Path,

src/_pytest/config/findpaths.py

@@ -7,7 +7,6 @@
 from typing import Optional
 from typing import Sequence
 from typing import Tuple
-from typing import TYPE_CHECKING
 from typing import Union
 
 import iniconfig
@@ -17,9 +16,6 @@
 from _pytest.pathlib import absolutepath
 from _pytest.pathlib import commonpath
 
-if TYPE_CHECKING:
-    from . import Config
-
 
 def _parse_ini_config(path: Path) -> iniconfig.IniConfig:
     """Parse the given generic '.ini' file using legacy IniConfig parser, returning
@@ -176,8 +172,21 @@ def determine_setup(
     inifile: Optional[str],
     args: Sequence[str],
     rootdir_cmd_arg: Optional[str] = None,
-    config: Optional["Config"] = None,
+    invocation_dir: Optional[Path] = None,
 ) -> Tuple[Path, Optional[Path], Dict[str, Union[str, List[str]]]]:
+    """Determine the rootdir, inifile and ini configuration values from the
+    command line arguments.
+
+    :param inifile:
+        The `--inifile` command line argument, if given.
+    :param args:
+        The free command line arguments.
+    :param rootdir_cmd_arg:
+        The `--rootdir` command line argument, if given.
+    :param invocation_dir:
+        The working directory when pytest was invoked, if known.
+        If not known, the current working directory is used.
+    """
     rootdir = None
     dirs = get_dirs_from_args(args)
     if inifile:
@@ -198,8 +207,8 @@ def determine_setup(
                 if dirs != [ancestor]:
                     rootdir, inipath, inicfg = locate_config(dirs)
                 if rootdir is None:
-                    if config is not None:
-                        cwd = config.invocation_params.dir
+                    if invocation_dir is not None:
+                        cwd = invocation_dir
                     else:
                         cwd = Path.cwd()
                     rootdir = get_common_ancestor([cwd, ancestor])