Fix #305, do not override add_psp_module for coverage test

Implements a differently-named function instead, which does mostly the same thing,
but does not directly re-add the original subdirectory.  This does repeat some
info (e.g. the list of FSW source files) that was not repeated otherwise, but
it works in general.

unit-test-coverage/modules/CMakeLists.txt

@@ -4,27 +4,33 @@
 #
 ######################################################################
 
-include_directories("${PSPCOVERAGE_SOURCE_DIR}/shared/inc")
-include_directories("${PSPCOVERAGE_SOURCE_DIR}/modules/inc")
 
-# redefine the add_psp_module to make a unit test variant
-# this needs to name the target differently, so it doesn't conflict with the FSW target.
-# this is similar to add_cfe_coverage_test() on the CFE side, but the set of interfaces
-# and link libraries is different from that of a CFS app
-function(add_psp_module MODULE_NAME)
+# Function to simplify adding a coverage test for a PSP module
+# This is very similar to the CFE "add_cfe_coverage" function, but
+# include directories and link libraries are set up differently for PSP code,
+# and it does not depend on having a FSW module by the same name.
+function(add_psp_covtest MODULE_NAME TESTCASE_SRC UT_SRCS)
 
+    # A consistent name convention for all targets generated by this funtion
     set(TEST_NAME "coverage-pspmod-${MODULE_NAME}")
     set(OBJECT_TARGET "${TEST_NAME}-object")
     set(RUNNER_TARGET "${TEST_NAME}-testrunner")
-    # Create the module
 
-    add_library(${OBJECT_TARGET} OBJECT ${ARGN})
+    # Compile the source unit(s) under test as a separate library
+    # This is done so that special coverage-specific compile flags can be used on these files
+    add_library(${OBJECT_TARGET} OBJECT
+        ${UT_SRCS} ${ARGN}
+    )
 
+    # Apply the UT_COVERAGE_COMPILE_FLAGS to the units under test
+    # This should enable coverage analysis on platforms that support this
     target_compile_options(${OBJECT_TARGET} PRIVATE
         ${UT_COVERAGE_COMPILE_FLAGS}
     )
+
     target_include_directories(${OBJECT_TARGET} PRIVATE
         ${PSPCOVERAGE_SOURCE_DIR}/ut-stubs/override_inc # overrides for system headers
+        ${PSPCOVERAGE_SOURCE_DIR}/shared/inc
         ${CFEPSP_SOURCE_DIR}/fsw/inc                    # PSP public API
         ${CFEPSP_SOURCE_DIR}/fsw/shared/inc             # all PSP shared headers
         ${CFE_SOURCE_DIR}/cmake/target/inc              # for sysconfig headers
@@ -34,7 +40,7 @@ function(add_psp_module MODULE_NAME)
     # Compile a test runner application, which contains the
     # actual coverage test code (test cases) and the unit under test
     add_executable(${RUNNER_TARGET}
-        ${PSPCOVERAGE_SOURCE_DIR}/modules/src/coveragetest-${MODULE_NAME}.c
+        ${TESTCASE_SRC}
         $<TARGET_OBJECTS:${OBJECT_TARGET}>
     )
 
@@ -49,19 +55,14 @@ function(add_psp_module MODULE_NAME)
         ut_assert
     )
 
+    # Add it to the set of tests to run as part of "make test"
     add_test(${TEST_NAME} ${RUNNER_TARGET})
     foreach(TGT ${INSTALL_TARGET_LIST})
         install(TARGETS ${RUNNER_TARGET} DESTINATION ${TGT}/${UT_INSTALL_SUBDIR})
     endforeach()
 
-endfunction(add_psp_module)
+endfunction(add_psp_covtest)
 
-# a list of modules for which there is a coverage test implemented
-set(UT_PSPMODLIST
-    timebase_vxworks
-)
-
-foreach(UT_PSPMOD ${UT_PSPMODLIST})
-    add_subdirectory(${CFEPSP_SOURCE_DIR}/fsw/modules/${UT_PSPMOD} ${UT_PSPMOD})
-endforeach()
 
+# a list of modules for which there is a coverage test implemented
+add_subdirectory(timebase_vxworks)

unit-test-coverage/modules/timebase_vxworks/CMakeLists.txt

@@ -0,0 +1,11 @@
+######################################################################
+#
+# CMAKE build recipe for white-box coverage tests of VxWorks timebase module
+#
+######################################################################
+
+include_directories("${CMAKE_CURRENT_SOURCE_DIR}/inc")
+
+add_psp_covtest(timebase_vxworks src/coveragetest-timebase_vxworks.c
+    ${CFEPSP_SOURCE_DIR}/fsw/modules/timebase_vxworks/cfe_psp_timebase_vxworks.c
+)