Avoid bailing out with uncaught PermissionError

Author: DimitriPapadopoulos

codespell_lib/_codespell.py

@@ -647,14 +647,26 @@ def parse_file(filename, colors, summary, misspellings, exclude_lines,
         if not os.path.isfile(filename):
             return bad_count
 
-        text = is_text_file(filename)
+        try:
+            text = is_text_file(filename)
+        except PermissionError as e:
+            print("WARNING: %s: %s" % (e.strerror, filename),
+                  file=sys.stderr)
+            return bad_count
+        except OSError:
+            return bad_count
+
         if not text:
             if not options.quiet_level & QuietLevels.BINARY_FILE:
                 print("WARNING: Binary file: %s" % filename, file=sys.stderr)
             return bad_count
         try:
             lines, encoding = file_opener.open(filename)
-        except Exception:
+        except PermissionError as e:
+            print("WARNING: %s: %s" % (e.strerror, filename),
+                  file=sys.stderr)
+            return bad_count
+        except OSError:
             return bad_count
 
     for i, line in enumerate(lines):

codespell_lib/tests/test_basic.py

@@ -117,6 +117,13 @@ def test_basic(tmpdir, capsys):
     assert stdout == stderr == ''
     assert cs.main(d) == 0
 
+    # unreadable file
+    if sys.platform == 'linux':  # cannot create unreadable file on Windows
+        with open(op.join(d, 'unreadable.txt'), 'w') as f:
+            os.chmod(f, 0o000)
+        code, _, stderr = cs.main(f.name, std=True)
+        assert 'WARNING:' in stderr
+
     # empty directory
     os.mkdir(op.join(d, 'test'))
     assert cs.main(d) == 0