Handle LVList parameter binding and test LVList type errors

Author: puddly

tests/test_commands.py

@@ -192,6 +192,21 @@ def test_command_param_binding():
     )
     assert isinstance(cmd.Value, t.ShortBytes)
 
+    # Lists are converted to typed LVLists
+    c.UtilCommands.BindAddEntry.Req(
+        DstAddrModeAddr=t.AddrModeAddress(mode=t.AddrMode.NWK, address=0x1234),
+        DstEndpoint=0x56,
+        ClusterIdList=[0x12, 0x45],
+    )
+
+    # Type errors within containers are also caught
+    with pytest.raises(ValueError):
+        c.UtilCommands.BindAddEntry.Req(
+            DstAddrModeAddr=t.AddrModeAddress(mode=t.AddrMode.NWK, address=0x1234),
+            DstEndpoint=0x56,
+            ClusterIdList=[0x12, 0x457890],  # 0x457890 doesn't fit into a uint8_t
+        )
+
 
 def test_command_immutability():
     command1 = c.SysCommands.NVWrite.Req(

tests/test_types_basic.py

@@ -111,6 +111,9 @@ def test_lvlist():
 
     assert isinstance(d, t.LVList(t.uint8_t))
 
+    with pytest.raises(OverflowError):
+        t.LVList(t.uint8_t)([1, 2, 0xFFFF, 4]).serialize()
+
 
 def test_lvlist_too_short():
     with pytest.raises(ValueError):

zigpy_znp/commands/types.py

@@ -318,6 +318,8 @@ def __init__(self, *, partial=False, **params):
                     param.type, (t.ShortBytes, t.LongBytes)
                 ):
                     value = param.type(value)
+                elif isinstance(value, list) and issubclass(param.type, list):
+                    value = param.type(value)
                 else:
                     raise ValueError(
                         f"In {type(self)}, param {param.name} is "