Rely on pathlib.Path.glob for simplicity.

python · Sep 17, 2024 · 8cd8fe9 · 8cd8fe9
1 parent 211b436
commit 8cd8fe9
Showing 1 changed file with 3 additions and 13 deletions.
diff --git a/Lib/test/test_importlib/resources/test_files.py b/Lib/test/test_importlib/resources/test_files.py
@@ -142,19 +142,9 @@ def _compile_importlib(self):
         c_resources = pathlib.Path(bin_site, 'c_resources')
         sources = pathlib.Path(resources.__file__).parent
 
-        for dirpath, dirnames, filenames in os.walk(sources):
-            try:
-                dirnames.remove('__pycache__')
-            except ValueError:
-                pass
-            source_dir_path = pathlib.Path(dirpath)
-            dir_relpath = pathlib.Path(source_dir_path).relative_to(sources)
-            c_dir_path = c_resources.joinpath(dir_relpath)
-            for filename in filenames:
-                if filename.endswith('.py'):
-                    source_path = source_dir_path / filename
-                    cfile = c_dir_path.joinpath(filename).with_suffix('.pyc')
-                    py_compile.compile(source_path, cfile)
+        for source_path in sources.glob('**/*.py'):
+            c_path = c_resources.joinpath(source_path.relative_to(sources)).with_suffix('.pyc')
+            py_compile.compile(source_path, c_path)
         self.fixtures.enter_context(import_helper.DirsOnSysPath(bin_site))
 
     def test_implicit_files_with_compiled_importlib(self):