scipy-doctest 1.5__tar.gz → 1.6__tar.gz

{scipy_doctest-1.5 → scipy_doctest-1.6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: scipy_doctest
-Version: 1.5
+Version: 1.6
 Summary: Configurable, whitespace-insensitive, floating-point-aware doctest helpers.
 Maintainer-email: SciPy developers <scipy-dev@python.org>
 Requires-Python: >=3.8

{scipy_doctest-1.5 → scipy_doctest-1.6}/scipy_doctest/__init__.py

@@ -3,7 +3,7 @@ Configurable, whitespace-insensitive, floating-point-aware doctest helpers.
 """
 
 
-__version__ = "1.5"
+__version__ = "1.6"
 
 try:
     # register internal modules with pytest; obscure errors galore otherwise

{scipy_doctest-1.5 → scipy_doctest-1.6}/scipy_doctest/frontend.py

@@ -90,14 +90,24 @@ def find_doctests(module, strategy=None,
     # Having collected the list of objects, extract doctests
     tests = []
     for item, name in zip(items, names):
-        full_name = module.__name__ + '.' + name
         if inspect.ismodule(item):
             # do not recurse, only inspect the module docstring
             _finder = DTFinder(recurse=False, config=config)
             t = _finder.find(item, name, globs=globs, extraglobs=extraglobs)
+            unique_t = set(t)
         else:
+            full_name = module.__name__ + '.' + name
             t = finder.find(item, full_name, globs=globs, extraglobs=extraglobs)
-        tests += t
+
+            unique_t = set(t)
+            if hasattr(item, '__mro__'):
+                # is a class, inspect superclasses
+                # cf https://github.com/scipy/scipy_doctest/issues/177
+                # item.__mro__ starts with itself, ends with `object`
+                for item_ in item.__mro__[1:-1]:
+                    t_ = finder.find(item_, full_name, globs=globs, extraglobs=extraglobs)
+                    unique_t.update(set(t_))
+        tests += list(unique_t)
 
     # If the skiplist contains methods of objects, their doctests may have been
     # left in the `tests` list. Remove them.

{scipy_doctest-1.5 → scipy_doctest-1.6}/scipy_doctest/impl.py

@@ -1,5 +1,6 @@
 import re
 import warnings
+import inspect
 import doctest
 from doctest import NORMALIZE_WHITESPACE, ELLIPSIS, IGNORE_EXCEPTION_DETAIL
 from itertools import zip_longest
@@ -538,6 +539,13 @@ class DTFinder(doctest.DocTestFinder):
         # XXX: does this make similar checks in testmod/testfile duplicate?
         if module not in self.config.skiplist:   
             tests = super().find(obj, name, module, globs, extraglobs)
+
+            if inspect.isclass(obj):
+                for name_, method in inspect.getmembers(obj):
+                    if inspect.isdatadescriptor(method):
+                        tests += super().find(
+                            method, f'{name}.{name_}', module, globs, extraglobs
+                        )
             return tests
 
 

scipy_doctest-1.6/scipy_doctest/tests/finder_cases_2.py

@@ -0,0 +1,22 @@
+"""
+Private method in subclasses
+"""
+
+__all__ = ["Klass"]
+
+class _PrivateKlass:
+    def private_method(self):
+        """
+        >>> 2 / 3
+        0.667
+        """
+        pass
+
+
+class Klass(_PrivateKlass):
+    def public_method(self):
+        """
+        >>> 3 / 4
+        0.74
+        """
+        pass

{scipy_doctest-1.5 → scipy_doctest-1.6}/scipy_doctest/tests/test_finder.py

@@ -1,10 +1,14 @@
 import pytest
 
+import numpy as np
+
 from . import finder_cases
+from . import finder_cases_2
 from ..util import get_all_list, get_public_objects
 from ..impl import DTFinder, DTConfig
 from ..frontend import find_doctests
 
+
 def test_get_all_list():
     items, depr, other = get_all_list(finder_cases)
     assert sorted(items) == ['Klass', 'func']
@@ -111,6 +115,7 @@ class TestSkiplist:
         #   - *private* stuff, which is not in `__all__`
         wanted_names = ['Klass', 'Klass.meth']
         wanted_names = [base] + [base + '.' + n for n in wanted_names]
+        wanted_names += [f'{base}.Klass.__weakref__']
 
         assert sorted(names) == sorted(wanted_names)
 
@@ -129,6 +134,7 @@ class TestSkiplist:
         #   - the 'base' module (via the strategy=<list>)
         wanted_names = ['Klass', 'Klass.meth']
         wanted_names = [base + '.' + n for n in wanted_names]
+        wanted_names += [f'{base}.Klass.__weakref__']
 
         assert sorted(names) == sorted(wanted_names)
 
@@ -137,9 +143,13 @@ def test_explicit_object_list():
     objs = [finder_cases.Klass]
     tests = find_doctests(finder_cases, strategy=objs)
 
+    names = sorted([test.name for test in tests])
+
     base = 'scipy_doctest.tests.finder_cases'
-    assert ([test.name for test in tests] ==
-            [base + '.Klass', base + '.Klass.meth', base + '.Klass.meth_2'])
+    expected = sorted([f'{base}.Klass', f'{base}.Klass.__weakref__',
+                       f'{base}.Klass.meth', f'{base}.Klass.meth_2',])
+
+    assert names == expected
 
 
 def test_explicit_object_list_with_module():
@@ -149,22 +159,64 @@ def test_explicit_object_list_with_module():
     objs = [finder_cases, finder_cases.Klass]
     tests = find_doctests(finder_cases, strategy=objs)
 
+    names = sorted([test.name for test in tests])
+
     base = 'scipy_doctest.tests.finder_cases'
-    assert ([test.name for test in tests] ==
-            [base, base + '.Klass', base + '.Klass.meth', base + '.Klass.meth_2'])
+    expected = sorted([base, f'{base}.Klass', f'{base}.Klass.__weakref__',
+                       f'{base}.Klass.meth', f'{base}.Klass.meth_2'])
+
+    assert names == expected
 
 
 def test_find_doctests_api():
     # Test that the module itself is included with strategy='api'
     tests = find_doctests(finder_cases, strategy='api')
 
+    names = sorted([test.name for test in tests])
+
     base = 'scipy_doctest.tests.finder_cases'
-    assert ([test.name for test in tests] ==
-            [base + '.func', base + '.Klass', base + '.Klass.meth',
-             base + '.Klass.meth_2', base])
+    expected = sorted([base + '.func', base + '.Klass', base + '.Klass.meth',
+             base + '.Klass.meth_2', base + '.Klass.__weakref__', base])
+
+    assert names == expected
 
 
 def test_dtfinder_config():
     config = DTConfig()
     finder = DTFinder(config=config)
     assert finder.config is config
+
+
+@pytest.mark.skipif(np.__version__ < '2', reason="XXX check if works on numpy 1.x")
+def test_descriptors_get_collected():
+    tests = find_doctests(np, strategy=[np.dtype])
+    names = [test.name for test in tests]
+    assert 'numpy.dtype.kind' in names   # was previously missing
+
+
+@pytest.mark.parametrize('strategy', [None, 'api'])
+def test_private_superclasses(strategy):
+    # Test that methods from inherited private superclasses get collected
+    tests = find_doctests(finder_cases_2, strategy=strategy)
+
+    names = set(test.name.split('.')[-1] for test in tests)
+    expected_names = ['finder_cases_2', 'public_method', 'private_method']
+    if strategy == 'api':
+        expected_names += ['__weakref__']
+
+    assert len(tests) == len(expected_names)
+    assert names == set(expected_names)
+
+
+def test_private_superclasses_2():
+    # similar to test_private_superclass, only with an explicit strategy=list
+    tests = find_doctests(finder_cases_2, strategy=[finder_cases_2.Klass])
+
+    names = set(test.name.split('.')[-1] for test in tests)
+    expected_names = ['public_method', 'private_method', '__weakref__']
+
+    assert len(tests) == len(expected_names)
+    assert names == set(expected_names)
+
+
+