Fix wrong clock message type (#928)

Co-authored-by: AntoninRousset <[email protected]>
RobotWebTools · Aug 18, 2023 · 13ec992 · 13ec992
1 parent 7081170
commit 13ec992
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/lib/time_source.js b/lib/time_source.js
@@ -176,7 +176,7 @@ class TimeSource {
   }
 
   _clockCallback(msg) {
-    this._lastTimeSet = Time.fromMsg(msg);
+    this._lastTimeSet = Time.fromMsg(msg.clock);
     this._associatedClocks.forEach((clock) => {
       clock.rosTimeOverride = this._lastTimeSet;
     });
@@ -185,7 +185,7 @@ class TimeSource {
   _subscribeToClockTopic() {
     if (!this._clockSubscription && this._node) {
       this._clockSubscription = this._node.createSubscription(
-        'builtin_interfaces/msg/Time',
+        'rosgraph_msgs/msg/Clock',
         CLOCK_TOPIC,
         this._clockCallback.bind(this)
       );

diff --git a/test/test-time-source.js b/test/test-time-source.js
@@ -45,10 +45,10 @@ describe('rclnodejs TimeSource testing', function () {
   });
 
   function publishClockMessage(node) {
-    let pub = node.createPublisher('builtin_interfaces/msg/Time', '/clock');
+    let pub = node.createPublisher('rosgraph_msgs/msg/Clock', '/clock');
     let count = 0;
     timer = setInterval(() => {
-      pub.publish({ sec: count, nanosec: 0 });
+      pub.publish({ clock: { sec: count, nanosec: 0 } });
       count += 1;
     }, 1000);
   }