Improve warning message to indicate the node id

nicoddemus · nicoddemus · commit 1b6b836c3307 · 2024-11-29T17:40:23.000-03:00
diff --git a/src/_pytest/fixtures.py b/src/_pytest/fixtures.py
@@ -1566,7 +1566,9 @@ def _getusefixturesnames(self, node: nodes.Item) -> Iterator[str]:
         for marker_node, mark in node.iter_markers_with_node(name="usefixtures"):
             if not mark.args:
                 marker_node.warn(
-                    PytestWarning("usefixtures() without arguments has no effect")
+                    PytestWarning(
+                        f"usefixtures() in {node.nodeid} without arguments has no effect"
+                    )
                 )
             yield from mark.args
 
diff --git a/testing/python/fixtures.py b/testing/python/fixtures.py
@@ -1448,7 +1448,8 @@ def test_one():
         )
         result = pytester.runpytest()
         result.stdout.fnmatch_lines(
-            "*PytestWarning: usefixtures() without arguments has no effect"
+            "*PytestWarning: usefixtures() in test_empty_usefixtures_marker.py::test_one"
+            " without arguments has no effect"
         )
 
     def test_usefixtures_ini(self, pytester: Pytester) -> None:

Original file line number	Diff line number	Diff line change
`@@ -1448,7 +1448,8 @@ def test_one():`
`1448`	`1448`	`)`
`1449`	`1449`	`result = pytester.runpytest()`
`1450`	`1450`	`result.stdout.fnmatch_lines(`
`1451`		`- "*PytestWarning: usefixtures() without arguments has no effect"`
	`1451`	`+ "*PytestWarning: usefixtures() in test_empty_usefixtures_marker.py::test_one"`
	`1452`	`+ " without arguments has no effect"`
`1452`	`1453`	`)`
`1453`	`1454`
`1454`	`1455`	`def test_usefixtures_ini(self, pytester: Pytester) -> None:`