Re: [PATCH v2] t4015: improve coverage of function context test

On Thu, Dec 19, 2019 at 06:35:43PM +0100, René Scharfe wrote:

Add a test that includes an actual function line in the test file to
check if context is expanded to include the whole function, and add an
ignored change before function context to check if that one stays hidden
while the originally ignored change within function context is shown.

Helped-by: Jeff King [off-list ref]
Signed-off-by: René Scharfe <redacted>

Thanks, the patch looks good.

Maybe worth noting in the commit message why there are two tests.
Something like:

  This differs from the existing test, which is concerned with the case
  where there is no function line at all in the file (and we might look
  past the beginning of the file).

-Peff