Subject: [PATCH 18/22] mailmap tests: add a test for comment syntax

From: Ævar Arnfjörð Bjarmason <[email-masked]>
Date: Tue, 12 Jan 2021 21:18:02 +0100

Add a test for mailmap comment syntax. As noted in [1] there was no
test coverage for this. Let's make sure a future change doesn't break
it.

1. https://lore.kernel.org/git/CAN0heSoKYWXqskCR=[email-masked]/

Reported-by: Martin Ågren <[email-masked]>
Signed-off-by: Ævar Arnfjörð Bjarmason <[email-masked]>
---
 t/t4203-mailmap.sh | 33 +++++++++++++++++++++++++++++++++
 1 file changed, 33 insertions(+)

diff --git a/t/t4203-mailmap.sh b/t/t4203-mailmap.sh
index 96ba5367c6..10e672e006 100755
--- a/t/t4203-mailmap.sh
+++ b/t/t4203-mailmap.sh
@@ -753,4 +753,37 @@ test_expect_success 'commit --author honors mailmap' '
 	test_cmp expect actual
 '
 
+test_expect_success 'comment syntax: setup' '
+	test_create_repo comm &&
+	test_commit -C comm --author "A <[email-masked]>" A &&
+	test_commit -C comm --author "B <[email-masked]>" B &&
+	test_commit -C comm --author "C <[email-masked]>" C &&
+	test_commit -C comm --author "D <d@e#ample.com>" D &&
+
+	test_config -C comm mailmap.file ../doc.map &&
+	cat >>doc.map <<-\EOF &&
+	# Ah <[email-masked]>
+
+	; Bee <[email-masked]>
+	Cee <[email-masked]> <[email-masked]>
+	Dee <[email-masked]> <d@e#ample.com>
+	EOF
+
+	cat >expect <<-\EOF &&
+	Author A <[email-masked]> maps to A <[email-masked]>
+	Committer C O Mitter <[email-masked]> maps to C O Mitter <[email-masked]>
+
+	Author B <[email-masked]> maps to ; Bee <[email-masked]>
+	Committer C O Mitter <[email-masked]> maps to C O Mitter <[email-masked]>
+
+	Author C <[email-masked]> maps to Cee <[email-masked]>
+	Committer C O Mitter <[email-masked]> maps to C O Mitter <[email-masked]>
+
+	Author D <d@e#ample.com> maps to Dee <[email-masked]>
+	Committer C O Mitter <[email-masked]> maps to C O Mitter <[email-masked]>
+	EOF
+	git -C comm log --reverse --pretty=format:"Author %an <%ae> maps to %aN <%aE>%nCommitter %cn <%ce> maps to %cN <%cE>%n" >actual &&
+	test_cmp expect actual
+'
+
 test_done
-- 
2.29.2.222.g5d2a92d10f8