use ctest and set path to dbc files with a compile definition

Author: Murmele

.github/workflows/tests.yml

@@ -17,13 +17,18 @@ jobs:
     - uses: actions/checkout@v3
 
     - name: Configure CMake
-      run: cmake -B ${{github.workspace}}/build -DCMAKE_BUILD_TYPE=${{env.BUILD_TYPE}}
+      run: |
+        mkdir build
+        cd build
+        cmake -DCMAKE_BUILD_TYPE=${{env.BUILD_TYPE}} ..
 
     - name: Build
-      run: cmake --build ${{github.workspace}}/build --config ${{env.BUILD_TYPE}} -- -j
+      run: |
+        cd build
+        make
 
     - name: Test
-      working-directory: ${{github.workspace}}/build
-      # Didn't configure to use ctest. Opted for a custom target to run instead
-      run: cmake --build ${{github.workspace}}/build --target test -- -j
+      run: |
+        cd build
+        ctest --output-on-failure
 

CMakeLists.txt

@@ -3,6 +3,7 @@ cmake_minimum_required(VERSION 3.10)
 project(dbc)
 
 option(DEBUG "use debug flag" NO)
+option(ENABLE_TESTS "Enable Unittests" ON)
 
 # specify the C++ standard
 set(CMAKE_CXX_STANDARD 11)
@@ -31,7 +32,11 @@ list(APPEND SOURCE ${PROJECT_SOURCE_DIR}/src/utils.cpp
 include_directories(src)
 include_directories(include)
 
-add_subdirectory(test)
+if(ENABLE_TESTS)
+    include(CTest)
+    add_subdirectory(test)
+endif()
+
 add_subdirectory(doc)
 
 add_library(${PROJECT_NAME} STATIC ${SOURCE})

test/CMakeLists.txt

@@ -1,4 +1,4 @@
-project(tests VERSION 0.1.0)
+enable_testing()
 
 # Download and build Catch2 test framework
 Include(FetchContent)
@@ -14,9 +14,11 @@ list(APPEND TEST_SOURCES
 			 test_utils.cpp)
 
 add_executable(tests ${TEST_SOURCES} ${SOURCE})
+target_compile_definitions(tests PRIVATE TESTDBCFILES_PATH="${CMAKE_CURRENT_SOURCE_DIR}/dbcs")
 target_link_libraries(tests PRIVATE Catch2::Catch2WithMain)
+target_sources(tests INTERFACE FILE_SET HEADERS
+                               TYPE HEADERS
+                               BASE_DIRS ${CMAKE_CURRENT_SOURCE_DIR}
+                               FILES defines.hpp)
 
-add_custom_target(test
-					COMMAND ${PROJECT_NAME}
-					WORKING_DIRECTORY ${PROJECT_SOURCE_DIR}
-					DEPENDS ${PROJECT_NAME})
+add_test(NAME tests COMMAND $<TARGET_FILE:tests>)

test/defines.hpp

@@ -1,11 +1,11 @@
 #include <string>
 
 // Correctly formated files
-static const std::string COMPLEX_DBC_FILE 	= "./dbcs/Complex.dbc";
-static const std::string SIMPLE_DBC_FILE	= "./dbcs/Simple.dbc";
+static const std::string COMPLEX_DBC_FILE 	= std::string(TESTDBCFILES_PATH) + "/Complex.dbc";
+static const std::string SIMPLE_DBC_FILE	= std::string(TESTDBCFILES_PATH) + "/Simple.dbc";
 
 // Files with Errors
-static const std::string MISSING_NEW_SYMBOLS_DBC_FILE	= "./dbcs/MissingNewSymbols.dbc";
-static const std::string MISSING_VERSION_DBC_FILE		= "./dbcs/MissingVersion.dbc";
-static const std::string MISSING_BIT_TIMING_DBC_FILE	= "./dbcs/MissingBitTiming.dbc";
-static const std::string TEXT_FILE 						= "./dbcs/TextFile.txt";
+static const std::string MISSING_NEW_SYMBOLS_DBC_FILE	= std::string(TESTDBCFILES_PATH) + "/MissingNewSymbols.dbc";
+static const std::string MISSING_VERSION_DBC_FILE		= std::string(TESTDBCFILES_PATH) + "/MissingVersion.dbc";
+static const std::string MISSING_BIT_TIMING_DBC_FILE	= std::string(TESTDBCFILES_PATH) + "/MissingBitTiming.dbc";
+static const std::string TEXT_FILE 						= std::string(TESTDBCFILES_PATH) + "/TextFile.txt";