refactor: use node= instead of nodeid= in all internal fixture registration

Update all internal call sites to use node= parameter instead of nodeid=
string for fixture registration. This enables node-based matching.

Changes:
- python.py: Module and Class xunit fixtures now use node=self
- python.py: Class.collect parsefactories uses holder=..., node=self
- unittest.py: UnitTestCase fixtures now use node=self
- unittest.py: UnitTestCase.collect parsefactories uses holder=..., node=self

This completes Phase 1 of migrating from string-based to node-based
fixture scoping.

Co-authored-by: Cursor AI <ai@cursor.sh>
Co-authored-by: Anthropic Claude Opus 4 <claude@anthropic.com>

Author: 3 people

src/_pytest/python.py

@@ -595,7 +595,7 @@ def xunit_setup_module_fixture(request) -> Generator[None]:
             # Use a unique name to speed up lookup.
             name=f"_xunit_setup_module_fixture_{self.obj.__name__}",
             func=xunit_setup_module_fixture,
-            nodeid=self.nodeid,
+            node=self,
             scope="module",
             autouse=True,
         )
@@ -631,7 +631,7 @@ def xunit_setup_function_fixture(request) -> Generator[None]:
             # Use a unique name to speed up lookup.
             name=f"_xunit_setup_function_fixture_{self.obj.__name__}",
             func=xunit_setup_function_fixture,
-            nodeid=self.nodeid,
+            node=self,
             scope="function",
             autouse=True,
         )
@@ -779,7 +779,9 @@ def collect(self) -> Iterable[nodes.Item | nodes.Collector]:
         self._register_setup_class_fixture()
         self._register_setup_method_fixture()
 
-        self.session._fixturemanager.parsefactories(self.newinstance(), self.nodeid)
+        self.session._fixturemanager.parsefactories(
+            holder=self.newinstance(), node=self
+        )
 
         return super().collect()
 
@@ -809,7 +811,7 @@ def xunit_setup_class_fixture(request) -> Generator[None]:
             # Use a unique name to speed up lookup.
             name=f"_xunit_setup_class_fixture_{self.obj.__qualname__}",
             func=xunit_setup_class_fixture,
-            nodeid=self.nodeid,
+            node=self,
             scope="class",
             autouse=True,
         )
@@ -843,7 +845,7 @@ def xunit_setup_method_fixture(request) -> Generator[None]:
             # Use a unique name to speed up lookup.
             name=f"_xunit_setup_method_fixture_{self.obj.__qualname__}",
             func=xunit_setup_method_fixture,
-            nodeid=self.nodeid,
+            node=self,
             scope="function",
             autouse=True,
         )

src/_pytest/unittest.py

@@ -101,7 +101,9 @@ def collect(self) -> Iterable[Item | Collector]:
             self._register_unittest_setup_class_fixture(cls)
             self._register_setup_class_fixture()
 
-        self.session._fixturemanager.parsefactories(self.newinstance(), self.nodeid)
+        self.session._fixturemanager.parsefactories(
+            holder=self.newinstance(), node=self
+        )
 
         loader = TestLoader()
         foundsomething = False
@@ -170,7 +172,7 @@ def unittest_setup_class_fixture(
             # Use a unique name to speed up lookup.
             name=f"_unittest_setUpClass_fixture_{cls.__qualname__}",
             func=unittest_setup_class_fixture,
-            nodeid=self.nodeid,
+            node=self,
             scope="class",
             autouse=True,
         )
@@ -200,7 +202,7 @@ def unittest_setup_method_fixture(
             # Use a unique name to speed up lookup.
             name=f"_unittest_setup_method_fixture_{cls.__qualname__}",
             func=unittest_setup_method_fixture,
-            nodeid=self.nodeid,
+            node=self,
             scope="function",
             autouse=True,
         )