Skip to content
This repository was archived by the owner on Feb 12, 2022. It is now read-only.

Commit 326fcc3

Browse files
committed
Fixing test build issues with conflicting/missing GMock.
1 parent 46ef7fe commit 326fcc3

File tree

2 files changed

+28
-14
lines changed

2 files changed

+28
-14
lines changed

health_metric_collector/CMakeLists.txt

Lines changed: 25 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -62,18 +62,29 @@ install(TARGETS health_metric_collector
6262

6363
if(CATKIN_ENABLE_TESTING)
6464
find_package(rostest REQUIRED)
65-
include_directories(/usr/include/gmock /usr/src/gmock)
66-
add_library(libgmock /usr/src/gmock/src/gmock-all.cc)
67-
68-
add_rostest_gtest(test_health_metric_collector
69-
test/test_health_metric_collector.test
70-
test/health_metric_collector_test.cpp
71-
)
72-
73-
target_include_directories(test_health_metric_collector
74-
PRIVATE include
75-
${catkin_INCLUDE_DIRS}
76-
)
77-
78-
target_link_libraries(test_health_metric_collector ${catkin_LIBRARIES} libgmock collector_lib)
65+
find_package(GMock QUIET)
66+
if(GMOCK_FOUND)
67+
add_rostest_gmock(test_health_metric_collector
68+
test/test_health_metric_collector.test
69+
test/health_metric_collector_test.cpp
70+
)
71+
target_include_directories(test_health_metric_collector
72+
PRIVATE include
73+
${catkin_INCLUDE_DIRS}
74+
${GMOCK_INCLUDE_DIRS}
75+
)
76+
target_link_libraries(test_health_metric_collector ${catkin_LIBRARIES} collector_lib ${GMOCK_BOTH_LIBRARIES})
77+
else()
78+
include_directories(/usr/include/gmock /usr/src/gmock)
79+
add_library(${PROJECT_NAME}_libgmock /usr/src/gmock/src/gmock-all.cc)
80+
add_rostest_gtest(test_health_metric_collector
81+
test/test_health_metric_collector.test
82+
test/health_metric_collector_test.cpp
83+
)
84+
target_include_directories(test_health_metric_collector
85+
PRIVATE include
86+
${catkin_INCLUDE_DIRS}
87+
)
88+
target_link_libraries(test_health_metric_collector ${catkin_LIBRARIES} collector_lib ${PROJECT_NAME}_libgmock)
89+
endif()
7990
endif()

health_metric_collector/package.xml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,4 +21,7 @@
2121

2222
<exec_depend>message_runtime</exec_depend>
2323
<exec_depend>cloudwatch_metrics_collector</exec_depend>
24+
25+
<test_depend>rostest</test_depend>
26+
<test_depend>google-mock</test_depend>
2427
</package>

0 commit comments

Comments
 (0)