fix(setup.cfg): silently ignore invalid sections to avoid exceptions

See #69 for a permanent solution.

nitpick/files/setup_cfg.py

@@ -1,7 +1,7 @@
 """Checker for the `setup.cfg <https://docs.python.org/3/distutils/configfile.html>` config file."""
 from configparser import ConfigParser
 from io import StringIO
-from typing import Any, List, Set, Tuple
+from typing import Any, Dict, List, Set, Tuple
 
 import dictdiffer
 
@@ -37,10 +37,12 @@ def get_missing_output(self, actual_sections: Set[str] = None) -> str:
         if self.missing_sections:
             missing_cfg = ConfigParser()
             for section in sorted(self.missing_sections):
-                # TODO this invalid configuration raises AttributeError: 'list' object has no attribute 'items':
-                # ["setup.cfg"]
-                # comma_separated_values = ["flake8.ignore", "flake8.exclude"]
-                missing_cfg[section] = self.file_dict[section]
+                expected_config = self.file_dict[section]  # type: Dict
+                if not isinstance(expected_config, dict):
+                    # Silently ignore invalid sections for now, to avoid exceptions.
+                    # This should be solved in https://github.com/andreoliwa/nitpick/issues/69
+                    continue
+                missing_cfg[section] = expected_config
             return self.get_example_cfg(missing_cfg)
         return ""
 

tests/test_setup_cfg.py

@@ -133,3 +133,28 @@ def test_different_missing_keys(request):
         max-line-length = 112\x1b[0m
         """
     )
+
+
+def test_invalid_configuration_comma_separated_values(request):
+    """Test an invalid configuration for comma_separated_values."""
+    ProjectMock(request).style(
+        """
+        ["setup.cfg".flake8]
+        max-line-length = 85
+        max-complexity = 12
+        ignore = "D100,D101,D102,D103,D104,D105,D106,D107,D202,E203,W503"
+        select = "E241,C,E,F,W,B,B9"
+
+        ["setup.cfg"]
+        comma_separated_values = ["flake8.ignore", "flake8.exclude"]
+        """
+    ).lint().assert_errors_contain(
+        """
+        NIP321 File setup.cfg was not found. Create it with this content:\x1b[92m
+        [flake8]
+        ignore = D100,D101,D102,D103,D104,D105,D106,D107,D202,E203,W503
+        max-complexity = 12
+        max-line-length = 85
+        select = E241,C,E,F,W,B,B9\x1b[0m
+        """
+    )