Re: [PATCH] fsck: check skiplist for object in fsck_blob()

[Jeff King <peff@peff.net>]

On Fri, Jul 13, 2018 at 03:41:19PM -0400, Jeff King wrote:

> >   not ok 18 - push rejects corrupt .gitmodules (policy)
> >   #	
> >   #		rm -rf dst.git &&
> >   #		git init --bare dst.git &&
> >   #		git -C dst.git config transfer.fsckObjects true &&
> >   #		git -C dst.git config fsck.gitmodulesParse error &&
> >   #		test_must_fail git -C corrupt push ../dst.git HEAD 2>output &&
> >   #		grep gitmodulesParse output &&
> >   #		test_i18ngrep ! "bad config" output
> 
> There are separate config slots for local fsck versus receiving objects.
> So I think you need to be setting receive.fsck.gitmodulesParse.

I confirmed that s/fsck/receive.fsck/ in your patch makes the tests
pass.

I didn't bother adding extra push tests in the patch I just sent, since
upgrading of config error classes is already covered elsewhere in t5504.

That said, I'm not opposed to adding more tests on top even if they are
slightly redundant (well, not redundant if you're into black-box
testing, but our current tests are usually written with an assumption of
where the module boundaries are, and what is likely to be a problem).

-Peff