Re: [PATCH] send-email: add test for Linux's get_maintainer.pl
- From
Alex Bennée <alex.bennee@linaro.org>
- Date
- Jan 5, 2018, 11:15 UTC
- Message-ID
- <87o9m8d09u.fsf@linaro.org>
- In-Reply-To
- <1515147109-8077-1-git-send-email-git@matthieu-moy.fr>
Matthieu Moy <git@matthieu-moy.fr> writes:
Show 46 quoted lines
> From: Alex Bennée <alex.bennee@linaro.org> > > We had a regression that broke Linux's get_maintainer.pl. Using > Mail::Address to parse email addresses fixed it, but let's protect > against future regressions. > > Patch-edited-by: Matthieu Moy <git@matthieu-moy.fr> > Signed-off-by: Alex Bennée <alex.bennee@linaro.org> > Signed-off-by: Matthieu Moy <git@matthieu-moy.fr> > --- > t/t9001-send-email.sh | 22 ++++++++++++++++++++++ > 1 file changed, 22 insertions(+) > > diff --git a/t/t9001-send-email.sh b/t/t9001-send-email.sh > index 4d261c2..f126177 100755 > --- a/t/t9001-send-email.sh > +++ b/t/t9001-send-email.sh > @@ -172,6 +172,28 @@ test_expect_success $PREREQ 'cc trailer with various syntax' ' > test_cmp expected-cc commandline1 > ' > > +test_expect_success $PREREQ 'setup fake get_maintainer.pl script for cc trailer' " > + cat >expected-cc-script.sh <<-EOF && > + #!/bin/sh > + echo 'One Person <one@example.com> (supporter:THIS (FOO/bar))' > + echo 'Two Person <two@example.com> (maintainer:THIS THING)' > + echo 'Third List <three@example.com> (moderated list:THIS THING (FOO/bar))' > + echo '<four@example.com> (moderated list:FOR THING)' > + echo 'five@example.com (open list:FOR THING (FOO/bar))' > + echo 'six@example.com (open list)' > + EOF > + chmod +x expected-cc-script.sh > +" > + > +test_expect_success $PREREQ 'cc trailer with get_maintainer.pl output' ' > + test_commit cc-trailer-getmaint && > + clean_fake_sendmail && > + git send-email -1 --to=recipient@example.com \ > + --cc-cmd="./expected-cc-script.sh" \ > + --smtp-server="$(pwd)/fake.sendmail" && > + test_cmp expected-cc commandline1 > +' > + > test_expect_success $PREREQ 'setup expect' " > cat >expected-show-all-headers <<\EOF > 0001-Second.patch
I think you need to apply Eric's suggestions from:
From: Eric Sunshine <sunshine@sunshineco.com> Date: Sat, 18 Nov 2017 21:54:46 -0500 Message-ID: <CAPig+cSh0tVVkh0xF9FwCfM4gngAWMSN_FXd2zhzHcy2trYXfw@mail.gmail.com>
-- Alex Bennée