search

ros2 / ros2cli

ROS 2 command line interface tools
Apache License 2.0
173 stars 159 forks source link

Fix tests with new service and param present #838

Closed emersonknapp closed 1 year ago

emersonknapp commented 1 year ago

Part of ros2/ros2#1159 Depends on ros2/rclpy#1139

ros2/rclpy#1139 starts a new service and introduces a new parameter to enable it. This PR fixes test expectations given the new situation, including turning off the new service on the daemon node.

emersonknapp commented 1 year ago

Pulls: ros2/rclpy#1139, ros2/ros2cli#838 Gist: https://gist.githubusercontent.com/emersonknapp/b636f6ea5ccf709b1fc5684de991cb07/raw/99521abc39bdfb6dd39d76e87727d54d198a9cce/ros2.repos BUILD args: --packages-above-and-dependencies rclpy ros2cli ros2param ros2service TEST args: --packages-above rclpy ros2cli ros2param ros2service ROS Distro: rolling Job: ci_launcher ci_launcher ran: https://ci.ros2.org/job/ci_launcher/12334