diff --git a/test/CMakeLists.txt b/test/CMakeLists.txt index 757b081..0add149 100644 --- a/test/CMakeLists.txt +++ b/test/CMakeLists.txt @@ -27,6 +27,15 @@ target_link_libraries(options_test_noregex cxxopts) target_compile_definitions(options_test_noregex PRIVATE CXXOPTS_NO_REGEX) add_test(options_no_regex options_test_noregex) +# Run the parser tests under UBSan so undefined behaviour in integer parsing +# (e.g. std::abs(INT64_MIN)) fails the build instead of going unnoticed. +if(CMAKE_CXX_COMPILER_ID MATCHES "Clang|GNU") + foreach(test_target options_test options_test_noregex) + target_compile_options(${test_target} PRIVATE -fsanitize=undefined -fno-sanitize-recover=undefined) + set_target_properties(${test_target} PROPERTIES LINK_FLAGS "-fsanitize=undefined -fno-sanitize-recover=undefined") + endforeach() +endif() + # test if the targets are findable from the build directory add_test(find-package-test ${CMAKE_CTEST_COMMAND} -C ${CMAKE_BUILD_TYPE} diff --git a/test/options.cpp b/test/options.cpp index fb495ed..b41d883 100644 --- a/test/options.cpp +++ b/test/options.cpp @@ -2,6 +2,7 @@ #include #include +#include #include "cxxopts.hpp" @@ -921,6 +922,8 @@ TEST_CASE("Overflow on boundary", "[integer]") CHECK_THROWS_AS((integer_parser("-42769", si16)), cxxopts::exceptions::incorrect_argument_type); CHECK_THROWS_AS((integer_parser("-75536", si16)), cxxopts::exceptions::incorrect_argument_type); + CHECK_NOTHROW((integer_parser("-9223372036854775808", si64))); + CHECK(si64 == (std::numeric_limits::min)()); CHECK_THROWS_AS((integer_parser("18446744073709551616", ui64)), cxxopts::exceptions::incorrect_argument_type); CHECK_THROWS_AS((integer_parser("28446744073709551616", ui64)), cxxopts::exceptions::incorrect_argument_type); CHECK_THROWS_AS((integer_parser("9223372036854775808", si64)), cxxopts::exceptions::incorrect_argument_type);