Problem
Only ~5 of ~40+ WS commands have integration test coverage. Most handlers are untested end-to-end:
device_command, read_attribute (beyond basic), ping_node
get_matter_fabrics, remove_matter_fabric
set_acl_entry, set_node_binding
check_node_update, update_node
discover_commissionable_nodes
set_default_fabric_label
get_loglevel, set_loglevel
- Error cases (
InvalidArguments, NodeNotReady, NodeNotExists)
The existing integration tests also have pre-existing failures on Windows (BigInt conversion issue cascading from commissioning).
Proposal
Systematically expand IntegrationTest.ts to cover all command handlers, including both success and error paths.
Labels: testing, enhancement
Problem
Only ~5 of ~40+ WS commands have integration test coverage. Most handlers are untested end-to-end:
device_command,read_attribute(beyond basic),ping_nodeget_matter_fabrics,remove_matter_fabricset_acl_entry,set_node_bindingcheck_node_update,update_nodediscover_commissionable_nodesset_default_fabric_labelget_loglevel,set_loglevelInvalidArguments,NodeNotReady,NodeNotExists)The existing integration tests also have pre-existing failures on Windows (BigInt conversion issue cascading from commissioning).
Proposal
Systematically expand
IntegrationTest.tsto cover all command handlers, including both success and error paths.Labels: testing, enhancement