Re: Git Test Coverage Report (Friday, Oct 12)

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Oct 15, 2018, 08:09 UTC
Message-ID: <nycvar.QRO.7.76.6.1810151007430.561@tvgsbejvaqbjf.bet>
In-Reply-To: <e30a9c05-87d8-1f2b-182c-6d6a5fefe43c@gmail.com>

Hi Stolee,

On Fri, 12 Oct 2018, Derrick Stolee wrote:

Show 5 quoted lines

> 
> In an effort to ensure new code is reasonably covered by the test suite, we
> now have contrib/coverage-diff.sh to combine the gcov output from 'make
> coverage-test ; make coverage-report' with the output from 'git diff A B' to
> discover _new_ lines of code that are not covered.

Thanks for doing this. I do notice, though, that there are a few mentions of BUG() lines, e.g.

> 0af129b2ed builtin/rebase--interactive2.c 267) BUG("invalid command '%d'",
> command);

I do not think that we will ever want to ensure that all of these lines get code coverage ;-) Maybe it would be possible to exclude those lines from the analysis?

Thank you, Dscho