Skip to content

Commit 15cf46b

Browse files
committed
[fix] move dep from gtest to test where it's actually used
1 parent 502f24a commit 15cf46b

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

tests/CMakeLists.txt

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,14 @@
22
file(GLOB TEST_SOURCE "*_test.cpp")
33

44
if(TEST_SOURCE)
5+
find_package(ament_index_cpp REQUIRED)
6+
57
foreach(TEST_FILE ${TEST_SOURCE})
68
get_filename_component(TEST_NAME ${TEST_FILE} NAME_WE)
79

810
add_executable(${PROJECT_NAME}_${TEST_NAME} ${TEST_FILE})
911
ament_target_dependencies(${PROJECT_NAME}_${TEST_NAME} ${PROJECT_DEPENDENCIES})
10-
target_link_libraries(${PROJECT_NAME}_${TEST_NAME} ${PROJECT_NAME})
12+
target_link_libraries(${PROJECT_NAME}_${TEST_NAME} ament_index_cpp::ament_index_cpp ${PROJECT_NAME})
1113
endforeach()
1214
endif()
1315

@@ -16,13 +18,12 @@ file(GLOB GTEST_SOURCE "*_gtest.cpp")
1618

1719
if(GTEST_SOURCE)
1820
find_package(ament_cmake_gtest REQUIRED)
19-
find_package(ament_index_cpp REQUIRED)
2021

2122
foreach(TEST_SOURCE ${GTEST_SOURCE})
2223
get_filename_component(TEST_NAME ${TEST_SOURCE} NAME_WE)
2324

2425
ament_add_gtest(${PROJECT_NAME}_${TEST_NAME} ${TEST_SOURCE})
2526
ament_target_dependencies(${PROJECT_NAME}_${TEST_NAME} ${PROJECT_DEPENDENCIES})
26-
target_link_libraries(${PROJECT_NAME}_${TEST_NAME} ament_index_cpp::ament_index_cpp gtest_main ${PROJECT_NAME})
27+
target_link_libraries(${PROJECT_NAME}_${TEST_NAME} gtest_main ${PROJECT_NAME})
2728
endforeach()
2829
endif()

0 commit comments

Comments
 (0)