From 4069bdd863e0b6d669afe7f4cbb08445deebd799 Mon Sep 17 00:00:00 2001 From: Kei Okada Date: Thu, 15 Dec 2022 11:04:01 +0900 Subject: [PATCH] add_rostest before check dpkg, to run test code on even in indigo, see #378 --- 3rdparty/aques_talk/CMakeLists.txt | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/3rdparty/aques_talk/CMakeLists.txt b/3rdparty/aques_talk/CMakeLists.txt index 9da594eff..9e5da855d 100644 --- a/3rdparty/aques_talk/CMakeLists.txt +++ b/3rdparty/aques_talk/CMakeLists.txt @@ -5,6 +5,11 @@ find_package(catkin REQUIRED) catkin_package() +if(CATKIN_ENABLE_TESTING) + find_package(catkin REQUIRED COMPONENTS rostest) + add_rostest(test/test_aques_talk.test) +endif() + # Considering the use of docker in addition to the usual use, # we use dpkg-architecture command instead of ${CMAKE_SYSTEM_PROCESSOR} to get the architecture. # https://stackoverflow.com/a/58222507 @@ -50,8 +55,3 @@ install(DIRECTORY launch DESTINATION ${CATKIN_PACKAGE_SHARE_DESTINATION} USE_SOURCE_PERMISSIONS) -if(CATKIN_ENABLE_TESTING) - find_package(catkin REQUIRED COMPONENTS rostest) - add_rostest(test/test_aques_talk.test) - add_dependencies(tests SampleTalk) # make sure to build SampleTalk before running tests -endif()