diff options
author | Junio C Hamano <gitster@pobox.com> | 2021-05-25 10:21:19 +0300 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2021-05-25 10:21:19 +0300 |
commit | 53cb2103ce9a1bb5239f5a9cf74a98ef568bb0ea (patch) | |
tree | d5cbb31d6363968a6869646fdf50c8b2cd93ab46 /t | |
parent | de88ac70f3a801262eb3aa087e5d9a712be0a54a (diff) | |
parent | 5b719b7552b9eca486a3a6507e9397572d16481d (diff) |
Merge branch 'ab/send-email-validate-errors-fix'
* ab/send-email-validate-errors-fix:
send-email: fix missing error message regression
Diffstat (limited to 't')
-rwxr-xr-x | t/t9001-send-email.sh | 23 |
1 files changed, 21 insertions, 2 deletions
diff --git a/t/t9001-send-email.sh b/t/t9001-send-email.sh index 65b3035371..2acf389837 100755 --- a/t/t9001-send-email.sh +++ b/t/t9001-send-email.sh @@ -644,14 +644,33 @@ test_expect_success $PREREQ 'In-Reply-To with --chain-reply-to' ' test_cmp expect actual ' +test_set_editor "$(pwd)/fake-editor" + +test_expect_success $PREREQ 'setup erroring fake editor' ' + write_script fake-editor <<-\EOF + echo >&2 "I am about to error" + exit 1 + EOF +' + +test_expect_success $PREREQ 'fake editor dies with error' ' + clean_fake_sendmail && + test_must_fail git send-email \ + --compose --subject foo \ + --from="Example <nobody@example.com>" \ + --to=nobody@example.com \ + --smtp-server="$(pwd)/fake.sendmail" \ + $patches 2>err && + grep "I am about to error" err && + grep "the editor exited uncleanly, aborting everything" err +' + test_expect_success $PREREQ 'setup fake editor' ' write_script fake-editor <<-\EOF echo fake edit >>"$1" EOF ' -test_set_editor "$(pwd)/fake-editor" - test_expect_success $PREREQ '--compose works' ' clean_fake_sendmail && git send-email \ |