Attention is currently required from: Martin Roth, Paul Fagerburg, Angel Pons. Jakub Czapiga has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/52444 )
Change subject: tests: enable code coverage for unit tests ......................................................................
Patch Set 3:
(2 comments)
File tests/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/52444/comment/9af40bed_26158f50 PS3, Line 165: build/tests I think you can use $(testobj) instead.
If you want to enable one to change coverage obj directory you can introduce variable e.g. TEST_OBJ_DIR (command-line argument) and use it to overwrite $(testobj) at the beginning of this makefile.
https://review.coreboot.org/c/coreboot/+/52444/comment/23033fc1_23b47d18 PS3, Line 205: Generate a code coverage report This might be misleading. This target does not only generate coverage report but builds all tests with --coverage flag. I think this description should include all of its behavior.