Merge pull request #118 from ev-br/ruff

MAINT: fix (some) style violations

Author: ev-br

scpdt/__init__.py

@@ -7,6 +7,6 @@
 
 __version__ = "0.1"
 
-from .impl import DTChecker, DTFinder, DTParser, DTRunner, DebugDTRunner, DTConfig
-from .frontend import testmod, testfile, find_doctests, run_docstring_examples
+from .impl import DTChecker, DTFinder, DTParser, DTRunner, DebugDTRunner, DTConfig  # noqa
+from .frontend import testmod, testfile, find_doctests, run_docstring_examples      # noqa
 

scpdt/frontend.py

@@ -229,7 +229,9 @@ def testmod(m=None, name=None, globs=None, verbose=None,
         runner = DTRunner(verbose=dtverbose, optionflags=flags, config=config)
 
     ### Find, parse, and run all tests in the given module.
-    tests = find_doctests(m, strategy, name, exclude_empty, globs, extraglobs, config=config)
+    tests = find_doctests(
+        m, strategy, name, exclude_empty, globs, extraglobs, config=config
+    )
 
     for test in tests:
         if verbose == 1:
@@ -356,9 +358,13 @@ def testfile(filename, module_relative=True, name=None, package=None,
     # Fail fast or run all tests
     verbose, dtverbose = _map_verbosity(verbose)
     if raise_on_error:
-        runner = DebugDTRunner(verbose=dtverbose, optionflags=optionflags, config=config)
+        runner = DebugDTRunner(
+            verbose=dtverbose, optionflags=optionflags, config=config
+        )
     else:
-        runner = DTRunner(verbose=dtverbose, optionflags=optionflags, config=config)
+        runner = DTRunner(
+            verbose=dtverbose, optionflags=optionflags, config=config
+        )
 
     ### Parse doctest examples out of the input file and run them.
     if parser is None:

scpdt/impl.py

@@ -386,7 +386,9 @@ class DebugDTRunner(DTRunner):
     Almost verbatim copy of `doctest.DebugRunner`.
     """
     def run(self, test, compileflags=None, out=None, clear_globs=True):
-        r = super().run(test, compileflags=compileflags, out=out, clear_globs=clear_globs)
+        r = super().run(
+            test, compileflags=compileflags, out=out, clear_globs=clear_globs
+        )
         if clear_globs:
             test.globs.clear()
         return r

scpdt/tests/module_cases.py

@@ -1,4 +1,7 @@
-__all__ = ['func', 'func2', 'func3', 'func4', 'func5', 'func6', 'func8', 'func7', 'manip_printoptions', 'array_abbreviation']
+__all__ = [
+    'func', 'func2', 'func3', 'func4', 'func5', 'func6', 'func8',
+    'func7', 'manip_printoptions', 'array_abbreviation'
+]
 
 def func():
     """

scpdt/tests/test_finder.py

@@ -156,7 +156,6 @@ def test_explicit_object_list_with_module():
 
 def test_find_doctests_api():
     # Test that the module itself is included with strategy='api'
-    objs = [finder_cases, finder_cases.Klass]
     tests = find_doctests(finder_cases, strategy='api')
 
     base = 'scpdt.tests.finder_cases'

scpdt/tests/test_parser.py

@@ -1,6 +1,3 @@
-import doctest
-import pytest
-
 from ..impl import DTConfig, DTParser, DebugDTRunner
 
 

scpdt/tests/test_skipmarkers.py

@@ -5,7 +5,9 @@
 
 
 class TestSyntaxErrors:
-    """Syntax errors trigger a doctest failure *unless* marked with +SKIP or as pseudocode.
+    """Syntax errors trigger a doctest failure *unless* marked.
+
+    Either mark it with +SKIP or as pseudocode.
     """
     def test_invalid_python(self):
         # This string raises

scpdt/tests/test_testmod.py

@@ -6,9 +6,9 @@
 import pytest
 
 try:
-    import scipy
+    import scipy    # noqa
     HAVE_SCIPY = True
-except:
+except Exception:
     HAVE_SCIPY = False
 
 from . import (module_cases as module,
@@ -17,7 +17,7 @@
                failure_cases,
                failure_cases_2,
                local_file_cases)
-from ..frontend import testmod as _testmod, find_doctests, run_docstring_examples
+from ..frontend import testmod as _testmod, run_docstring_examples
 from ..util import warnings_errors
 from ..impl import DTConfig
 
@@ -59,7 +59,6 @@ def test_run_docstring_examples():
 
 def test_global_state():
     # Make sure doctesting does not alter the global state, as much as reasonable
-    objs = [module.manip_printoptions]
     opts = np.get_printoptions()
     _testmod(module)
     new_opts = np.get_printoptions()
@@ -68,7 +67,7 @@ def test_global_state():
 
 def test_module_debugrunner():
     with pytest.raises((doctest.UnexpectedException, doctest.DocTestFailure)):
-        res = _testmod(failure_cases, raise_on_error=True)
+        _testmod(failure_cases, raise_on_error=True)
 
 
 def test_verbosity_1():