RobotWebTools / rclnodejs / 21021718233

86%

push

github

Correct the type of return value for Time.toMsg() (#1371)

This pull request corrects the return type of the `Time.toMsg()` method to return JavaScript numbers instead of bigints for the `sec` and `nanosec` fields, aligning with ROS2's builtin_interfaces.msg.Time message specification which uses int32 for sec and uint32 for nanosec.

**Changes:**
- Modified `Time.toMsg()` to convert bigint values to numbers using `Number()`
- Updated test expectations to verify numbers are returned instead of bigints
- Added test case verifying `Time.fromMsg()` compatibility with number types

Fix: #1370

1383 of 1755 branches covered (78.8%)

Branch coverage included in aggregate %.

2861 of 3205 relevant lines covered (89.27%)

471.11 hits per line