Handle invalid .cfg when assessing if they contain a pylint config

pylint-dev · Mar 31, 2022 · bb84699 · bb84699
1 parent 592f133
commit bb84699
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/pylint/config/find_default_config_files.py b/pylint/config/find_default_config_files.py
@@ -27,7 +27,10 @@ def _toml_has_config(path: Union[Path, str]) -> bool:
 
 def _cfg_has_config(path: Union[Path, str]) -> bool:
     parser = configparser.ConfigParser()
-    parser.read(path, encoding="utf-8")
+    try:
+        parser.read(path, encoding="utf-8")
+    except configparser.Error:
+        return False
     return any(section.startswith("pylint.") for section in parser.sections())
 
 

diff --git a/tests/config/test_find_default_config_files.py b/tests/config/test_find_default_config_files.py
@@ -41,6 +41,7 @@ def test_toml_has_config(content: str, expected: bool, tmp_path: Path) -> None:
     "content,expected",
     [
         ["", False],
+        ["(not valid .cfg)", False],
         [
             """
 [metadata]