onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 1 | include(CMakeParseArguments) |
| 2 | |
| 3 | # cache this for use inside of the function |
| 4 | set(CURRENT_LIST_DIR_CACHED ${CMAKE_CURRENT_LIST_DIR}) |
| 5 | |
onqtam | 119cfb6 | 2017-04-17 10:46:55 +0300 | [diff] [blame] | 6 | set_property(GLOBAL PROPERTY USE_FOLDERS ON) |
| 7 | |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 8 | enable_testing() |
| 9 | |
onqtam | 7b7d161 | 2018-08-17 15:56:09 +0300 | [diff] [blame] | 10 | find_package(Threads) |
| 11 | |
onqtam | 56d9170 | 2017-04-16 21:03:58 +0300 | [diff] [blame] | 12 | set(DOCTEST_TEST_MODE "COMPARE" CACHE STRING "Test mode - normal/run through valgrind/collect output/compare with output") |
| 13 | set_property(CACHE DOCTEST_TEST_MODE PROPERTY STRINGS "NORMAL;VALGRIND;COLLECT;COMPARE") |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 14 | |
onqtam | c6cc9ff | 2019-03-15 17:22:50 +0200 | [diff] [blame] | 15 | function(doctest_add_test_impl) |
| 16 | cmake_parse_arguments(ARG "NO_VALGRIND;NO_OUTPUT;XML_OUTPUT" "NAME" "COMMAND" ${ARGN}) |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 17 | if(NOT "${ARG_UNPARSED_ARGUMENTS}" STREQUAL "" OR "${ARG_NAME}" STREQUAL "" OR "${ARG_COMMAND}" STREQUAL "") |
onqtam | db5eee9 | 2016-09-15 17:12:50 +0300 | [diff] [blame] | 18 | message(FATAL_ERROR "doctest_add_test() called with wrong options!") |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 19 | endif() |
| 20 | |
| 21 | set(the_test_mode NORMAL) |
| 22 | |
| 23 | # construct the command that will be called by the exec_test.cmake script |
| 24 | set(the_command "") |
onqtam | 56d9170 | 2017-04-16 21:03:58 +0300 | [diff] [blame] | 25 | if(${DOCTEST_TEST_MODE} STREQUAL "VALGRIND" AND NOT ARG_NO_VALGRIND) |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 26 | set(the_test_mode VALGRIND) |
| 27 | set(the_command "valgrind -v --leak-check=full --track-origins=yes --error-exitcode=1") |
| 28 | endif() |
| 29 | foreach(cur ${ARG_COMMAND}) |
| 30 | set(the_command "${the_command} ${cur}") |
| 31 | endforeach() |
onqtam | c6cc9ff | 2019-03-15 17:22:50 +0200 | [diff] [blame] | 32 | if(ARG_XML_OUTPUT) |
onqtam | 2377a0c | 2019-03-20 21:04:24 +0200 | [diff] [blame] | 33 | set(the_command "${the_command} --reporters=xml") |
onqtam | c6cc9ff | 2019-03-15 17:22:50 +0200 | [diff] [blame] | 34 | set(ARG_NAME ${ARG_NAME}_xml) |
| 35 | endif() |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 36 | # append the argument for removing paths from filenames in the output so tests give the same output everywhere |
| 37 | set(the_command "${the_command} --dt-no-path-filenames=1") |
onqtam | 1fc3dc7 | 2017-03-14 14:30:09 +0200 | [diff] [blame] | 38 | # append the argument for substituting source line numbers with 0 in the output so tests give the same output when lines change a bit |
| 39 | set(the_command "${the_command} --dt-no-line-numbers=1") |
onqtam | 9f934f8 | 2016-08-02 12:42:19 +0300 | [diff] [blame] | 40 | # append the argument for ignoring the exit code of the test programs because some are intended to have failing tests |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 41 | set(the_command "${the_command} --dt-no-exitcode=1") |
onqtam | 7907692 | 2018-05-09 21:22:31 +0300 | [diff] [blame] | 42 | # append the argument for using the same line format in the output - so gcc/non-gcc builds have the same output |
| 43 | set(the_command "${the_command} --dt-gnu-file-line=0") |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 44 | |
| 45 | string(STRIP ${the_command} the_command) |
| 46 | |
onqtam | 56d9170 | 2017-04-16 21:03:58 +0300 | [diff] [blame] | 47 | if(${DOCTEST_TEST_MODE} STREQUAL "COLLECT" OR ${DOCTEST_TEST_MODE} STREQUAL "COMPARE") |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 48 | if(NOT ARG_NO_OUTPUT) |
| 49 | file(MAKE_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}/test_output/) |
onqtam | 56d9170 | 2017-04-16 21:03:58 +0300 | [diff] [blame] | 50 | set(the_test_mode ${DOCTEST_TEST_MODE}) |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 51 | list(APPEND ADDITIONAL_FLAGS -DTEST_OUTPUT_FILE=${CMAKE_CURRENT_SOURCE_DIR}/test_output/${ARG_NAME}.txt) |
onqtam | 366c368 | 2019-03-19 13:58:02 +0200 | [diff] [blame] | 52 | list(APPEND ADDITIONAL_FLAGS -DTEST_TEMP_FILE=${CMAKE_CURRENT_BINARY_DIR}${CMAKE_FILES_DIRECTORY}/temp_test_output_${ARG_NAME}.txt) |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 53 | endif() |
| 54 | endif() |
| 55 | |
| 56 | list(APPEND ADDITIONAL_FLAGS -DTEST_MODE=${the_test_mode}) |
| 57 | |
onqtam | db5eee9 | 2016-09-15 17:12:50 +0300 | [diff] [blame] | 58 | add_test(NAME ${ARG_NAME} COMMAND ${CMAKE_COMMAND} -DCOMMAND=${the_command} ${ADDITIONAL_FLAGS} -P ${CURRENT_LIST_DIR_CACHED}/exec_test.cmake) |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 59 | endfunction() |
| 60 | |
onqtam | c6cc9ff | 2019-03-15 17:22:50 +0200 | [diff] [blame] | 61 | # a custom version of add_test() to suite my needs |
| 62 | function(doctest_add_test) |
| 63 | doctest_add_test_impl(${ARGN}) |
onqtam | 2377a0c | 2019-03-20 21:04:24 +0200 | [diff] [blame] | 64 | doctest_add_test_impl(${ARGN} XML_OUTPUT) |
onqtam | c6cc9ff | 2019-03-15 17:22:50 +0200 | [diff] [blame] | 65 | endfunction() |
| 66 | |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 67 | macro(add_compiler_flags) |
| 68 | foreach(flag ${ARGV}) |
| 69 | set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${flag}") |
| 70 | endforeach() |
| 71 | endmacro() |
| 72 | |
| 73 | if(CMAKE_CXX_COMPILER_ID MATCHES "GNU|Clang") |
| 74 | add_compiler_flags(-Werror) |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 75 | add_compiler_flags(-pedantic) |
| 76 | add_compiler_flags(-pedantic-errors) |
| 77 | add_compiler_flags(-fvisibility=hidden) |
| 78 | add_compiler_flags(-fstrict-aliasing) |
| 79 | endif() |
| 80 | |
| 81 | if(CMAKE_CXX_COMPILER_ID MATCHES "GNU") |
onqtam | 5aba9f5 | 2019-03-31 13:55:03 +0300 | [diff] [blame] | 82 | #add_compiler_flags(-Wno-unknown-pragmas) |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 83 | add_compiler_flags(-Wall) |
| 84 | add_compiler_flags(-Wextra) |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 85 | add_compiler_flags(-fdiagnostics-show-option) |
| 86 | add_compiler_flags(-Wconversion) |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 87 | add_compiler_flags(-Wold-style-cast) |
| 88 | add_compiler_flags(-Wfloat-equal) |
| 89 | add_compiler_flags(-Wlogical-op) |
| 90 | add_compiler_flags(-Wundef) |
| 91 | add_compiler_flags(-Wredundant-decls) |
| 92 | add_compiler_flags(-Wshadow) |
| 93 | add_compiler_flags(-Wstrict-overflow=5) |
| 94 | add_compiler_flags(-Wwrite-strings) |
| 95 | add_compiler_flags(-Wpointer-arith) |
| 96 | add_compiler_flags(-Wcast-qual) |
| 97 | add_compiler_flags(-Wformat=2) |
| 98 | add_compiler_flags(-Wswitch-default) |
| 99 | add_compiler_flags(-Wmissing-include-dirs) |
| 100 | add_compiler_flags(-Wcast-align) |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 101 | add_compiler_flags(-Wswitch-enum) |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 102 | add_compiler_flags(-Wnon-virtual-dtor) |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 103 | add_compiler_flags(-Wctor-dtor-privacy) |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 104 | add_compiler_flags(-Wsign-conversion) |
| 105 | add_compiler_flags(-Wdisabled-optimization) |
| 106 | add_compiler_flags(-Weffc++) |
| 107 | add_compiler_flags(-Winline) |
| 108 | add_compiler_flags(-Winvalid-pch) |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 109 | add_compiler_flags(-Wmissing-declarations) |
| 110 | add_compiler_flags(-Woverloaded-virtual) |
onqtam | 8c3e296 | 2019-03-31 13:34:09 +0300 | [diff] [blame] | 111 | add_compiler_flags(-Wunused-but-set-variable) |
| 112 | add_compiler_flags(-Wunused-result) |
onqtam | 5aba9f5 | 2019-03-31 13:55:03 +0300 | [diff] [blame] | 113 | |
onqtam | c1de39c | 2019-03-31 14:01:53 +0300 | [diff] [blame^] | 114 | # add_compiler_flags(-Wsuggest-override) |
onqtam | 2990d73 | 2019-03-31 13:59:26 +0300 | [diff] [blame] | 115 | # add_compiler_flags(-Wmultiple-inheritance) |
| 116 | # add_compiler_flags(-Wcatch-value) |
onqtam | 5aba9f5 | 2019-03-31 13:55:03 +0300 | [diff] [blame] | 117 | # add_compiler_flags(-Wsuggest-attribute=cold) |
| 118 | # add_compiler_flags(-Wsuggest-attribute=const) |
| 119 | # add_compiler_flags(-Wsuggest-attribute=format) |
| 120 | # add_compiler_flags(-Wsuggest-attribute=malloc) |
| 121 | # add_compiler_flags(-Wsuggest-attribute=noreturn) |
| 122 | # add_compiler_flags(-Wsuggest-attribute=pure) |
| 123 | # add_compiler_flags(-Wsuggest-final-methods) |
| 124 | # add_compiler_flags(-Wsuggest-final-types) |
| 125 | |
| 126 | if(NOT CMAKE_CXX_COMPILER_VERSION VERSION_LESS 4.6) |
| 127 | add_compiler_flags(-Wnoexcept) |
| 128 | endif() |
onqtam | 8c3e296 | 2019-03-31 13:34:09 +0300 | [diff] [blame] | 129 | |
onqtam | abf39d2 | 2017-10-28 21:30:45 +0300 | [diff] [blame] | 130 | # no way to silence it in the expression decomposition macros: _Pragma() in macros doesn't work for the c++ front-end of g++ |
| 131 | # https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55578 |
| 132 | # https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69543 |
| 133 | # Also the warning is completely worthless nowadays - http://stackoverflow.com/questions/14016993 |
| 134 | #add_compiler_flags(-Waggregate-return) |
onqtam | 5aba9f5 | 2019-03-31 13:55:03 +0300 | [diff] [blame] | 135 | |
| 136 | if(NOT CMAKE_CXX_COMPILER_VERSION VERSION_LESS 5.0) |
| 137 | add_compiler_flags(-Wdouble-promotion) |
| 138 | add_compiler_flags(-Wtrampolines) |
| 139 | add_compiler_flags(-Wzero-as-null-pointer-constant) |
| 140 | add_compiler_flags(-Wuseless-cast) |
| 141 | add_compiler_flags(-Wvector-operation-performance) |
| 142 | endif() |
| 143 | |
| 144 | if(NOT CMAKE_CXX_COMPILER_VERSION VERSION_LESS 6.0) |
| 145 | add_compiler_flags(-Wshift-overflow=2) |
| 146 | add_compiler_flags(-Wnull-dereference) |
| 147 | add_compiler_flags(-Wduplicated-cond) |
| 148 | endif() |
| 149 | |
| 150 | if(NOT CMAKE_CXX_COMPILER_VERSION VERSION_LESS 7.0) |
| 151 | add_compiler_flags(-Walloc-zero) |
| 152 | add_compiler_flags(-Walloca) |
| 153 | add_compiler_flags(-Wduplicated-branches) |
| 154 | endif() |
| 155 | |
| 156 | if(NOT CMAKE_CXX_COMPILER_VERSION VERSION_LESS 8.0) |
| 157 | add_compiler_flags(-Wcast-align=strict) |
| 158 | endif() |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 159 | endif() |
| 160 | |
| 161 | if(CMAKE_CXX_COMPILER_ID MATCHES "Clang") |
| 162 | add_compiler_flags(-Weverything) |
onqtam | 4d87d56 | 2018-07-03 22:23:42 +0300 | [diff] [blame] | 163 | add_compiler_flags(-Wno-c++98-compat) |
| 164 | add_compiler_flags(-Wno-c++98-compat-pedantic) |
| 165 | add_compiler_flags(-Wno-c++98-compat-bind-to-temporary-copy) |
| 166 | add_compiler_flags(-Wno-c++98-compat-local-type-template-args) |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 167 | add_compiler_flags(-Qunused-arguments -fcolor-diagnostics) # needed for ccache integration on travis |
| 168 | endif() |
| 169 | |
Martin Moene | d8a7edc | 2017-08-01 17:08:08 +0200 | [diff] [blame] | 170 | if(MSVC) |
onqtam | 8edeaaf | 2016-08-16 07:12:58 +0300 | [diff] [blame] | 171 | add_compiler_flags(/std:c++latest) # for post c++14 updates in MSVC |
onqtam | 50cbb80 | 2017-12-09 17:17:39 +0200 | [diff] [blame] | 172 | add_compiler_flags(/permissive-) # force standard conformance - this is the better flag than /Za |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 173 | add_compiler_flags(/WX) |
onqtam | abf39d2 | 2017-10-28 21:30:45 +0300 | [diff] [blame] | 174 | add_compiler_flags(/Wall) # turns on warnings from levels 1 through 4 which are off by default - https://msdn.microsoft.com/en-us/library/23k5d385.aspx |
| 175 | |
| 176 | add_compiler_flags( |
| 177 | /wd4514 # unreferenced inline function has been removed |
| 178 | /wd4571 # SEH related |
| 179 | /wd4710 # function not inlined |
| 180 | /wd4711 # function 'x' selected for automatic inline expansion |
onqtam | 5ed699b | 2017-10-28 21:57:15 +0300 | [diff] [blame] | 181 | |
| 182 | /wd4616 # invalid compiler warnings - https://msdn.microsoft.com/en-us/library/t7ab6xtd.aspx |
| 183 | /wd4619 # invalid compiler warnings - https://msdn.microsoft.com/en-us/library/tacee08d.aspx |
| 184 | |
| 185 | #/wd4820 # padding in structs |
| 186 | #/wd4625 # copy constructor was implicitly defined as deleted |
| 187 | #/wd4626 # assignment operator was implicitly defined as deleted |
| 188 | #/wd5027 # move assignment operator was implicitly defined as deleted |
| 189 | #/wd5026 # move constructor was implicitly defined as deleted |
| 190 | #/wd4623 # default constructor was implicitly defined as deleted |
onqtam | abf39d2 | 2017-10-28 21:30:45 +0300 | [diff] [blame] | 191 | ) |
onqtam | 4a65563 | 2016-05-26 14:20:52 +0300 | [diff] [blame] | 192 | endif() |