awesomewm-mirrors
/
awesome
mirror of https://github.com/awesomeWM/awesome.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

cmake: remove check-unit-coverage (#2568) 

Just configure `check-unit` to use `--coverage` with `busted` with
`DO_COVERAGE=1`.

I got confused why `check-unit` would not generate coverage.
This keeps it in line with other targets that generate coverage when
used with the same name.
This commit is contained in:
Daniel Hahler 2019-01-10 13:30:58 +01:00 committed by GitHub
parent 7e68305f98
commit b392e2ce74
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 7 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
.travis.yml
View File

@ -184,7 +184,7 @@ script:
&& make)
do_codecov samples
make check-unit-coverage || exit 1
make check-unit || exit 1
do_codecov unittests
make check-integration || exit 1
make check-themes || exit 1

15
CMakeLists.txt
View File

@ -424,20 +424,17 @@ add_custom_target(check-requires
list(APPEND CHECK_QA_TARGETS check-requires)
a_find_program(BUSTED_EXECUTABLE busted FALSE)
if(BUSTED_EXECUTABLE)
# Keep the arguments in sync with the version below!
if(DO_COVERAGE)
set(BUSTED_ARGS "--coverage")
else()
set(BUSTED_ARGS)
endif()
add_custom_target(check-unit
${BUSTED_EXECUTABLE}
COMMAND ${BUSTED_EXECUTABLE} ${BUSTED_ARGS}
WORKING_DIRECTORY ${CMAKE_SOURCE_DIR}
COMMENT "Running unit tests"
VERBATIM)
list(APPEND CHECK_TARGETS check-unit)
# Same as above, but with --coverage argument
add_custom_target(check-unit-coverage
${BUSTED_EXECUTABLE} "--coverage"
WORKING_DIRECTORY ${CMAKE_SOURCE_DIR}
COMMENT "Running unit tests under LuaCov"
VERBATIM)
else()
add_custom_target(check-unit true
COMMENT "Skipping check-unit, since busted was not found"