fsfw/fsfw
fsfw/fsfw
10
3
Fork 6
Code Issues 31 Pull Requests 4 Releases 10 Wiki Activity

Tests can now be built as part of FSFW and versioning moved to CMake #500

Merged
mohr merged 33 commits from KSat/fsfw:mueller/integrated-unittests into development 2021-10-18 14:42:53 +02:00
Conversation 6 Commits 33 Files Changed 56 +2
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 348975ba5f - Show all commits

2
CMakeLists.txt
View File

@ -194,6 +194,8 @@ if(FSFW_BUILD_UNITTESTS)
elseif(UNIX)
set(COVERAGE_EXCLUDES
"/usr/include/*" "/usr/bin/*" "Catch2/*"
mohr commented 2021-10-11 17:15:27 +02:00
Review
Copy Link

for me, "*/catch2-src/*" was needed to get catch2 out of the reports.

How do you feel about adding "*/fsfw_tests/*" as well, seems to be a bit too meta to coverage the coverage tests.

for me, `"*/catch2-src/*"` was needed to get catch2 out of the reports. How do you feel about adding `"*/fsfw_tests/*"` as well, seems to be a bit too meta to coverage the coverage tests.
muellerr commented 2021-10-11 17:26:29 +02:00
Review
Copy Link

I actually installed catch2 and did not have that issue, I can add that directive. Adding the tests makes sense as well

I actually installed catch2 and did not have that issue, I can add that directive. Adding the tests makes sense as well
muellerr commented 2021-10-11 17:31:22 +02:00
Review
Copy Link

Added the excludes

Added the excludes
mohr commented 2021-10-11 17:47:18 +02:00
Review
Copy Link

missing a leading * as in "*/catch2-src/*", as catch2 is stashed away somewhere by cmake.

missing a leading * as in `"*/catch2-src/*"`, as catch2 is stashed away somewhere by cmake.
muellerr commented 2021-10-11 17:52:35 +02:00
Review
Copy Link

Done

Done
"/usr/local/include/*" "*/fsfw_tests/*"
"/catch2-src/*"
)
endif()