remove the __in_named_ctor concpet, its not needed with the simplification

Author: RonnyPfannschmidt

src/_pytest/nodes.py

@@ -70,19 +70,11 @@ class NodeMeta(type):
     __in_named_ctor = False
 
     def __call__(self, *k, **kw):
-        if not self.__in_named_ctor:
-            warnings.warn(NODE_USE_FROM_PARENT.format(name=self.__name__), stacklevel=3)
-        return self._create(*k, **kw)
-
-    @contextmanager
-    def __ctor_entered(self):
-        self.__in_named_ctor = True
-        yield
-        self.__in_named_ctor = False
+        warnings.warn(NODE_USE_FROM_PARENT.format(name=self.__name__), stacklevel=3)
+        return super().__call__(*k, **kw)
 
     def _create(self, *k, **kw):
-        with self.__ctor_entered():
-            return super().__call__(*k, **kw)
+        return super().__call__(*k, **kw)
 
 
 class Node(metaclass=NodeMeta):
@@ -128,8 +120,8 @@ def __init__(
                 self._nodeid += "::" + self.name
 
     @classmethod
-    def from_parent(cls, parent, **kw):
-        return cls._create(**kw, parent=parent)
+    def from_parent(cls, parent, *, name):
+        return cls._create(parent=parent, name=name)
 
     @property
     def ihook(self):
@@ -411,6 +403,10 @@ def __init__(self, fspath, parent=None, config=None, session=None, nodeid=None):
 
         super().__init__(name, parent, config, session, nodeid=nodeid, fspath=fspath)
 
+    @classmethod
+    def from_parent(cls, parent, *, fspath):
+        return cls._create(parent=parent, fspath=fspath)
+
 
 class File(FSCollector):
     """ base class for collecting tests from a file. """

src/_pytest/python.py

@@ -1401,6 +1401,12 @@ def __init__(
         #: .. versionadded:: 3.0
         self.originalname = originalname
 
+    @classmethod
+    def from_parent(
+        cls, parent, **kw
+    ):
+        return cls._create(parent=parent, **kw)
+
     def _initrequest(self):
         self.funcargs = {}
         self._request = fixtures.FixtureRequest(self)