fuzz: exclude the SDK and CRT from code coverage reporting#23
Conversation
Right now, our coverage sits at 41%. If we exclude the code we did not author, that number should grow to _79%_.
| ! *vctools* | ||
| ! *externalapis* | ||
| ! *internal_shared.h | ||
| ! *type_traits |
There was a problem hiding this comment.
Does this match against a full path? I wonder if there's something we can match against that would capture the whole STL instead.
There was a problem hiding this comment.
Or maybe instead of the global wildcard above, maybe we could capture just our code?
There was a problem hiding this comment.
I wanted to, but I was worried that I didn't know where the source path was rooted from onefuzz's perspective.
(Also, it puts the ExternalAPIs folder in /src (???))
There was a problem hiding this comment.
(rooted: because I wanted to put in src/lib/* rather than */src/lib/*)
There was a problem hiding this comment.
whole STL instead.
Ugh, I hate this honestly. It looks like this:
so somewhat "not really", but also "somewhat"?
There was a problem hiding this comment.
I assume it just pulls source paths from the pdb(?), so it would be specific to the CI machine I'd think. But that's also probably not consistent/guaranteed. Anyway, not that big of a deal regardless
2 participants
Right now, our coverage sits at 41%. If we exclude the code we did not author, that number should grow to 79%.