Skip to content

Commit cbbfef5

Browse files
authored
Fix Buildfarm Tests (#23)
1 parent 5611184 commit cbbfef5

File tree

9 files changed

+15
-13
lines changed

9 files changed

+15
-13
lines changed

dlux_global_planner/CMakeLists.txt

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -49,13 +49,6 @@ if (CATKIN_ENABLE_TESTING)
4949
find_package(rostest REQUIRED)
5050
catkin_add_gtest(kernel_test test/kernel_test.cpp)
5151
target_link_libraries(kernel_test dgp)
52-
53-
find_package(global_planner_tests)
54-
include_directories(include ${global_planner_tests_INCLUDE_DIRS} ${catkin_INCLUDE_DIRS} ${GTEST_INCLUDE_DIRS})
55-
link_directories(${GTEST_LIBRARY_DIRS})
56-
57-
add_rostest_gtest(got test/global_oscillation_test.launch test/global_oscillation_test.cpp)
58-
target_link_libraries(got dgp ${global_planner_tests_LIBRARIES} ${catkin_LIBRARIES} ${GTEST_LIBRARIES})
5952
endif()
6053

6154
install(TARGETS planner_node

dlux_global_planner/package.xml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@
1818
<depend>pluginlib</depend>
1919
<depend>roscpp</depend>
2020
<depend>visualization_msgs</depend>
21-
<test_depend>global_planner_tests</test_depend>
2221
<test_depend>roslint</test_depend>
2322
<export>
2423
<nav_core2 plugin="${prefix}/nav_core2_plugins.xml"/>

dlux_global_planner/test/global_oscillation_test.launch

Lines changed: 0 additions & 3 deletions
This file was deleted.

dlux_plugins/CMakeLists.txt

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,9 @@ if (CATKIN_ENABLE_TESTING)
3232
# add_rostest_gtest(full_planner_test test/full_planner_test.launch test/full_planner_test.cpp)
3333
# target_link_libraries(full_planner_test ${global_planner_tests_LIBRARIES} ${catkin_LIBRARIES} ${GTEST_LIBRARIES})
3434

35+
add_rostest_gtest(got test/global_oscillation_test.launch test/global_oscillation_test.cpp)
36+
target_link_libraries(got ${global_planner_tests_LIBRARIES} ${catkin_LIBRARIES} ${GTEST_LIBRARIES})
37+
3538
find_package(roslint)
3639
roslint_cpp()
3740
roslint_add_test()

dlux_global_planner/test/global_oscillation_test.cpp renamed to dlux_plugins/test/global_oscillation_test.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@
4242
#include <string>
4343
#include <vector>
4444

45-
const std::string map_path = "package://dlux_global_planner/test/robert_frost.png";
45+
const std::string map_path = "package://dlux_plugins/test/robert_frost.png";
4646

4747
int barrier_x = 5;
4848
int barrier_y0 = 1;
Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
<launch>
2+
<test test-name="global_oscillation_test" pkg="dlux_plugins" type="got" time-limit="300"/>
3+
</launch>
Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
11
<launch>
2-
<test test-name="planner_test" pkg="dlux_plugins" type="planner_test" time-limit="300"/>
2+
<!-- Planner Test contains 6 configurations, each of which can take over a minute on a shiny developer laptop. -->
3+
<!-- Conservatively, we limit each configuration to 3 minutes, allowing for the slowness of buildfarms, etc. -->
4+
<test test-name="planner_test" pkg="dlux_plugins" type="planner_test" time-limit="1080"/>
35
</launch>
File renamed without changes.

nav_core_adapter/package.xml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,11 @@
2626
<test_depend>roslint</test_depend>
2727
<test_depend>rostest</test_depend>
2828

29+
<!-- When running the unload test, it loads DWB, so these are needed -->
30+
<test_depend>dwb_local_planner</test_depend>
31+
<test_depend>dwb_plugins</test_depend>
32+
<test_depend>dwb_critics</test_depend>
33+
2934
<export>
3035
<nav_core plugin="${prefix}/nav_core_plugins.xml"/>
3136
<nav_core2 plugin="${prefix}/nav_core2_plugins.xml"/>

0 commit comments

Comments
 (0)