From 044a654c5e9d004935fdcc1abf755bc0e49151bd Mon Sep 17 00:00:00 2001
From: Bryce Kwon <bryce@brycekwon.com>
Date: Sat, 7 Nov 2026 13:11:28 -1000
Subject: Test enable-plain-email=0 where it changes output

No test set it, though it switches the patch to a format of cgit's own
and drops the address from the feed and the trailer idents.
---
 tests/t0109-patch.sh    | 10 ++++++++++
 tests/t0111-atom.sh     |  8 ++++++++
 tests/t0206-trailers.sh | 10 ++++++++++
 3 files changed, 28 insertions(+)

(limited to 'tests')

diff --git a/tests/t0109-patch.sh b/tests/t0109-patch.sh
index 6aae556..88e94ae 100755
--- a/tests/t0109-patch.sh
+++ b/tests/t0109-patch.sh
@@ -70,6 +70,16 @@ test_expect_success 'the signature names the version when asked to' '
 	check_cgit_signature "$CGIT_VERSION"
 '
 
+# Without plain email the patch is written from a format of cgit's own, whose
+# From line carries the name alone.
+test_expect_success 'enable-plain-email=0 leaves the address off the patch' '
+	cgitrc_copy plainrc enable-plain-email=0 &&
+	cgit_ok plainrc "url=foo/patch" >tmp &&
+	grep "^From: A U Thor$" tmp &&
+	grep "^Subject: commit 5$" tmp &&
+	! grep "author@example.com" tmp
+'
+
 # The old name of a file renamed in the commit joins the limit, so the patch
 # carries the rename rather than a file added whole.
 test_expect_success 'a renamed file is patched as the rename' '
diff --git a/tests/t0111-atom.sh b/tests/t0111-atom.sh
index 6d3e2c6..568fea4 100755
--- a/tests/t0111-atom.sh
+++ b/tests/t0111-atom.sh
@@ -57,6 +57,14 @@ test_expect_success 'a path with pathspec magic is refused' '
 	grep "^Status: 200" tmp
 '
 
+test_expect_success 'enable-plain-email=0 leaves the address out of the feed' '
+	cgitrc_copy plainrc enable-plain-email=0 &&
+	cgit_ok plainrc "url=foo/atom" >tmp &&
+	grep "<name>A U Thor</name>" tmp &&
+	! grep "<email>" tmp &&
+	! grep "author@example.com" tmp
+'
+
 test_expect_success 'the feed link encodes the repository and the branch' '
 	cgitrc_copy feedrc &&
 	CGIT_CONFIG="$PWD/feedrc" HTTP_HOST=example.org QUERY_STRING="url=foo%2Bbar/log/&h=1%2B2" cgit >tmp &&
diff --git a/tests/t0206-trailers.sh b/tests/t0206-trailers.sh
index 15c3600..70f10bd 100755
--- a/tests/t0206-trailers.sh
+++ b/tests/t0206-trailers.sh
@@ -78,6 +78,16 @@ test_expect_success 'the escaped value is what the filter sees' '
 	grep "<tr><th scope=.row.>Link</th><td>Link commit HTTPS://EXAMPLE.COM/?A=1&AMP;B=2</td></tr>" tmp
 '
 
+# The dump filter is handed the address whatever the setting, so the page is
+# drawn without it here.
+test_expect_success 'enable-plain-email=0 leaves the addresses off the idents' '
+	cgitrc_repo plainrc trailers enable-commit-trailers=1 enable-plain-email=0 &&
+	cgit_ok plainrc "url=trailers/commit/" >tmp &&
+	grep "<tr><th scope=.row.>Signed-off-by</th><td>Mapped Name<br>C O Mitter</td></tr>" tmp &&
+	grep "<tr><th scope=.row.>Acked-by</th><td>Some One (maintainer)</td></tr>" tmp &&
+	! grep "@example.com" tmp
+'
+
 test_expect_success 'a message with no block gets no table' '
 	plain=$(git -C repos/trailers rev-parse HEAD~1) &&
 	cgit_url "trailers/commit/&id=$plain" >tmp &&
-- 
cgit 2.12.1