Junio C Hamano <gitster@xxxxxxxxx> writes:
> I am not sure if three combinations (vanilla, -M and -C -C) need to be
> tested to produce an empty result. If so, it would make it easier to read
> if they are split into three tests, or at least have a blank line between
> them, but I suspect that you would agree that it is not worth to have
> three separate test_expect_success for these.
I think it makes sense to cull these three cases into one for the case we
expect the command to stop without doing anything, but we would still want
to validate the output for three variants in the "config" case.
Also I forgot to say that the new "grep" invocations added to check the
error output might have to be test_i18ngrep. Please check with
make GETTEXT_POISON=YesPlease test
The configuration variable names and typo in user input should appear
somewhere in the output for any real locale, but I think gettext-poison
would throw these away.
By the way, should the following two entries make any difference, and if
so how?
[diff]
dirstat = unknown,0,lines
dirstat = 0,lines,unknown
--
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