diff --git a/CMakeModules/testing_cpp.cmake b/CMakeModules/testing_cpp.cmake index 08b832cd57bf3f0ade824eef38ab6268f0e35506..fb082f7f910c0261dd0b6bdb066183c056cbc33d 100644 --- a/CMakeModules/testing_cpp.cmake +++ b/CMakeModules/testing_cpp.cmake @@ -12,6 +12,10 @@ if (BUILD_TESTS) set(gtest_SOURCE_DIR $ENV{gtest_SOURCE_DIR}) endif () message(STATUS "Will look for google test at ${gtest_SOURCE_DIR}") + if (CMAKE_COMPILER_IS_GNUCXX) + include(CodeCoverage) + APPEND_COVERAGE_COMPILER_FLAGS() + endif() endif () function(gtest target test_source_files linktarget) @@ -42,8 +46,6 @@ function(gtest target test_source_files linktarget) message(STATUS "Added test 'test-${target}'") if (CMAKE_COMPILER_IS_GNUCXX) - include(CodeCoverage) - APPEND_COVERAGE_COMPILER_FLAGS() set(COVERAGE_EXCLUDES "*/unittests/*" "*/3d_party/*" ) if (ARGN) set(COVERAGE_EXCLUDES ${COVERAGE_EXCLUDES} ${ARGN}) diff --git a/common/cpp/include/unittests/MockIO.h b/common/cpp/include/unittests/MockIO.h index d739483fad6d14d0ad87007822a5cee5d8556cdf..fbe0b95103385cbb837fd27541d65f1b312c6e5e 100644 --- a/common/cpp/include/unittests/MockIO.h +++ b/common/cpp/include/unittests/MockIO.h @@ -24,7 +24,7 @@ class MockIO : public IO { } MOCK_CONST_METHOD3(GetDataFromFile_t, - uint8_t*(const std::string& fname, uint64_t fsize, IOErrors* err)); + uint8_t* (const std::string& fname, uint64_t fsize, IOErrors* err)); MOCK_CONST_METHOD2(FilesInFolder, FileInfos( const std::string& folder, IOErrors diff --git a/worker/api/cpp/unittests/test_server_broker.cpp b/worker/api/cpp/unittests/test_server_broker.cpp index f9cbca9a499c31796df680b3c81846f74451b195..6e083a32d39597bb63299af2dc9f5913f4e78dcc 100644 --- a/worker/api/cpp/unittests/test_server_broker.cpp +++ b/worker/api/cpp/unittests/test_server_broker.cpp @@ -73,8 +73,8 @@ TEST_F(ServerDataBrokerTests, GetNextReturnsErrorOnWrongInput) { TEST_F(ServerDataBrokerTests, GetNextUsesCorrectUri) { EXPECT_CALL(mock_http_client, Get("test/next?database=database", _)).WillOnce(DoAll( - SetArgPointee<1>(WorkerErrorCode::kOK), - Return(""))); + SetArgPointee<1>(WorkerErrorCode::kOK), + Return(""))); data_broker->GetNext(&info, nullptr); } @@ -152,7 +152,7 @@ TEST_F(ServerDataBrokerTests, GetNextCallsReadFromFile) { EXPECT_CALL(mock_io, GetDataFromFile_t("relative_path/base_name", 100, _)). - WillOnce(DoAll(SetArgPointee<2>(IOErrors::kReadError), testing::Return(nullptr))); + WillOnce(DoAll(SetArgPointee<2>(IOErrors::kReadError), testing::Return(nullptr))); FileData data; data_broker->GetNext(&info, &data);