Fix line detection for properties in doctest tests

Grammar fix

Co-Authored-By: Daniel Hahler <[email protected]>

Add python issue link

Explicit check for the property

Test when lineno is not detected

Fix changelog entry

Author: Nikolay Kondratyev

changelog/6082.bugfix.rst

@@ -0,0 +1 @@
+Fix line detection for doctest samples inside ``property`` docstrings, as workaround to `bpo-17446 <https://bugs.python.org/issue17446>`__.

src/_pytest/doctest.py

@@ -401,6 +401,16 @@ class MockAwareDocTestFinder(doctest.DocTestFinder):
             https://bugs.python.org/issue25532
             """
 
+            def _find_lineno(self, obj, source_lines):
+                """
+                Doctest code does not take into account @property, hackish way to fix it.
+
+                https://bugs.python.org/issue17446
+                """
+                if isinstance(obj, property):
+                    obj = getattr(obj, "fget", obj)
+                return doctest.DocTestFinder._find_lineno(self, obj, source_lines)
+
             def _find(self, tests, obj, name, module, source_lines, globs, seen):
                 if _is_mocked(obj):
                     return

testing/test_doctest.py

@@ -287,13 +287,50 @@ def test(self):
             )
         )
         result = testdir.runpytest("--doctest-modules")
+        result.stdout.fnmatch_lines(
+            ["*hello*", "006*>>> 1/0*", "*UNEXPECTED*ZeroDivision*", "*1 failed*"]
+        )
+
+    def test_doctest_linedata_on_property(self, testdir):
+        testdir.tmpdir.join("hello.py").write(
+            textwrap.dedent(
+                """\
+                class Sample(object):
+                    @property
+                    def some_property(self):
+                        '''
+                        >>> Sample().some_property
+                        'another thing'
+                        '''
+                        return 'something'
+                """
+            )
+        )
+        result = testdir.runpytest("--doctest-modules")
+        result.stdout.fnmatch_lines(["*hello*", "005*>>> Sample().some_property"])
+
+    def test_doctest_no_linedata_on_overriden_property(self, testdir):
+        testdir.tmpdir.join("hello.py").write(
+            textwrap.dedent(
+                """\
+                class Sample(object):
+                    @property
+                    def some_property(self):
+                        '''
+                        >>> Sample().some_property
+                        'another thing'
+                        '''
+                        return 'something'
+                    some_property = property(some_property.__get__, None, None, some_property.__doc__)
+                """
+            )
+        )
+        result = testdir.runpytest("--doctest-modules")
         result.stdout.fnmatch_lines(
             [
                 "*hello*",
                 "*EXAMPLE LOCATION UNKNOWN, not showing all tests of that example*",
-                "*1/0*",
-                "*UNEXPECTED*ZeroDivision*",
-                "*1 failed*",
+                "???*>>> Sample().some_property",
             ]
         )