Fix -vv overriding --durations-min (#12938)

Author: JulianJvn

AUTHORS

@@ -225,6 +225,7 @@ Joseph Hunkeler
 Joseph Sawaya
 Josh Karpel
 Joshua Bronson
+Julian Valentin
 Jurko Gospodnetić
 Justice Ndou
 Justyna Janczyszyn

changelog/12938.bugfix.rst

@@ -0,0 +1 @@
+Fixed ``--durations-min`` argument not respected if ``-vv`` is used.

src/_pytest/runner.py

@@ -61,7 +61,7 @@ def pytest_addoption(parser: Parser) -> None:
         "--durations-min",
         action="store",
         type=float,
-        default=0.005,
+        default=None,
         metavar="N",
         help="Minimal duration in seconds for inclusion in slowest list. "
         "Default: 0.005.",
@@ -74,6 +74,8 @@ def pytest_terminal_summary(terminalreporter: TerminalReporter) -> None:
     verbose = terminalreporter.config.get_verbosity()
     if durations is None:
         return
+    if durations_min is None:
+        durations_min = 0.005 if verbose < 2 else 0.0
     tr = terminalreporter
     dlist = []
     for replist in tr.stats.values():
@@ -90,10 +92,16 @@ def pytest_terminal_summary(terminalreporter: TerminalReporter) -> None:
         dlist = dlist[:durations]
 
     for i, rep in enumerate(dlist):
-        if verbose < 2 and rep.duration < durations_min:
+        if rep.duration < durations_min:
             tr.write_line("")
             tr.write_line(
-                f"({len(dlist) - i} durations < {durations_min:g}s hidden.  Use -vv to show these durations.)"
+                f"({len(dlist) - i} durations < {durations_min:g}s hidden."
+                + (
+                    "  Use -vv to show these durations."
+                    if terminalreporter.config.option.durations_min is None
+                    else ""
+                )
+                + ")"
             )
             break
         tr.write_line(f"{rep.duration:02.2f}s {rep.when:<8} {rep.nodeid}")

testing/acceptance_test.py

@@ -1,6 +1,7 @@
 # mypy: allow-untyped-defs
 from __future__ import annotations
 
+from collections.abc import Sequence
 import dataclasses
 import importlib.metadata
 import os
@@ -970,28 +971,39 @@ def test_calls_showall(self, pytester: Pytester, mock_timing) -> None:
         pytester.makepyfile(self.source)
         result = pytester.runpytest_inprocess("--durations=0")
         assert result.ret == 0
-
-        tested = "3"
-        for x in tested:
-            for y in ("call",):  # 'setup', 'call', 'teardown':
-                for line in result.stdout.lines:
-                    if (f"test_{x}") in line and y in line:
-                        break
-                else:
-                    raise AssertionError(f"not found {x} {y}")
+        print(result.stdout)
+        TestDurations.check_tests_in_output(result.stdout.lines, "23")
 
     def test_calls_showall_verbose(self, pytester: Pytester, mock_timing) -> None:
         pytester.makepyfile(self.source)
         result = pytester.runpytest_inprocess("--durations=0", "-vv")
         assert result.ret == 0
+        TestDurations.check_tests_in_output(result.stdout.lines, "123")
 
-        for x in "123":
-            for y in ("call",):  # 'setup', 'call', 'teardown':
-                for line in result.stdout.lines:
-                    if (f"test_{x}") in line and y in line:
-                        break
-                else:
-                    raise AssertionError(f"not found {x} {y}")
+    def test_calls_showall_durationsmin(self, pytester: Pytester, mock_timing) -> None:
+        pytester.makepyfile(self.source)
+        result = pytester.runpytest_inprocess("--durations=0", "--durations-min=0.015")
+        assert result.ret == 0
+        TestDurations.check_tests_in_output(result.stdout.lines, "3")
+
+    def test_calls_showall_durationsmin_verbose(
+        self, pytester: Pytester, mock_timing
+    ) -> None:
+        pytester.makepyfile(self.source)
+        result = pytester.runpytest_inprocess(
+            "--durations=0", "--durations-min=0.015", "-vv"
+        )
+        assert result.ret == 0
+        TestDurations.check_tests_in_output(result.stdout.lines, "3")
+
+    @staticmethod
+    def check_tests_in_output(lines: Sequence[str], expected_test_numbers: str) -> None:
+        found_test_numbers = "".join(
+            test_number
+            for test_number in "123"
+            if any(f"test_{test_number}" in line and " call " in line for line in lines)
+        )
+        assert found_test_numbers == expected_test_numbers
 
     def test_with_deselected(self, pytester: Pytester, mock_timing) -> None:
         pytester.makepyfile(self.source)