Skip to content

Commit 976e098

Browse files
authored
Merge pull request PR2#255 from furushchev/test-launch
pr2_bringup: test launch files
2 parents 48cdc2e + 6e5c976 commit 976e098

File tree

2 files changed

+15
-0
lines changed

2 files changed

+15
-0
lines changed

pr2_bringup/CMakeLists.txt

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,3 +27,15 @@ install(FILES
2727
pr2_recalibrate.launch
2828
pr2-c2.launch
2929
DESTINATION ${CATKIN_PACKAGE_SHARE_DESTINATION})
30+
31+
if (CATKIN_ENABLE_TESTING)
32+
find_package(rostest REQUIRED)
33+
find_package(roslaunch REQUIRED)
34+
# Tests that PR2 diagnostic analyzers configuration loads
35+
add_rostest(test/pr2_analyzers_load_test.launch)
36+
# Disable because pr2.launch needs to resolve c1/c2 hostname
37+
# roslaunch_add_file_check(pr2.launch ROS_ENV_LOADER=/opt/ros/$ENV{ROS_DISTRO}/env.sh)
38+
# roslaunch_add_file_check(pr2-c2.launch)
39+
roslaunch_add_file_check(pr2_recalibrate.launch)
40+
endif()
41+

pr2_bringup/package.xml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,9 @@
1212

1313
<buildtool_depend>catkin</buildtool_depend>
1414

15+
<!-- for testing -->
1516
<build_depend>rostest</build_depend>
17+
<build_depend>roslaunch</build_depend>
1618

1719
<run_depend>pr2_description</run_depend>
1820
<run_depend>pr2_machine</run_depend>
@@ -47,4 +49,5 @@
4749
<run_depend>pr2_computer_monitor</run_depend>
4850
<run_depend>diagnostic_aggregator</run_depend>
4951
<run_depend>pr2_dashboard_aggregator</run_depend>
52+
<run_depend>imu_monitor</run_depend>
5053
</package>

0 commit comments

Comments
 (0)