MAINT: plugin: run tests in a tempdir, stop blank filtering DeprecationWarnings

Author: ev-br

scpdt/plugin.py

@@ -13,10 +13,11 @@
 
 from scpdt.impl import DTChecker, DTParser, DebugDTRunner
 from scpdt.conftest import dt_config
-from scpdt.util import np_errstate, matplotlib_make_nongui
+from scpdt.util import np_errstate, matplotlib_make_nongui, temp_cwd
 from scpdt.frontend import find_doctests
 
 
+# XXX: unused global?
 copied_files = []
 
 
@@ -107,6 +108,7 @@ def pytest_collection_modifyitems(config, items):
         items[:] = unique_items
 
 
+# XXX: unused?
 def copy_local_files(local_resources, destination_dir):
     """
     Copy necessary local files for doctests to the current working directory.
@@ -165,8 +167,8 @@ def collect(self):
                     raise
 
         # Copy local files specified by the `local_resources` attribute to the current working directory
-        if self.config.dt_config.local_resources:
-            copy_local_files(self.config.dt_config.local_resources, os.getcwd())
+    #    if self.config.dt_config.local_resources:
+    #        copy_local_files(self.config.dt_config.local_resources, os.getcwd())
 
         optionflags = dt_config.optionflags
 
@@ -254,10 +256,15 @@ def run(self, test, compileflags=None, out=None, clear_globs=False):
             *unless* the `mpl()` context mgr has a chance to filter them out
             *before* they become errors in `config.user_context_mgr()`.
             """
+            dt_config = config.dt_config
+
+
             with np_errstate():
-                with config.dt_config.user_context_mgr(test):
+                with dt_config.user_context_mgr(test):
                     with matplotlib_make_nongui():
-                        super().run(test, compileflags=compileflags, out=out, clear_globs=clear_globs)
+                        # XXX: local_resourses needed? they seem to be, w/o pytest
+                        with temp_cwd(test, dt_config.local_resources):
+                            super().run(test, compileflags=compileflags, out=out, clear_globs=clear_globs)
 
         """
         Almost verbatim copy of `_pytest.doctest.PytestDoctestRunner` except we utilize

scpdt/tests/test_pytest_configuration.py

@@ -8,7 +8,7 @@
 
 pytest_plugins = ['pytester']
 
-
+'''
 @pytest.fixture(autouse=True)
 def copy_files():
     """
@@ -36,13 +36,10 @@ def copy_files():
                 os.remove(filepath)
             except FileNotFoundError:
                 pass
-
-
-"""
-Test that pytest uses the DTChecker for doctests
-"""
+'''
 
 def test_module_cases(pytester):
+    """Test that pytest uses the DTChecker for doctests."""
     path_str = module_cases.__file__
     python_file = PosixPath(path_str)
     result = pytester.inline_run(python_file, "--doctest-modules")
@@ -58,21 +55,35 @@ def test_failure_cases(pytester):
     assert result.ret == pytest.ExitCode.TESTS_FAILED
 
 
-"""
-Test that pytest uses the DTParser for doctests
-"""
 def test_stopword_cases(pytester):
+    """Test that pytest uses the DTParser for doctests."""
     path_str = stopwords_cases.__file__
     python_file = PosixPath(path_str)
     result = pytester.inline_run(python_file, "--doctest-modules")
     assert result.ret == pytest.ExitCode.OK
 
 
-"""
-Test that local files are found for use in doctests
-"""
+#@pytest.mark.xfail
 def test_local_file_cases(pytester):
+    """Test that local files are found for use in doctests.
+
+    XXX: this one fails because nobody told pytest how to find those local files.
+    cf test_testmod.py::TestLocalFiles
+    """
     path_str = local_file_cases.__file__
     python_file = PosixPath(path_str)
+
+  #  pytester.makeconftest(
+  #      """
+  #      import pytest
+  #      from scpdt.conftest import dt_config
+  #
+  #      dt_config.local_resources = {'scpdt.tests.local_file_cases.local_files':
+  #                                   ['local_file.txt']}
+  #      """
+  #  )
+
+  #  breakpoint()
+
     result = pytester.inline_run(python_file, "--doctest-modules")
     assert result.ret == pytest.ExitCode.OK

scpdt/util.py

@@ -58,7 +58,7 @@ def temp_cwd(test, local_resources=None):
         # local files requested; copy the files
         path, _ = os.path.split(test.filename)
         for fname in local_resources[test.name]:
-            shutil.copy(os.path.join(path, fname), tmpdir)      
+            shutil.copy(os.path.join(path, fname), tmpdir)
     try:
         os.chdir(tmpdir)
         yield tmpdir