Git fork

t7407: use test_grep

There are a few grep calls here that can benefit from test_grep, which
produces more user-friendly output when it fails.

One of these calls also passes "-sq", which is curious. The "-q" option
suppresses the matched output. But test output is either already
redirected to /dev/null in non-verbose mode, and in verbose mode it's
better to see the output. The "-s" option suppresses errors opening
files, but we are just grepping in the "expected" file we just
generated, so it should not be needed. Neither of these was really
hurting anything, but they are not a style we'd like to see emulated. So
get rid of them.

(It is also curious to grep in the expected file in the first place, but
that is because we are auto-generating the expectation from a Git
command. So this is double-checking it did what we wanted).

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

authored by

Jeff King and committed by
Junio C Hamano
ddb52878 4f71522d

+2 -2
+2 -2
t/t7407-submodule-foreach.sh
··· 426 426 git submodule foreach "echo be --quiet" > ../expected && 427 427 git submodule foreach echo be --quiet > ../actual 428 428 ) && 429 - grep -sq -e "--quiet" expected && 429 + test_grep -e "--quiet" expected && 430 430 test_cmp expected actual 431 431 ' 432 432 433 433 test_expect_success 'option-like arguments passed to foreach recurse correctly' ' 434 434 git -C clone2 submodule foreach --recursive "echo be --an-option" >expect && 435 435 git -C clone2 submodule foreach --recursive echo be --an-option >actual && 436 - grep -e "--an-option" expect && 436 + test_grep -e "--an-option" expect && 437 437 test_cmp expect actual 438 438 ' 439 439