test: remove redundant test

Author: Glyphack

src/_pytest/fixtures.py

@@ -1742,15 +1742,13 @@ def parsefactories(
             # The attribute can be an arbitrary descriptor, so the attribute
             # access below can raise. safe_getattr() ignores such exceptions.
             obj_ub = safe_getattr(holderobj_tp, name, None)
-            marker = getfixturemarker(obj_ub)
-            if not isinstance(marker, FixtureFunctionMarker):
-                # Magic globals  with __getattr__ might have got us a wrong
-                # fixture attribute.
-                continue
-            if isinstance(obj, FixtureFunctionDefinition):
+            if isinstance(obj_ub, FixtureFunctionDefinition):
+                marker = getfixturemarker(obj_ub)
+                if marker is None:
+                    raise Exception("marker was none")
                 if marker.name:
                     name = marker.name
-                func = obj_ub.get_real_func()
+                func = get_real_func(obj_ub)
                 self._register_fixture(
                     name=name,
                     nodeid=nodeid,

testing/code/test_source.py

@@ -478,9 +478,6 @@ def deco_mark():
     def deco_fixture():
         assert False
 
-    # Since deco_fixture is now an instance of FixtureFunctionDef the getsource function will not work on it.
-    with pytest.raises(TypeError, match=r"FixtureFunctionDefinition"):
-        inspect.getsource(deco_fixture)
     src = inspect.getsource(deco_fixture._get_wrapped_function())
     assert src == "    @pytest.fixture\n    def deco_fixture():\n        assert False\n"
     # Make sure the decorator is not a wrapped function