Merge pull request #13202 from Kojoley/fix-unintentional-skipping-of-module-level-doctests

Fix unintentional skipping of module level doctests

IPython/core/completer.py

@@ -142,7 +142,7 b' from traitlets.config.configurable import Configurable'
 import __main__
 
 # skip module docstests
-skip_doctest = True
+__skip_doctest__ = True
 
 try:
     import jedi

IPython/core/debugger.py

@@ -116,6 +116,8 b' from IPython.utils import coloransi, py3compat'
 from IPython.core.excolors import exception_colors
 from IPython.testing.skipdoctest import skip_doctest
 
+# skip module docstests
+__skip_doctest__ = True
 
 prompt = 'ipdb> '
 

IPython/extensions/autoreload.py

@@ -97,7 +97,7 b' Some of the known remaining caveats are:'
 - C extension modules cannot be reloaded, and so cannot be autoreloaded.
 """
 
-skip_doctest = True
+__skip_doctest__ = True
 
 # -----------------------------------------------------------------------------
 #  Copyright (C) 2000 Thomas Heller

IPython/testing/plugin/ipdoctest.py

@@ -125,7 +125,7 b' class DocTestFinder(doctest.DocTestFinder):'
         add them to `tests`.
         """
         print('_find for:', obj, name, module)  # dbg
-        if hasattr(obj,"skip_doctest"):
+        if bool(getattr(obj, "__skip_doctest__", False)):
             #print 'SKIPPING DOCTEST FOR:',obj  # dbg
             obj = DocTestSkip(obj)
         

IPython/testing/skipdoctest.py

@@ -15,5 +15,5 b' def skip_doctest(f):'
     This decorator allows you to mark a function whose docstring you wish to
     omit from testing, while preserving the docstring for introspection, help,
     etc."""
-    f.skip_doctest = True
+    f.__skip_doctest__ = True
     return f