Fix action support to interface_loader.loadInterfaceByPath()

RobotWebTools / rclnodejs

Node.js version of ROS 2.0 client

https://docs.ros.org/en/humble/Concepts/Basic/About-Client-Libraries.html?highlight=rclnodejs#community-maintained

Apache License 2.0

316 stars 70 forks source link

Fix action support to interface_loader.loadInterfaceByPath() #730

Closed wayneparrott closed 3 years ago

wayneparrott commented 3 years ago

Commit Summary

interface_loader.js

fixed bug in loadInterfaceByPath() when loading action messages

test-message-object.js

added 'require() and create instance of every message in package' unit test

Fix #729

wayneparrott commented 3 years ago

My Bad! For some reason I assumed nav2_msgs were part of a std ros2 install. Fixing build break now by using tf2_msgs.

coveralls commented 3 years ago

Coverage increased (+0.05%) to 91.563% when pulling 3e6f3901e48c277fa34f435aed69d79c8cfebb91 on wayneparrott:load_interface_action_fix into b09a171b9a97756a79ea93bcc383dfcf2fbe3c5c on RobotWebTools:develop.