On 8/6/2014 9:02 PM, Matthieu Moy wrote:
> Tanay Abhra <tanay...@gmail.com> writes:
>> +test_expect_success 'find string value for a key' '
>> +    check_config get_string case.baz hask &&
>> +    check_config expect_code 1 get_string case.ba "Value not found for 
>> \"case.ba\""
>> +'
>> +
>> +test_expect_success 'check line error when NULL string is queried' '
>> +    test_expect_code 128 test-config get_string case.foo 2>result &&
>> +    grep "line 7.*.git/config\|.git/config.*line 7" result
>> +'
> This is still dependant on the locale ("line" is translated). You need
> to use test_i18ngrep instead of grep here (see its definition and
> comment in t/test-lib.sh).

Oh, and I was searching t/README for a hint. Maybe we should add a line there
to hint future readers.
Thanks. :)

> I don't think you need these two alternatives OTOH.
> BTW, Junio, I don't understand your remark "This test is too tight (the
> full string)" in the previous iteration. Can you elaborate?

I think he meant we must search for the relevant snippets instead of the whole 
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to