MAINT: rm dead code, reformat

Author: ev-br

scpdt/plugin.py

@@ -17,10 +17,6 @@
 from scpdt.frontend import find_doctests
 
 
-# XXX: unused global?
-copied_files = []
-
-
 def pytest_configure(config):
     """
     Perform initial configuration for the pytest plugin.
@@ -34,20 +30,6 @@ def pytest_configure(config):
     pydoctest.DoctestTextfile = DTTextfile
 
 
-def pytest_unconfigure(config):
-    """
-    Called before exiting the test process.
-    """
-
-    # Delete all locally copied files in the current working directory
-    if copied_files:
-        try:
-            for filepath in copied_files:
-                os.remove(filepath)
-        except FileNotFoundError:
-            pass
-
-
 def pytest_ignore_collect(collection_path, config):
     """
     Determine whether to ignore the specified collection path.
@@ -106,32 +88,6 @@ def pytest_collection_modifyitems(config, items):
 
         # Replace the original list of test items with the unique ones
         items[:] = unique_items
-    
-
-# XXX: unused?
-def copy_local_files(local_resources, destination_dir):
-    """
-    Copy necessary local files for doctests to the current working directory.
-    
-    This function copies files specified in the `local_resources` attribute of a DTConfig instance
-    to the specified `destination_dir`.
-    
-    Args:
-        local_resources (dict): A dictionary of resources to be copied.
-        destination_dir (str): The destination directory where files will be copied.
-    
-    Returns:
-        list: A list of paths to the copied files.
-    """
-    for value in local_resources.values():
-        for filepath in value:
-            basename = os.path.basename(filepath)
-            dest_path = os.path.join(destination_dir, basename)
-
-            if not os.path.exists(dest_path):
-                shutil.copy(filepath, destination_dir)
-                copied_files.append(dest_path)    
-    return copied_files
 
 
 class DTModule(DoctestModule):
@@ -166,10 +122,6 @@ def collect(self):
                 else:
                     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())
-
         optionflags = dt_config.optionflags
 
         # Plug in the custom runner: `PytestDTRunner` 
@@ -210,10 +162,6 @@ def collect(self):
 
         optionflags = pydoctest.get_optionflags(self)
 
-        # 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())
-
         # Plug in the custom runner: `PytestDTRunner`
         runner = _get_runner(self.config,
             verbose=False,

scpdt/tests/local_file_cases.py

@@ -1,15 +1,15 @@
 from scpdt.conftest import dt_config
 
-
 # Specify local files required by doctests
-dt_config.local_resources = {'scpdt.tests.local_file_cases.local_files':
-                                  ['local_file.txt'],
-                            'scpdt.tests.local_file_cases.sio':
-                                  ['octave_a.mat']   
-                                  }
+dt_config.local_resources = {
+    'scpdt.tests.local_file_cases.local_files': ['local_file.txt'],
+    'scpdt.tests.local_file_cases.sio': ['octave_a.mat']
+}
+
 
 __all__ = ['local_files', 'sio']
 
+
 def local_files():
     """
     A doctest that tries to read a local file

scpdt/tests/test_pytest_configuration.py

@@ -1,42 +1,14 @@
 import pytest
-import os
 from pathlib import PosixPath, Path
 
 from . import module_cases, failure_cases, failure_cases_2, stopwords_cases, local_file_cases
-from scpdt.plugin import copy_local_files
 from scpdt.conftest import dt_config
 
-pytest_plugins = ['pytester']
-
-'''
-@pytest.fixture(autouse=True)
-def copy_files():
-    """
-    Copy necessary local files for doctests to the temporary directory used by pytester. 
-    The files to be copied are defined by the `local_resources` attribute of a DTConfig instance.
-    When testing is done, all copied files are deleted.
-    """
-    try:
-        dirname = os.path.dirname(Path(__file__))
-
-        # Update the file path of each filename
-        for value in dt_config.local_resources.values():
-            for i, path in enumerate(value):
-                value[i] = os.path.join(dirname, os.path.basename(path))
+# XXX: this is a bit hacky and repetetive. Can rework?
 
-        # Copy the files
-        copied_files = copy_local_files(dt_config.local_resources, os.getcwd())
 
-        yield copied_files
+pytest_plugins = ['pytester']
 
-    finally:
-        # Perform clean-up
-        for filepath in copied_files:
-            try:
-                os.remove(filepath)
-            except FileNotFoundError:
-                pass
-'''
 
 def test_module_cases(pytester):
     """Test that pytest uses the DTChecker for doctests."""
@@ -63,27 +35,10 @@ def test_stopword_cases(pytester):
     assert result.ret == pytest.ExitCode.OK
 
 
-#@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