fix: item_dict returns empty stanzas

This fix is a result of migrating from splunk-appinspect to this library in pytest-splunk-addon.

Reference PR: splunk/pytest-splunk-addon#464

Closes #36.

Author: artemrys

addonfactory_splunk_conf_parser_lib.py

@@ -227,6 +227,5 @@ def item_dict(self):
                 ):
                     continue
                 kv[k] = v
-            if kv:
-                res[section] = kv
+            res[section] = kv
         return res

test_addonfactory_splunk_conf_parser_lib.py

@@ -163,6 +163,41 @@ def test_item_dict(self):
         }
         self.assertEqual(expected_item_dict, parser.item_dict())
 
+    def test_item_dict_when_there_is_empty_stanza(self):
+        conf = """
+[stanza_with_something]
+key = value
+
+[empty_stanza]
+"""
+        parser = conf_parser.TABConfigParser()
+        parser.read_string(conf)
+        expected_item_dict = {
+            "stanza_with_something": {
+                "key": "value",
+            },
+            "empty_stanza": {},
+        }
+        self.assertEqual(expected_item_dict, parser.item_dict())
+
+    def test_item_dict_when_there_is_empty_stanza_and_comment(self):
+        conf = """
+[stanza_with_something]
+key = value
+
+[empty_stanza]
+# This is a comment
+"""
+        parser = conf_parser.TABConfigParser()
+        parser.read_string(conf)
+        expected_item_dict = {
+            "stanza_with_something": {
+                "key": "value",
+            },
+            "empty_stanza": {},
+        }
+        self.assertEqual(expected_item_dict, parser.item_dict())
+
     def test_remove_existing_section(self):
         conf = """
 # Comment