RobotWebTools / rclnodejs / 23940327136

85%

push

github

Add ParameterEventHandler node filtering support (#1474)

- add `ParameterEventHandler.configureNodesFilter(nodeNames?)` to apply or clear `/parameter_events` subscription content filters for selected nodes
- resolve relative node names against the handler node namespace
- reuse `Node.getFullyQualifiedName()` for handler node fully qualified name resolution instead of duplicating that logic
- add TypeScript declarations for `configureNodesFilter()`
- add focused `ParameterEventHandler` tests for:
  - absolute node filters
  - relative node name resolution
  - clearing filters when `nodeNames` is omitted
  - clearing filters when `nodeNames` is empty
  - invalid `nodeNames` validation

Testing:
- `npx mocha test/test-parameter-event-handler.js`
- `npx tsd`

Fix: #1473

1565 of 1992 branches covered (78.56%)

Branch coverage included in aggregate %.

31 of 32 new or added lines in 1 file covered. (96.88%)

3177 of 3560 relevant lines covered (89.24%)

445.48 hits per line