Re: [PATCH v3] diff-no-index: exit(1) if 'diff --quiet <repo file> <external file>' finds changes

[Tim Henigan <tim.henigan@xxxxxxxxx>]

On Wed, Jun 20, 2012 at 12:06 PM, Jeff King <peff@xxxxxxxx> wrote:
> On Wed, Jun 20, 2012 at 09:38:15AM -0400, Tim Henigan wrote:
>>
>> Are you planning to send this patch to the list?  If not, can I
>> include it as 1 of 2 before my patch?  If we go that route, I'm not
>> sure how to properly show you as the author...
>
> I'd probably get to it eventually, but I haven't touched it since I sent
> it. If you want to include some tests and package it with a commit
> message, that would make me very happy.

Thanks, I will do that.

It looks like the best place to add tests is t4010-diff-pathspec.sh.
The only cases not tested through other means appear to be:

    git diff <file in repo> <relative path outside repo>
    git diff <relative path outside repo> <relative path outside repo>

Other pathspec variations seem to be covered extensively by other
tests (mostly as a side effect).  Am I missing other variations that
should be checked?

In both cases shown above, we are simply verifying that giving a
relative path to diff does not cause it to abort.  So it may be
sufficient to only test one of the above.

The tests that I added to t4035-diff-quiet.sh already cover both of
the cases listed above.  Is it worthwhile to duplicate some of those
tests in t4010?
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html