RobotWebTools / rclnodejs / 23940401301

85%

develop: 85%

Build Type

push

github

Committed by minggangw

Commit Message

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

Run Details

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%)

59 existing lines in 1 file now uncovered.

3177 of 3560 relevant lines covered (89.24%)

445.38 hits per line