git/t/t7513-interpret-trailers.sh at 2c3cb52c6c5868410a6aa8f0c6c9ee89dbe4bb15 - mirrors/git - Incest Forge: Beyond sex. We incest.

mirrors/git

mirror of https://github.com/git/git.git synced 2024-10-31 14:27:54 +01:00

Junio C Hamano 9d939886db Merge branch 'cc/trailers-corner-case-fix' into maint

The "interpret-trailers" helper mistook a multi-paragraph title of
a commit log message with a colon in it as the end of the trailer
block.

* cc/trailers-corner-case-fix:
  trailer: support multiline title
  trailer: retitle a test and correct an in-comment message
  trailer: ignore first line of message

2015-09-03 19:18:03 -07:00

922 lines

24 KiB

Bash

Executable file

Raw Blame History

 #!/bin/sh
 #
 # Copyright (c) 2013, 2014 Christian Couder
 #
 test_description='git interpret-trailers'
 . ./test-lib.sh
 # When we want one trailing space at the end of each line, let's use sed
 # to make sure that these spaces are not removed by any automatic tool.
 test_expect_success 'setup' '
 	: >empty &&
 	cat >basic_message <<-\EOF &&
 		subject
 		body
 	EOF
 	cat >complex_message_body <<-\EOF &&
 		my subject
 		my body which is long
 		and contains some special
 		chars like : = ? !
 	EOF
 	sed -e "s/ Z\$/ /" >complex_message_trailers <<-\EOF &&
 		Fixes: Z
 		Acked-by: Z
 		Reviewed-by: Z
 		Signed-off-by: Z
 	EOF
 	cat >basic_patch <<-\EOF
 		---
 		 foo.txt | 2 +-
 file changed, 1 insertion(+), 1 deletion(-)
 		diff --git a/foo.txt b/foo.txt
 		index 0353767..1d91aa1 100644
 		--- a/foo.txt
 		+++ b/foo.txt
