]> granicus.if.org Git - git/commitdiff
t4151: consolidate multiple calls to test_i18ngrep
authorRamsay Jones <ramsay@ramsayjones.plus.com>
Mon, 12 Feb 2018 00:17:20 +0000 (00:17 +0000)
committerJunio C Hamano <gitster@pobox.com>
Mon, 12 Feb 2018 20:01:52 +0000 (12:01 -0800)
Attempting to grep the output of test_i18ngrep will not work under a
poison build, since the output is (almost) guaranteed not to have the
string you are looking for. In this case, we have a test_i18ngrep call
which attempts to filter the contents of a file, which was itself the
result of a call to test_i18ngrep. In this case, we can achieve the
same effect with a single call to test_i18ngrep (without creating the
intermediate file), since the second regular expression can be used
without change to filter the original input.

Also, replace a call to test_i18ncmp with test_cmp, since the content
being compared is not subject to i18n anyway.

Signed-off-by: Ramsay Jones <ramsay@ramsayjones.plus.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/t4151-am-abort.sh

index 9473c2779ef0df109f6943a949583676ea5ceabf..16432781d2e0d52c33dfb1444fcce4459b417e8b 100755 (executable)
@@ -46,9 +46,8 @@ do
 
        test_expect_success "am$with3 --skip continue after failed am$with3" '
                test_must_fail git am$with3 --skip >output &&
-               test_i18ngrep "^Applying" output >output.applying &&
-               test_i18ngrep "^Applying: 6$" output.applying &&
-               test_i18ncmp file-2-expect file-2 &&
+               test_i18ngrep "^Applying: 6$" output &&
+               test_cmp file-2-expect file-2 &&
                test ! -f .git/MERGE_RR
        '