Apply bluetech suggestions on PR

src/_pytest/python.py

@@ -433,7 +433,9 @@ def _genfunctions(self, name, funcobj):
         self.ihook.pytest_generate_tests.call_extra(methods, dict(metafunc=metafunc))
 
         if not metafunc._calls:
-            yield Function.from_parent(self, name=name, fixtureinfo=fixtureinfo)
+            yield Function.from_parent(
+                self, name=name, fixtureinfo=fixtureinfo, originalname=name
+            )
         else:
             # add funcargs() as fixturedefs to fixtureinfo.arg2fixturedefs
             fixtures.add_funcarg_pseudo_fixture_def(self, metafunc, fm)
@@ -1457,7 +1459,7 @@ def __init__(
         #: parametrization adds a ``"[...]"`` suffix to function names).
         #:
         #: .. versionadded:: 3.0
-        self.originalname = originalname if originalname is not None else name
+        self.originalname = originalname
 
         self.keywords.update(self.obj.__dict__)
         self.own_markers.extend(get_unpacked_marks(self.obj))

testing/python/collect.py

@@ -680,7 +680,7 @@ def test_no_param():
         ]
 
     def test_function_with_square_brackets(self, testdir: Testdir) -> None:
-        """Check that Function._getobj uses originalname."""
+        """Check that functions with square brackets don't cause trouble."""
         p1 = testdir.makepyfile(
             """
             locals()["test_foo[name]"] = lambda: None