I couldn't get fuzz coverage to work, so couldn't verify what it would look like if made.
docs: Add reference to total.coverage report #28673
pull instagibbs wants to merge 1 commits into bitcoin:master from instagibbs:cov_docs changing 1 files +3 −1-
instagibbs commented at 1:28 PM on October 18, 2023: member
-
docs: Add reference to total.coverage report 2ce7e31d2d
-
DrahtBot commented at 1:28 PM on October 18, 2023: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--006a51241073e994b41acfe9ec718e94-->
Code Coverage
For detailed information about the code coverage, see the test coverage report.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process.
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
- DrahtBot added the label Docs on Oct 18, 2023
- dergoegge approved
-
dergoegge commented at 1:34 PM on October 18, 2023: member
ACK 2ce7e31d2d6bf0e4d831e35563bcf90c68c39650
-
darosior commented at 7:19 AM on October 19, 2023: member
ACK 2ce7e31d2d6bf0e4d831e35563bcf90c68c39650
I couldn't get fuzz coverage to work
I personally use LLVM's source-based coverage along with
grcovto generate the report. -
maflcko commented at 7:26 AM on October 19, 2023: member
I couldn't get fuzz coverage to work
It is a bit more involved, because you'll need to clone the fuzz inputs into a "magic" (hard-coded) location. Other than that, make sure to compile with libfuzzer.
-
maflcko commented at 7:28 AM on October 19, 2023: member
Something like this should work:
https://github.com/maflcko/b-c-cov/blob/83bc0912e2a33cfac2195645ab3a12d092fa7ba3/.cirrus.yml#L65-L69
and give:
- fanquake merged this on Oct 19, 2023
- fanquake closed this on Oct 19, 2023
- Frank-GER referenced this in commit 07a669f825 on Oct 21, 2023
- bitcoin locked this on Oct 18, 2024
