Apply suggestions from code review

Author: nicoddemus

src/_pytest/fixtures.py

@@ -1725,7 +1725,7 @@ def parsefactories(
         if holderobj in self._holderobjseen:
             return
 
-        # avoid accessing `@property` (and other descriptors) when iterating fixtures
+        # Avoid accessing `@property` (and other descriptors) when iterating fixtures.
         if not safe_isclass(holderobj) and not isinstance(holderobj, types.ModuleType):
             holderobj_tp: object = type(holderobj)
         else:
@@ -1742,7 +1742,7 @@ def parsefactories(
                 # fixture attribute.
                 continue
 
-            # ok we know it is a fixture -- now safe to look up on the _instance_
+            # OK we know it is a fixture -- now safe to look up on the _instance_.
             obj = getattr(holderobj, name)
 
             if marker.name:

testing/python/collect.py

@@ -264,6 +264,7 @@ def prop(self):
         assert result.ret == ExitCode.NO_TESTS_COLLECTED
 
     def test_does_not_discover_properties(self, pytester: Pytester) -> None:
+        """Regression test for #12446."""
         pytester.makepyfile(
             """\
             class TestCase:
@@ -276,6 +277,7 @@ def oops(self):
         assert result.ret == ExitCode.NO_TESTS_COLLECTED
 
     def test_does_not_discover_instance_descriptors(self, pytester: Pytester) -> None:
+        """Regression test for #12446."""    
         pytester.makepyfile(
             """\
             # not `@property`, but it acts like one