Improve testing

4 files changed, 35 insertions, 12 deletions

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 9723294..fcd07d9 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -40,4 +40,4 @@ jobs:
       - name: make
         run: cmake --build .
       - name: test
-        run: ctest -V
+        run: ctest -C Debug -V
diff --git a/CMakeLists.txt b/CMakeLists.txt
index fef8144..5f17030 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -2,13 +2,15 @@ cmake_minimum_required(VERSION 2.8.11)
 project(relocate C)
 set(VERSION "1.2.2")
 set(CMAKE_C_STANDARD 99)
-if(MSVC)
-    set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -D_CRT_SECURE_NO_WARNINGS /Wall")
-else()
+if("${CMAKE_C_COMPILER_ID}" STREQUAL "GNU"
+   OR "${CMAKE_C_COMPILER_ID}" STREQUAL "Clang")
     set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -std=c99 -Wall -Wextra")
+elseif("${CMAKE_C_COMPILER_ID}" STREQUAL "MSVC")
+    set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -D_CRT_SECURE_NO_WARNINGS /Wall")
 endif()
 
 enable_testing()
+include(CTest)
 add_subdirectory(test)
 
 configure_file(version.h.in version.h)
diff --git a/test/CMakeLists.txt b/test/CMakeLists.txt
index 6b79d98..56cdae2 100644
--- a/test/CMakeLists.txt
+++ b/test/CMakeLists.txt
@@ -1,12 +1,33 @@
 project(tests)
 
+set(INPUT_SEARCH
+	/_0123456789_0123456789_0123456789_0123456789_0123456789_0123456789_0123456789_)
+set(INPUT_REPLACE
+	_ctest_replaced_this)
+set(INPUT_FN
+	test_data.bin)
+set(OUTPUT_FN
+	test_data.out)
+
+configure_file(${INPUT_FN} ${INPUT_FN} COPYONLY)
+file(REMOVE ${OUTPUT_FN})
+
 include_directories("${CMAKE_SOURCE_DIR}")
 include_directories("${CMAKE_BINARY_DIR}")
-add_executable(test_reloc_match test_reloc_match.c)
-add_executable(test_reloc_read test_reloc_read.c)
 
-target_link_libraries(test_reloc_match relocate)
-target_link_libraries(test_reloc_read relocate)
+function(add_reloc_test target)
+	add_executable(${target} ${ARGN})
+	target_link_libraries(${target} relocate)
+	add_test(${target} ${target})
+endfunction()
+
+function(add_exec_test target)
+	add_test(NAME ${target} COMMAND ${ARGN})
+endfunction()
+
+add_reloc_test(test_reloc_match test_reloc_match.c)
+add_reloc_test(test_reloc_read test_reloc_read.c)
+add_exec_test(test_exec_success reloc "${INPUT_SEARCH}" "${INPUT_REPLACE}" "${INPUT_FN}" "${OUTPUT_FN}")
 
-add_test(test_reloc_match test_reloc_match)
-add_test(test_reloc_read test_reloc_read)
+# Windows users need grep too. "findstr" and "find" were not reliable here
+add_exec_test(test_exec_result grep "${INPUT_REPLACE}" "${OUTPUT_FN}")
diff --git a/test/test_reloc_read.c b/test/test_reloc_read.c
index 39d5eb8..934bb9f 100644
--- a/test/test_reloc_read.c
+++ b/test/test_reloc_read.c
@@ -19,7 +19,7 @@ int test_reloc_read() {
     myassert("info->size is incorrect", info->size == sizeof(test_case));
     myassert("info->path is incorrect", !strcmp(info->path, input_file));
     myassert("info->data is uninitialized", info->data);
-    free(info);
+    reloc_deinit_data(info);
     return 0;
 }
 
@@ -27,7 +27,7 @@ int test_reloc_read_verify() {
     RelocData *info = reloc_read(input_file);
     myassert("failed to populate RelocData struct", info);
     myassert("info->data != input_data contents", !memcmp(test_case, info->data, info->size));
-    free(info);
+    reloc_deinit_data(info);
     return 0;
 }