Skip to content
Snippets Groups Projects
Commit 425d7646 authored by Zack Galbreath's avatar Zack Galbreath Committed by Brad King
Browse files

CTestCoverageCollectGCOV: Honor CTEST_EXTRA_COVERAGE_GLOB

Teach CTestCoverageCollectGCOV to honor the
CTEST_EXTRA_COVERAGE_GLOB variable.  When this variable is set,
this module will glob for matching source files that were not
covered and include them in the resulting tar file.
parent 00e4d122
No related branches found
No related tags found
No related merge requests found
......@@ -172,6 +172,21 @@ function(ctest_coverage_collect_gcov)
set(unfiltered_gcov_files)
file(GLOB_RECURSE unfiltered_gcov_files RELATIVE ${binary_dir} "${coverage_dir}/*.gcov")
# if CTEST_EXTRA_COVERAGE_GLOB was specified we search for files
# that might be uncovered
if (DEFINED CTEST_EXTRA_COVERAGE_GLOB)
set(uncovered_files)
foreach(search_entry IN LISTS CTEST_EXTRA_COVERAGE_GLOB)
if(NOT GCOV_QUIET)
message("Add coverage glob: ${search_entry}")
endif()
file(GLOB_RECURSE matching_files "${source_dir}/${search_entry}")
if (matching_files)
list(APPEND uncovered_files "${matching_files}")
endif()
endforeach()
endif()
set(gcov_files)
foreach(gcov_file ${unfiltered_gcov_files})
file(STRINGS ${binary_dir}/${gcov_file} first_line LIMIT_COUNT 1 ENCODING UTF-8)
......@@ -195,20 +210,62 @@ function(ctest_coverage_collect_gcov)
endif()
endforeach()
get_filename_component(resolved_source_file "${source_file}" ABSOLUTE)
foreach(uncovered_file IN LISTS uncovered_files)
get_filename_component(resolved_uncovered_file "${uncovered_file}" ABSOLUTE)
if (resolved_uncovered_file STREQUAL resolved_source_file)
list(REMOVE_ITEM uncovered_files "${uncovered_file}")
endif()
endforeach()
if(NOT is_excluded)
list(APPEND gcov_files ${gcov_file})
endif()
endforeach()
foreach (uncovered_file ${uncovered_files})
# Check if this uncovered file should be excluded.
set(is_excluded false)
foreach(exclude_entry IN LISTS CTEST_CUSTOM_COVERAGE_EXCLUDE)
if(uncovered_file MATCHES "${exclude_entry}")
set(is_excluded true)
if(NOT GCOV_QUIET)
message("Excluding coverage for: ${uncovered_file} which matches ${exclude_entry}")
endif()
break()
endif()
endforeach()
if(is_excluded)
continue()
endif()
# Copy from source to binary dir, preserving any intermediate subdirectories.
get_filename_component(filename "${uncovered_file}" NAME)
get_filename_component(relative_path "${uncovered_file}" DIRECTORY)
string(REPLACE "${source_dir}" "" relative_path "${relative_path}")
if (relative_path)
# Strip leading slash.
string(SUBSTRING "${relative_path}" 1 -1 relative_path)
endif()
file(COPY ${uncovered_file} DESTINATION ${binary_dir}/uncovered/${relative_path})
if(relative_path)
list(APPEND uncovered_files_for_tar uncovered/${relative_path}/${filename})
else()
list(APPEND uncovered_files_for_tar uncovered/${filename})
endif()
endforeach()
# tar up the coverage info with the same date so that the md5
# sum will be the same for the tar file independent of file time
# stamps
string(REPLACE ";" "\n" gcov_files "${gcov_files}")
string(REPLACE ";" "\n" label_files "${label_files}")
string(REPLACE ";" "\n" uncovered_files_for_tar "${uncovered_files_for_tar}")
file(WRITE "${coverage_dir}/coverage_file_list.txt"
"${gcov_files}
${coverage_dir}/data.json
${label_files}
${uncovered_files_for_tar}
")
if (GCOV_QUIET)
......
......@@ -15,7 +15,9 @@ list(APPEND CTEST_CUSTOM_COVERAGE_EXCLUDE
"/foo/something"
"/3rdparty/"
"/bar/somethingelse"
"/CMakeFiles/"
)
list(APPEND CTEST_EXTRA_COVERAGE_GLOB "*.cpp")
include(CTestCoverageCollectGCOV)
set(tar_file ${CTEST_BINARY_DIRECTORY}/gcov.tar)
......@@ -41,6 +43,8 @@ set(expected_out
Testing/CoverageInfo/data.json
Testing/CoverageInfo/extra.cpp.gcov
Testing/CoverageInfo/main.cpp.gcov
uncovered/extra/uncovered1.cpp
uncovered/uncovered2.cpp
)
if("${out}" STREQUAL "${expected_out}")
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment