Attention is currently required from: Thomas Heijligen, Evan Benn.
Anastasia Klimchuk has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/69268 )
Change subject: tests: Add llvm-cov option and run target for code coverage ......................................................................
Patch Set 11:
(2 comments)
Commit Message:
https://review.coreboot.org/c/flashrom/+/69268/comment/6b315a32_75a571f8 PS11, Line 14: TEST=meson test; ninja llvm-cov-tests Can you test scenario when feature disables? when tests disabled? Obviously, when feature disabled then coverage is not measured. The point of testing is to check flashrom builds and runs normally (without coverage and/or tests). And please add those to the test scenarios in commit message. Thanks!
File meson.build:
https://review.coreboot.org/c/flashrom/+/69268/comment/12518494_58fff455 PS5, Line 534: if get_option('llvm_cov')
coverage can be used independently of unit tests
I am confused, but what does it mean, coverage without running tests?