fix: throw meaningful error message for wrong dtype of repeated dict (#81)

Fix #80.

---------

Signed-off-by: Jinzhe Zeng <[email protected]>

Author: njzjz

dargs/dargs.py

@@ -391,9 +391,14 @@ def _traverse_sub(
         variant_hook: HookVrntType = _DUMMYHOOK,
         path: list[str] | None = None,
     ):
-        assert isinstance(value, dict)
         if path is None:
             path = [self.name]
+        if not isinstance(value, dict):
+            raise ArgumentTypeError(
+                path,
+                f"key `{path[-1]}` gets wrong value type, "
+                f"requires dict but {type(value).__name__} is given",
+            )
         sub_hook(self, value, path)
         for subvrnt in self.sub_variants.values():
             variant_hook(subvrnt, value, path)

tests/test_checker.py

@@ -156,6 +156,14 @@ def test_sub_repeat_dict(self):
         }
         with self.assertRaises(ArgumentTypeError):
             ca.check(err_dict2)
+        err_dict3 = {
+            "base": {
+                "item1": {"sub1": 10, "sub2": "hello"},
+                "item2": "not_a_dict_error",
+            }
+        }
+        with self.assertRaises(ArgumentTypeError):
+            ca.check(err_dict3)
 
     def test_sub_variants(self):
         ca = Argument(