Suppress OSErrors when checking for script presence. Fixes #70.

jaraco · Apr 15, 2023 · 2057380 · 2057380
1 parent 30cf35c
commit 2057380
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/CHANGES.rst b/CHANGES.rst
@@ -1,3 +1,9 @@
+v10.0.7
+=======
+
+#70: Avoid OSError when a parameter to Python exceeds the
+allowed filename length.
+
 v10.0.6
 =======
 

diff --git a/pip_run/scripts.py b/pip_run/scripts.py
@@ -69,7 +69,8 @@ def search(cls, params):
         in a script indicated in the parameters. Only honor the
         first file found.
         """
-        files = filter(pathlib.Path.is_file, map(pathlib.Path, params))
+        safe_is_file = suppress(OSError)(pathlib.Path.is_file)
+        files = filter(safe_is_file, map(pathlib.Path, params))
         return cls.try_read(next(files, None)).params()
 
     def read(self):

diff --git a/tests/test_scripts.py b/tests/test_scripts.py
@@ -102,7 +102,6 @@ def test_comment_style(self):
         reqs = scripts.DepsReader(script).read()
         assert reqs == ['foo==3.1']
 
-    @pytest.mark.xfail(reason="#70")
     def test_search_long_parameter(self):
         """
         A parameter that is too long to be a filename should not fail.