Use option for coverage and test build flag (#6224)
author오형석/On-Device Lab(SR)/Staff Engineer/삼성전자 <hseok82.oh@samsung.com>
Mon, 5 Aug 2019 08:47:59 +0000 (17:47 +0900)
committerGitHub Enterprise <noreply-CODE@samsung.com>
Mon, 5 Aug 2019 08:47:59 +0000 (17:47 +0900)
Use cmake option to set ENABLE_TEST and ENABLE_COVERAGE

Signed-off-by: Hyeongseok Oh <hseok82.oh@samsung.com>
infra/nncc/CMakeLists.txt
infra/nnfw/CMakeLists.txt
infra/nnfw/cmake/CfgOptionFlags.cmake

index f60799c..12b8406 100644 (file)
@@ -96,19 +96,13 @@ option(BUILD_GTEST "Build Google Test from the downloaded source" ON)
 
 nncc_find_package(GTest QUIET)
 
-if(NOT DEFINED ENABLE_TEST)
-  # Enable test by default
-  set(ENABLE_TEST ${GTest_FOUND})
-endif(NOT DEFINED ENABLE_TEST)
+option(ENABLE_TEST "Build Tests using Google Test" ${GTest_FOUND})
 
 if(${ENABLE_TEST} AND NOT ${GTest_FOUND})
   message(FATAL_ERROR "Google Test is required to enable test")
 endif(${ENABLE_TEST} AND NOT ${GTest_FOUND})
 
-if(NOT DEFINED ENABLE_COVERAGE)
-  set(ENABLE_COVERAGE FALSE)
-endif(NOT DEFINED ENABLE_COVERAGE)
-
+option(ENABLE_COVERAGE "Build for coverage test" OFF)
 if(${ENABLE_COVERAGE} AND NOT ${ENABLE_TEST})
   message(FATAL_ERROR "Test should be enabled to measure test coverage")
 endif(${ENABLE_COVERAGE} AND NOT ${ENABLE_TEST})
index 1a93d03..868381a 100644 (file)
@@ -57,19 +57,12 @@ include("cmake/CfgOptionFlags.cmake")
 
 nnfw_find_package(GTest QUIET)
 
-if(NOT DEFINED ENABLE_TEST)
-  # Enable test by default
-  set(ENABLE_TEST ${GTest_FOUND})
-endif(NOT DEFINED ENABLE_TEST)
+option(ENABLE_TEST "Build Tests using Google Test" ${GTest_FOUND})
 
 if(${ENABLE_TEST} AND NOT ${GTest_FOUND})
   message(FATAL_ERROR "Google Test is required to enable test")
 endif(${ENABLE_TEST} AND NOT ${GTest_FOUND})
 
-if(NOT DEFINED ENABLE_COVERAGE)
-  set(ENABLE_COVERAGE FALSE)
-endif(NOT DEFINED ENABLE_COVERAGE)
-
 if(${ENABLE_COVERAGE} AND NOT ${ENABLE_TEST})
   message(FATAL_ERROR "Test should be enabled to measure test coverage")
 endif(${ENABLE_COVERAGE} AND NOT ${ENABLE_TEST})
index 4cc3c94..7041626 100644 (file)
@@ -58,3 +58,4 @@ option(BUILD_BOOST "Build boost source" OFF)
 #
 option(BUILD_GTEST "Download and build Google Test" ON)
 option(ENABLE_STRICT_BUILD "Treat warning as error" ON)
+option(ENABLE_COVERAGE "Build for coverage test" OFF)