Attention is currently required from: Martin Roth, Jakub Czapiga, Angel Pons. Paul Fagerburg has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/52444 )
Change subject: tests: enable code coverage for unit tests ......................................................................
Patch Set 2:
(2 comments)
File tests/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/52444/comment/2fad1044_b21c4666 PS1, Line 164: coverage-unit-tests: unit-tests : lcov -o build/tests/tests.info -c -d build/tests --exclude '*/tests/*' : genhtml -q -o build/tests/coverage_rpt -t "coreboot unit tests" \ : -s build/tests/tests.info
Correction, I do not mean that this is a must-have. […]
I agree, it would be very nice to have. This is just a first iteration, and I'm sure we'll be adding more features later on.
File tests/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/52444/comment/ef5bf1c8_7ca78d95 PS2, Line 200:
nit: I'd either use tabs or spaces, but not both
My editor decided to use tabs because it's a makefile, Thanks for catching that.