Accept suggestions from new toolchain

Author: rec

impall/__init__.py

@@ -150,7 +150,7 @@ def test_all(self) -> None:
         failed_to_fail = [m for m in successes if m in expected]
 
         if failed_to_fail:
-            print('Didn\'t fail when expected:', *sorted(failed_to_fail))
+            print("Didn't fail when expected:", *sorted(failed_to_fail))
 
         first = True
         for module, ex in failed:
@@ -171,7 +171,7 @@ def impall(self) -> t.Tuple[t.List[str], t.List[t.Tuple[str, str]]]:
         failures: t.List[t.Tuple[str, str]] = []
         paths = _split(self.PATHS or path_to_import(os.getcwd())[0])
 
-        warnings.simplefilter(self.WARNINGS_ACTION)  # type: ignore[arg-type]
+        warnings.simplefilter(self.WARNINGS_ACTION)
         for file in self._all_imports(paths):
             self._import(file, successes, failures)
 
@@ -192,7 +192,12 @@ def _all_imports(self, paths: t.Sequence[str]) -> t.Iterator[str]:
                     if f.endswith('.py') and not _is_ignored(f):
                         yield os.path.join(directory, f)
 
-    def _import(self, file: str, successes: t.List[str], failures: t.List[t.Tuple[str, str]]) -> None:
+    def _import(
+        self,
+        file: str,
+        successes: t.List[str],
+        failures: t.List[t.Tuple[str, str]],
+    ) -> None:
         root, module = path_to_import(file)
         path = file[:-3] if file.endswith('.py') else file
 
@@ -230,7 +235,7 @@ def _accept_dir(self, directory: str) -> bool:
         return not _is_ignored(directory)
 
 
-@functools.lru_cache()
+@functools.lru_cache
 def path_to_import(path: str) -> t.Tuple[str, str]:
     """
     Return a (path, module) pair that allows you to import the Python file or
@@ -332,7 +337,7 @@ def report() -> None:
         print()
 
     if failures:
-        fail = ['%s (%s)' % (m, e) for (m, e) in failures]
+        fail = [f'{m} ({e})' for (m, e) in failures]
         print('Failures', *fail, sep='\n  ', file=sys.stderr)
         print(file=sys.stderr)
 

test/test_impall.py

@@ -1,8 +1,9 @@
-import impall
 import os
 import pathlib
 import unittest
 
+import impall
+
 
 class PropertiesTest(impall.ImpAllTest):
     PATHS = str(pathlib.Path(__file__).parent / 'edge' / 'edge')
@@ -41,7 +42,7 @@ def test_one(self):
         assert actual == ('test/edge', 'edge.yes')
 
     def test_two(self):
-        assert impall.path_to_import('impall.py') == ('', 'impall')
+        assert impall.path_to_import('impall/__init__.py') == ('', 'impall.__init__')
 
     def test_three(self):
         root = 'test/edge/deep/one/two/three/four/five/six.py'