[Fix] Merging with wrong format now raises Validation error #351

netjsonconfig/utils.py

@@ -22,9 +22,8 @@ def merge_config(template, config, list_identifiers=None):
     """
     result = deepcopy(template)
     for key, value in config.items():
-        if isinstance(value, dict):
-            node = result.get(key, OrderedDict())
-            result[key] = merge_config(node, value)
+        if isinstance(value, dict) and isinstance(result.get(key), dict):
+            result[key] = merge_config(result.get(key), value, list_identifiers)
         elif isinstance(value, list) and isinstance(result.get(key), list):
             result[key] = merge_list(result[key], value, list_identifiers)
         else:

tests/openwrt/test_default.py

@@ -251,3 +251,28 @@ def test_render_invalid_uci_name(self):
 """
         )
         self.assertEqual(o.render(), expected)
+
+    def test_merge_invalid_format(self):
+        invalid = {
+            "dhcp": {
+                "lan": {
+                    "interface": "lan",
+                    "start": 100,
+                    "limit": 150,
+                    "leasetime": "12h",
+                }
+            }
+        }
+        valid = {
+            "dhcp": [
+                {
+                    "dhcpv6": "disabled",
+                    "ignore": True,
+                    "ra": "disabled",
+                    "config_value": "lan",
+                    "config_name": "dhcp",
+                }
+            ]
+        }
+        o = OpenWrt({}, templates=[valid, invalid])
+        o.validate()