@@ -1,3 +1,3 @@
 		-bar
 		+baz
 		--
 .9.rc0.11.ga562ddc
 	EOF
 '
 test_expect_success 'without config' '
 	sed -e "s/ Z\$/ /" >expected <<-\EOF &&
 		ack: Peff
 		Reviewed-by: Z
 		Acked-by: Johan
 	EOF
 	git interpret-trailers --trailer "ack = Peff" --trailer "Reviewed-by" \
 		--trailer "Acked-by: Johan" empty >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'without config in another order' '
 	sed -e "s/ Z\$/ /" >expected <<-\EOF &&
 		Acked-by: Johan
 		Reviewed-by: Z
 		ack: Peff
 	EOF
 	git interpret-trailers --trailer "Acked-by: Johan" --trailer "Reviewed-by" \
 		--trailer "ack = Peff" empty >actual &&
 	test_cmp expected actual
 '
 test_expect_success '--trim-empty without config' '
 	cat >expected <<-\EOF &&
 		ack: Peff
 		Acked-by: Johan
 	EOF
 	git interpret-trailers --trim-empty --trailer ack=Peff \
 		--trailer "Reviewed-by" --trailer "Acked-by: Johan" \
 		--trailer "sob:" empty >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'with config option on the command line' '
 	cat >expected <<-\EOF &&
 		Acked-by: Johan
 		Reviewed-by: Peff
 	EOF
 	{ echo; echo "Acked-by: Johan"; } |
 	git -c "trailer.Acked-by.ifexists=addifdifferent" interpret-trailers \
 		--trailer "Reviewed-by: Peff" --trailer "Acked-by: Johan" >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'with only a title in the message' '
 	cat >expected <<-\EOF &&
 		area: change
 		Reviewed-by: Peff
 		Acked-by: Johan
 	EOF
 	echo "area: change" |
 	git interpret-trailers --trailer "Reviewed-by: Peff" \
 		--trailer "Acked-by: Johan" >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'with multiline title in the message' '
 	cat >expected <<-\EOF &&
 		place of
 		code: change
 		Reviewed-by: Peff
 		Acked-by: Johan
 	EOF
 	printf "%s\n" "place of" "code: change" |
 	git interpret-trailers --trailer "Reviewed-by: Peff" \
 		--trailer "Acked-by: Johan" >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'with config setup' '
 	git config trailer.ack.key "Acked-by: " &&
 	cat >expected <<-\EOF &&
 		Acked-by: Peff
 	EOF
 	git interpret-trailers --trim-empty --trailer "ack = Peff" empty >actual &&
 	test_cmp expected actual &&
 	git interpret-trailers --trim-empty --trailer "Acked-by = Peff" empty >actual &&
 	test_cmp expected actual &&
 	git interpret-trailers --trim-empty --trailer "Acked-by :Peff" empty >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'with config setup and ":=" as separators' '
 	git config trailer.separators ":=" &&
 	git config trailer.ack.key "Acked-by= " &&
 	cat >expected <<-\EOF &&
 		Acked-by= Peff
 	EOF
 	git interpret-trailers --trim-empty --trailer "ack = Peff" empty >actual &&
 	test_cmp expected actual &&
 	git interpret-trailers --trim-empty --trailer "Acked-by= Peff" empty >actual &&
 	test_cmp expected actual &&
 	git interpret-trailers --trim-empty --trailer "Acked-by : Peff" empty >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'with config setup and "%" as separators' '
 	git config trailer.separators "%" &&
 	cat >expected <<-\EOF &&
 		bug% 42
 		count% 10
 		bug% 422
 	EOF
 	git interpret-trailers --trim-empty --trailer "bug = 42" \
 		--trailer count%10 --trailer "test: stuff" \
 		--trailer "bug % 422" empty >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'with "%" as separators and a message with trailers' '
 	cat >special_message <<-\EOF &&
 		Special Message
 		bug% 42
 		count% 10
 		bug% 422
 	EOF
 	cat >expected <<-\EOF &&
 		Special Message
 		bug% 42
 		count% 10
 		bug% 422
 		count% 100
 	EOF
 	git interpret-trailers --trailer count%100 \
 		special_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'with config setup and ":=#" as separators' '
 	git config trailer.separators ":=#" &&
 	git config trailer.bug.key "Bug #" &&
 	cat >expected <<-\EOF &&
 		Bug #42
 	EOF
 	git interpret-trailers --trim-empty --trailer "bug = 42" empty >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'with commit basic message' '
 	cat basic_message >expected &&
 	echo >>expected &&
 	git interpret-trailers <basic_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'with basic patch' '
 	cat basic_message >input &&
 	cat basic_patch >>input &&
 	cat basic_message >expected &&
 	echo >>expected &&
 	cat basic_patch >>expected &&
 	git interpret-trailers <input >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'with commit complex message as argument' '
 	cat complex_message_body complex_message_trailers >complex_message &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Fixes: Z
 		Acked-by= Z
 		Reviewed-by: Z
 		Signed-off-by: Z
 	EOF
 	git interpret-trailers complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'with 2 files arguments' '
 	cat basic_message >>expected &&
 	echo >>expected &&
 	cat basic_patch >>expected &&
 	git interpret-trailers complex_message input >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'with message that has comments' '
 	cat basic_message >message_with_comments &&
 	sed -e "s/ Z\$/ /" >>message_with_comments <<-\EOF &&
 		# comment
 		# other comment
 		Cc: Z
 		# yet another comment
 		Reviewed-by: Johan
 		Reviewed-by: Z
 		# last comment
 	EOF
 	cat basic_patch >>message_with_comments &&
 	cat basic_message >expected &&
 	cat >>expected <<-\EOF &&
 		# comment
 		Reviewed-by: Johan
 		Cc: Peff
 		# last comment
 	EOF
 	cat basic_patch >>expected &&
 	git interpret-trailers --trim-empty --trailer "Cc: Peff" message_with_comments >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'with message that has an old style conflict block' '
 	cat basic_message >message_with_comments &&
 	sed -e "s/ Z\$/ /" >>message_with_comments <<-\EOF &&
 		# comment
 		# other comment
 		Cc: Z
 		# yet another comment
 		Reviewed-by: Johan
 		Reviewed-by: Z
 		# last comment
 		Conflicts:
 	EOF
 	cat basic_message >expected &&
 	cat >>expected <<-\EOF &&
 		# comment
 		Reviewed-by: Johan
 		Cc: Peff
 		# last comment
 		Conflicts:
 	EOF
 	git interpret-trailers --trim-empty --trailer "Cc: Peff" message_with_comments >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'with commit complex message and trailer args' '
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Fixes: Z
 		Acked-by= Z
 		Reviewed-by: Z
 		Signed-off-by: Z
 		Acked-by= Peff
 		Bug #42
 	EOF
 	git interpret-trailers --trailer "ack: Peff" \
 		--trailer "bug: 42" <complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'with complex patch, args and --trim-empty' '
 	cat complex_message >complex_patch &&
 	cat basic_patch >>complex_patch &&
 	cat complex_message_body >expected &&
 	cat >>expected <<-\EOF &&
 		Acked-by= Peff
 		Bug #42
 	EOF
 	cat basic_patch >>expected &&
 	git interpret-trailers --trim-empty --trailer "ack: Peff" \
 		--trailer "bug: 42" <complex_patch >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'using "where = before"' '
 	git config trailer.bug.where "before" &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Bug #42
 		Fixes: Z
 		Acked-by= Z
 		Reviewed-by: Z
 		Signed-off-by: Z
 		Acked-by= Peff
 	EOF
 	git interpret-trailers --trailer "ack: Peff" \
 		--trailer "bug: 42" complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'using "where = after"' '
 	git config trailer.ack.where "after" &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Bug #42
 		Fixes: Z
 		Acked-by= Z
 		Acked-by= Peff
 		Reviewed-by: Z
 		Signed-off-by: Z
 	EOF
 	git interpret-trailers --trailer "ack: Peff" \
 		--trailer "bug: 42" complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'using "where = end"' '
 	git config trailer.review.key "Reviewed-by" &&
 	git config trailer.review.where "end" &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Fixes: Z
 		Acked-by= Z
 		Acked-by= Peff
 		Reviewed-by: Z
 		Signed-off-by: Z
 		Reviewed-by: Junio
 		Reviewed-by: Johannes
 	EOF
 	git interpret-trailers --trailer "ack: Peff" \
 		--trailer "Reviewed-by: Junio" --trailer "Reviewed-by: Johannes" \
 		complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'using "where = start"' '
 	git config trailer.review.key "Reviewed-by" &&
 	git config trailer.review.where "start" &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Reviewed-by: Johannes
 		Reviewed-by: Junio
 		Fixes: Z
 		Acked-by= Z
 		Acked-by= Peff
 		Reviewed-by: Z
 		Signed-off-by: Z
 	EOF
 	git interpret-trailers --trailer "ack: Peff" \
 		--trailer "Reviewed-by: Junio" --trailer "Reviewed-by: Johannes" \
 		complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'using "where = before" for a token in the middle of the message' '
 	git config trailer.review.key "Reviewed-by:" &&
 	git config trailer.review.where "before" &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Bug #42
 		Fixes: Z
 		Acked-by= Z
 		Acked-by= Peff
 		Reviewed-by:Johan
 		Reviewed-by:
 		Signed-off-by: Z
 	EOF
 	git interpret-trailers --trailer "ack: Peff" --trailer "bug: 42" \
 		--trailer "review: Johan" <complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'using "where = before" and --trim-empty' '
 	cat complex_message_body >expected &&
 	cat >>expected <<-\EOF &&
 		Bug #46
 		Bug #42
 		Acked-by= Peff
 		Reviewed-by:Johan
 	EOF
 	git interpret-trailers --trim-empty --trailer "ack: Peff" \
 		--trailer "bug: 42" --trailer "review: Johan" \
 		--trailer "Bug: 46" <complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'the default is "ifExists = addIfDifferentNeighbor"' '
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Bug #42
 		Fixes: Z
 		Acked-by= Z
 		Acked-by= Peff
 		Acked-by= Junio
 		Acked-by= Peff
 		Reviewed-by:
 		Signed-off-by: Z
 	EOF
 	git interpret-trailers --trailer "ack: Peff" --trailer "review:" \
 		--trailer "ack: Junio" --trailer "bug: 42" --trailer "ack: Peff" \
 		--trailer "ack: Peff" <complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'default "ifExists" is now "addIfDifferent"' '
 	git config trailer.ifexists "addIfDifferent" &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Bug #42
 		Fixes: Z
 		Acked-by= Z
 		Acked-by= Peff
 		Acked-by= Junio
 		Reviewed-by:
 		Signed-off-by: Z
 	EOF
 	git interpret-trailers --trailer "ack: Peff" --trailer "review:" \
 		--trailer "ack: Junio" --trailer "bug: 42" --trailer "ack: Peff" \
 		--trailer "ack: Peff" <complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'using "ifExists = addIfDifferent" with "where = end"' '
 	git config trailer.ack.ifExists "addIfDifferent" &&
 	git config trailer.ack.where "end" &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Bug #42
 		Fixes: Z
 		Acked-by= Z
 		Reviewed-by:
 		Signed-off-by: Z
 		Acked-by= Peff
 	EOF
 	git interpret-trailers --trailer "ack: Peff" --trailer "review:" \
 		--trailer "bug: 42" --trailer "ack: Peff" \
 		<complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'using "ifExists = addIfDifferent" with "where = before"' '
 	git config trailer.ack.ifExists "addIfDifferent" &&
 	git config trailer.ack.where "before" &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Bug #42
 		Fixes: Z
 		Acked-by= Peff
 		Acked-by= Z
 		Reviewed-by:
 		Signed-off-by: Z
 	EOF
 	git interpret-trailers --trailer "ack: Peff" --trailer "review:" \
 		--trailer "bug: 42" --trailer "ack: Peff" \
 		<complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'using "ifExists = addIfDifferentNeighbor" with "where = end"' '
 	git config trailer.ack.ifExists "addIfDifferentNeighbor" &&
 	git config trailer.ack.where "end" &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Bug #42
 		Fixes: Z
 		Acked-by= Z
 		Reviewed-by:
 		Signed-off-by: Z
 		Acked-by= Peff
 		Acked-by= Junio
 		Tested-by: Jakub
 		Acked-by= Junio
 		Acked-by= Peff
 	EOF
 	git interpret-trailers --trailer "ack: Peff" --trailer "review:" \
 		--trailer "ack: Junio" --trailer "bug: 42" \
 		--trailer "Tested-by: Jakub" --trailer "ack: Junio" \
 		--trailer "ack: Junio" --trailer "ack: Peff" <complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'using "ifExists = addIfDifferentNeighbor"  with "where = after"' '
 	git config trailer.ack.ifExists "addIfDifferentNeighbor" &&
 	git config trailer.ack.where "after" &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Bug #42
 		Fixes: Z
 		Acked-by= Z
 		Acked-by= Peff
 		Acked-by= Junio
 		Acked-by= Peff
 		Reviewed-by:
 		Signed-off-by: Z
 		Tested-by: Jakub
 	EOF
 	git interpret-trailers --trailer "ack: Peff" --trailer "review:" \
 		--trailer "ack: Junio" --trailer "bug: 42" \
 		--trailer "Tested-by: Jakub" --trailer "ack: Junio" \
 		--trailer "ack: Junio" --trailer "ack: Peff" <complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'using "ifExists = addIfDifferentNeighbor" and --trim-empty' '
 	git config trailer.ack.ifExists "addIfDifferentNeighbor" &&
 	cat complex_message_body >expected &&
 	cat >>expected <<-\EOF &&
 		Bug #42
 		Acked-by= Peff
 		Acked-by= Junio
 		Acked-by= Peff
 	EOF
 	git interpret-trailers --trim-empty --trailer "ack: Peff" \
 		--trailer "Acked-by= Peff" --trailer "review:" \
 		--trailer "ack: Junio" --trailer "bug: 42" \
 		--trailer "ack: Peff" <complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'using "ifExists = add" with "where = end"' '
 	git config trailer.ack.ifExists "add" &&
 	git config trailer.ack.where "end" &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Bug #42
 		Fixes: Z
 		Acked-by= Z
 		Reviewed-by:
 		Signed-off-by: Z
 		Acked-by= Peff
 		Acked-by= Peff
 		Tested-by: Jakub
 		Acked-by= Junio
 		Tested-by: Johannes
 		Acked-by= Peff
 	EOF
 	git interpret-trailers --trailer "ack: Peff" \
 		--trailer "Acked-by= Peff" --trailer "review:" \
 		--trailer "Tested-by: Jakub" --trailer "ack: Junio" \
 		--trailer "bug: 42" --trailer "Tested-by: Johannes" \
 		--trailer "ack: Peff" <complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'using "ifExists = add" with "where = after"' '
 	git config trailer.ack.ifExists "add" &&
 	git config trailer.ack.where "after" &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Bug #42
 		Fixes: Z
 		Acked-by= Z
 		Acked-by= Peff
 		Acked-by= Peff
 		Acked-by= Junio
 		Acked-by= Peff
 		Reviewed-by:
 		Signed-off-by: Z
 	EOF
 	git interpret-trailers --trailer "ack: Peff" \
 		--trailer "Acked-by= Peff" --trailer "review:" \
 		--trailer "ack: Junio" --trailer "bug: 42" \
 		--trailer "ack: Peff" <complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'using "ifExists = replace"' '
 	git config trailer.fix.key "Fixes: " &&
 	git config trailer.fix.ifExists "replace" &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Bug #42
 		Acked-by= Z
 		Acked-by= Junio
 		Acked-by= Peff
 		Reviewed-by:
 		Signed-off-by: Z
 		Fixes: 22
 	EOF
 	git interpret-trailers --trailer "review:" \
 		--trailer "fix=53" --trailer "ack: Junio" --trailer "fix=22" \
 		--trailer "bug: 42" --trailer "ack: Peff" \
 		<complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'using "ifExists = replace" with "where = after"' '
 	git config trailer.fix.where "after" &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Bug #42
 		Fixes: 22
 		Acked-by= Z
 		Acked-by= Junio
 		Acked-by= Peff
 		Reviewed-by:
 		Signed-off-by: Z
 	EOF
 	git interpret-trailers --trailer "review:" \
 		--trailer "fix=53" --trailer "ack: Junio" --trailer "fix=22" \
 		--trailer "bug: 42" --trailer "ack: Peff" \
 		<complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'using "ifExists = doNothing"' '
 	git config trailer.fix.ifExists "doNothing" &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Bug #42
 		Fixes: Z
 		Acked-by= Z
 		Acked-by= Junio
 		Acked-by= Peff
 		Reviewed-by:
 		Signed-off-by: Z
 	EOF
 	git interpret-trailers --trailer "review:" --trailer "fix=53" \
 		--trailer "ack: Junio" --trailer "fix=22" \
 		--trailer "bug: 42" --trailer "ack: Peff" \
 		<complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'the default is "ifMissing = add"' '
 	git config trailer.cc.key "Cc: " &&
 	git config trailer.cc.where "before" &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Bug #42
 		Cc: Linus
 		Fixes: Z
 		Acked-by= Z
 		Acked-by= Junio
 		Acked-by= Peff
 		Reviewed-by:
 		Signed-off-by: Z
 	EOF
 	git interpret-trailers --trailer "review:" --trailer "fix=53" \
 		--trailer "cc=Linus" --trailer "ack: Junio" \
 		--trailer "fix=22" --trailer "bug: 42" --trailer "ack: Peff" \
 		<complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'when default "ifMissing" is "doNothing"' '
 	git config trailer.ifmissing "doNothing" &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Fixes: Z
 		Acked-by= Z
 		Acked-by= Junio
 		Acked-by= Peff
 		Reviewed-by:
 		Signed-off-by: Z
 	EOF
 	git interpret-trailers --trailer "review:" --trailer "fix=53" \
 		--trailer "cc=Linus" --trailer "ack: Junio" \
 		--trailer "fix=22" --trailer "bug: 42" --trailer "ack: Peff" \
 		<complex_message >actual &&
 	test_cmp expected actual &&
 	git config trailer.ifmissing "add"
 '
 test_expect_success 'using "ifMissing = add" with "where = end"' '
 	git config trailer.cc.key "Cc: " &&
 	git config trailer.cc.where "end" &&
 	git config trailer.cc.ifMissing "add" &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Bug #42
 		Fixes: Z
 		Acked-by= Z
 		Acked-by= Junio
 		Acked-by= Peff
 		Reviewed-by:
 		Signed-off-by: Z
 		Cc: Linus
 	EOF
 	git interpret-trailers --trailer "review:" --trailer "fix=53" \
 		--trailer "ack: Junio" --trailer "fix=22" \
 		--trailer "bug: 42" --trailer "cc=Linus" --trailer "ack: Peff" \
 		<complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'using "ifMissing = add" with "where = before"' '
 	git config trailer.cc.key "Cc: " &&
 	git config trailer.cc.where "before" &&
 	git config trailer.cc.ifMissing "add" &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Cc: Linus
 		Bug #42
 		Fixes: Z
 		Acked-by= Z
 		Acked-by= Junio
 		Acked-by= Peff
 		Reviewed-by:
 		Signed-off-by: Z
 	EOF
 	git interpret-trailers --trailer "review:" --trailer "fix=53" \
 		--trailer "ack: Junio" --trailer "fix=22" \
 		--trailer "bug: 42" --trailer "cc=Linus" --trailer "ack: Peff" \
 		<complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'using "ifMissing = doNothing"' '
 	git config trailer.cc.ifMissing "doNothing" &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Bug #42
 		Fixes: Z
 		Acked-by= Z
 		Acked-by= Junio
 		Acked-by= Peff
 		Reviewed-by:
 		Signed-off-by: Z
 	EOF
 	git interpret-trailers --trailer "review:" --trailer "fix=53" \
 		--trailer "cc=Linus" --trailer "ack: Junio" \
 		--trailer "fix=22" --trailer "bug: 42" --trailer "ack: Peff" \
 		<complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'default "where" is now "after"' '
 	git config trailer.where "after" &&
 	git config --unset trailer.ack.where &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Bug #42
 		Fixes: Z
 		Acked-by= Z
 		Acked-by= Peff
 		Acked-by= Peff
 		Acked-by= Junio
 		Acked-by= Peff
 		Reviewed-by:
 		Signed-off-by: Z
 		Tested-by: Jakub
 		Tested-by: Johannes
 	EOF
 	git interpret-trailers --trailer "ack: Peff" \
 		--trailer "Acked-by= Peff" --trailer "review:" \
 		--trailer "Tested-by: Jakub" --trailer "ack: Junio" \
 		--trailer "bug: 42" --trailer "Tested-by: Johannes" \
 		--trailer "ack: Peff" <complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'with simple command' '
 	git config trailer.sign.key "Signed-off-by: " &&
 	git config trailer.sign.where "after" &&
 	git config trailer.sign.ifExists "addIfDifferentNeighbor" &&
 	git config trailer.sign.command "echo \"A U Thor <author@example.com>\"" &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Fixes: Z
 		Acked-by= Z
 		Reviewed-by:
 		Signed-off-by: Z
 		Signed-off-by: A U Thor <author@example.com>
 	EOF
 	git interpret-trailers --trailer "review:" --trailer "fix=22" \
 		<complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'with command using commiter information' '
 	git config trailer.sign.ifExists "addIfDifferent" &&
 	git config trailer.sign.command "echo \"\$GIT_COMMITTER_NAME <\$GIT_COMMITTER_EMAIL>\"" &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Fixes: Z
 		Acked-by= Z
 		Reviewed-by:
 		Signed-off-by: Z
 		Signed-off-by: C O Mitter <committer@example.com>
 	EOF
 	git interpret-trailers --trailer "review:" --trailer "fix=22" \
 		<complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'with command using author information' '
 	git config trailer.sign.key "Signed-off-by: " &&
 	git config trailer.sign.where "after" &&
 	git config trailer.sign.ifExists "addIfDifferentNeighbor" &&
 	git config trailer.sign.command "echo \"\$GIT_AUTHOR_NAME <\$GIT_AUTHOR_EMAIL>\"" &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-\EOF &&
 		Fixes: Z
 		Acked-by= Z
 		Reviewed-by:
 		Signed-off-by: Z
 		Signed-off-by: A U Thor <author@example.com>
 	EOF
 	git interpret-trailers --trailer "review:" --trailer "fix=22" \
 		<complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'setup a commit' '
 	echo "Content of the first commit." > a.txt &&
 	git add a.txt &&
 	git commit -m "Add file a.txt"
 '
 test_expect_success 'with command using $ARG' '
 	git config trailer.fix.ifExists "replace" &&
 	git config trailer.fix.command "git log -1 --oneline --format=\"%h (%s)\" --abbrev-commit --abbrev=14 \$ARG" &&
 	FIXED=$(git log -1 --oneline --format="%h (%s)" --abbrev-commit --abbrev=14 HEAD) &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-EOF &&
 		Fixes: $FIXED
 		Acked-by= Z
 		Reviewed-by:
 		Signed-off-by: Z
 		Signed-off-by: A U Thor <author@example.com>
 	EOF
 	git interpret-trailers --trailer "review:" --trailer "fix=HEAD" \
 		<complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'with failing command using $ARG' '
 	git config trailer.fix.ifExists "replace" &&
 	git config trailer.fix.command "false \$ARG" &&
 	cat complex_message_body >expected &&
 	sed -e "s/ Z\$/ /" >>expected <<-EOF &&
 		Fixes: Z
 		Acked-by= Z
 		Reviewed-by:
 		Signed-off-by: Z
 		Signed-off-by: A U Thor <author@example.com>
 	EOF
 	git interpret-trailers --trailer "review:" --trailer "fix=HEAD" \
 		<complex_message >actual &&
 	test_cmp expected actual
 '
 test_expect_success 'with empty tokens' '
 	git config --unset trailer.fix.command &&
 	cat >expected <<-EOF &&
 		Signed-off-by: A U Thor <author@example.com>
 	EOF
 	git interpret-trailers --trailer ":" --trailer ":test" >actual <<-EOF &&
 	EOF
 	test_cmp expected actual
 '
 test_expect_success 'with command but no key' '
 	git config --unset trailer.sign.key &&
 	cat >expected <<-EOF &&
 		sign: A U Thor <author@example.com>
 	EOF
 	git interpret-trailers >actual <<-EOF &&
 	EOF
 	test_cmp expected actual
 '
 test_expect_success 'with no command and no key' '
 	git config --unset trailer.review.key &&
 	cat >expected <<-EOF &&
 		review: Junio
 		sign: A U Thor <author@example.com>
 	EOF
 	git interpret-trailers --trailer "review:Junio" >actual <<-EOF &&
 	EOF
 	test_cmp expected actual
 '
 test_done