Skip to content

Commit

Permalink
Merge pull request #20 from jphickey/fix-19-ut-coverage-flags
Browse files Browse the repository at this point in the history
Fix #19, Update coverage compile/link flag options
  • Loading branch information
astrogeco authored Apr 28, 2020
2 parents a561fbf + 78edacf commit b93753b
Showing 1 changed file with 4 additions and 7 deletions.
11 changes: 4 additions & 7 deletions unit-test/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -62,10 +62,9 @@ foreach(SRCFILE ${LIB_SRC_FILES})
${UNIT_SOURCE_FILE}
)

# Apply the UT_C_FLAGS to the units under test
# Apply the UT_COVERAGE_COMPILE_FLAGS to the units under test
# This should enable coverage analysis on platforms that support this
set_target_properties(ut_${TESTNAME}_object PROPERTIES
COMPILE_FLAGS "${UT_C_FLAGS}")
target_compile_options(ut_${TESTNAME}_object PRIVATE ${UT_COVERAGE_COMPILE_FLAGS})

# For this object target only, the "override" includes should be injected
# into the include path BEFORE any other include path. This is so the
Expand All @@ -80,13 +79,11 @@ foreach(SRCFILE ${LIB_SRC_FILES})
$<TARGET_OBJECTS:ut_${TESTNAME}_object>
)

# This also needs to be linked with UT_C_FLAGS (for coverage)
set_target_properties(${TESTNAME}-testrunner PROPERTIES
LINK_FLAGS "${UT_C_FLAGS}")

# This also needs to be linked with UT_COVERAGE_LINK_FLAGS (for coverage)
# This is also linked with any other stub libraries needed,
# as well as the UT assert framework
target_link_libraries(${TESTNAME}-testrunner
${UT_COVERAGE_LINK_FLAGS}
ut_${UT_NAME}_stubs
ut_${UT_NAME}_overrides
ut_cfe-core_stubs
Expand Down

0 comments on commit b93753b

Please sign in to comment.