diff options
author | Pranit Bauva <pranit.bauva@gmail.com> | 2017-01-03 22:57:07 +0300 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2017-01-08 00:20:01 +0300 |
commit | c7cf956618c4af9f3151a90d7552b05767e6444f (patch) | |
tree | ea074b87fcb872c13df8b0b5d2c3d2b12c5a89d6 /t/t9001-send-email.sh | |
parent | a274e0a036ea886a31f8b216564ab1b4a3142f6c (diff) |
don't use test_must_fail with grep
test_must_fail should only be used for testing git commands. To test the
failure of other commands use `!`.
Reported-by: Stefan Beller <sbeller@google.com>
Signed-off-by: Pranit Bauva <pranit.bauva@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t9001-send-email.sh')
-rwxr-xr-x | t/t9001-send-email.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/t/t9001-send-email.sh b/t/t9001-send-email.sh index 3dc4a3454d..0f398dd160 100755 --- a/t/t9001-send-email.sh +++ b/t/t9001-send-email.sh @@ -50,7 +50,7 @@ test_no_confirm () { --smtp-server="$(pwd)/fake.sendmail" \ $@ \ $patches >stdout && - test_must_fail grep "Send this email" stdout && + ! grep "Send this email" stdout && >no_confirm_okay } |