Fix dependencies for source build (#1533)

* Don't use std::system call inside test

* Give launch_ros as explicit dependency

* Test the source build on this branch

* Revert "Test the source build on this branch"

This reverts commit 9884b11.

controller_interface/test/test_controller_interface.cpp

@@ -88,7 +88,8 @@ TEST(TestableControllerInterface, setting_update_rate_in_configure)
   ASSERT_EQ(controller.get_update_rate(), 2812u);
 
   // Test updating of update_rate parameter
-  EXPECT_EQ(std::system("ros2 param set /testable_controller_interface update_rate 623"), 0);
+  auto res = controller.get_node()->set_parameter(rclcpp::Parameter("update_rate", 623));
+  EXPECT_EQ(res.successful, true);
   // Keep the same update rate until transition from 'UNCONFIGURED' TO 'INACTIVE' does not happen
   controller.configure();  // No transition so the update rate should stay intact
   ASSERT_NE(controller.get_update_rate(), 623u);

joint_limits/package.xml

@@ -20,6 +20,7 @@
   <depend>rclcpp_lifecycle</depend>
   <depend>urdf</depend>
 
+  <test_depend>launch_ros</test_depend>
   <test_depend>launch_testing_ament_cmake</test_depend>
   <test_depend>ament_cmake_gtest</test_depend>