t3321: add test cases about the notes stripspace behavior

Signed-off-by: Teng Long <dyroneteng@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

diff --git a/t/t3321-notes-stripspace.sh b/t/t3321-notes-stripspace.sh
new file mode 100755 (executable)
index 0000000..8997787
--- /dev/null
+++ b/t/t3321-notes-stripspace.sh
@@ -0,0 +1,291 @@
+#!/bin/sh
+#
+# Copyright (c) 2023 Teng Long
+#
+
+test_description='Test commit notes with stripspace behavior'
+
+. ./test-lib.sh
+
+MULTI_LF="$LF$LF$LF"
+write_script fake_editor <<\EOF
+echo "$MSG" >"$1"
+echo "$MSG" >&2
+EOF
+GIT_EDITOR=./fake_editor
+export GIT_EDITOR
+
+test_expect_success 'setup the commit' '
+       test_commit 1st
+'
+
+test_expect_success 'add note by editor' '
+       test_when_finished "git notes remove" &&
+       cat >expect <<-EOF &&
+       first-line
+
+       second-line
+       EOF
+
+       MSG="${LF}first-line${MULTI_LF}second-line${LF}" git notes add  &&
+       git notes show >actual &&
+       test_cmp expect actual
+'
+
+test_expect_success 'add note by specifying single "-m"' '
+       test_when_finished "git notes remove" &&
+       cat >expect <<-EOF &&
+       first-line
+
+       second-line
+       EOF
+
+       git notes add -m "${LF}first-line${MULTI_LF}second-line${LF}" &&
+       git notes show >actual &&
+       test_cmp expect actual
+'
+
+test_expect_success 'add note by specifying multiple "-m"' '
+       test_when_finished "git notes remove" &&
+       cat >expect <<-EOF &&
+       first-line
+
+       second-line
+       EOF
+
+       git notes add -m "${LF}" \
+                     -m "first-line" \
+                     -m "${MULTI_LF}" \
+                     -m "second-line" \
+                     -m "${LF}" &&
+       git notes show >actual &&
+       test_cmp expect actual
+'
+
+
+test_expect_success 'append note by editor' '
+       test_when_finished "git notes remove" &&
+       cat >expect <<-EOF &&
+       first-line
+
+       second-line
+       EOF
+
+       git notes add -m "first-line" &&
+       MSG="${MULTI_LF}second-line${LF}" git notes append  &&
+       git notes show >actual &&
+       test_cmp expect actual
+'
+
+test_expect_success 'append note by specifying single "-m"' '
+       test_when_finished "git notes remove" &&
+       cat >expect <<-EOF &&
+       first-line
+
+       second-line
+       EOF
+
+       git notes add -m "${LF}first-line" &&
+       git notes append -m "${MULTI_LF}second-line${LF}" &&
+       git notes show >actual &&
+       test_cmp expect actual
+'
+
+test_expect_success 'append note by specifying multiple "-m"' '
+       test_when_finished "git notes remove" &&
+       cat >expect <<-EOF &&
+       first-line
+
+       second-line
+       EOF
+
+       git notes add -m "${LF}first-line" &&
+       git notes append -m "${MULTI_LF}" \
+                     -m "second-line" \
+                     -m "${LF}" &&
+       git notes show >actual &&
+       test_cmp expect actual
+'
+
+test_expect_success 'add note by specifying single "-F"' '
+       test_when_finished "git notes remove" &&
+       cat >expect <<-EOF &&
+       first-line
+
+       second-line
+       EOF
+
+       cat >note-file <<-EOF &&
+       ${LF}
+       first-line
+       ${MULTI_LF}
+       second-line
+       ${LF}
+       EOF
+
+       git notes add -F note-file &&
+       git notes show >actual &&
+       test_cmp expect actual
+'
+
+test_expect_success 'add notes by specifying multiple "-F"' '
+       test_when_finished "git notes remove" &&
+       cat >expect <<-EOF &&
+       file-1-first-line
+
+       file-1-second-line
+
+       file-2-first-line
+
+       file-2-second-line
+       EOF
+
+       cat >note-file-1 <<-EOF &&
+       ${LF}
+       file-1-first-line
+       ${MULTI_LF}
+       file-1-second-line
+       ${LF}
+       EOF
+
+       cat >note-file-2 <<-EOF &&
+       ${LF}
+       file-2-first-line
+       ${MULTI_LF}
+       file-2-second-line
+       ${LF}
+       EOF
+
+       git notes add -F note-file-1 -F note-file-2 &&
+       git notes show >actual &&
+       test_cmp expect actual
+'
+
+test_expect_success 'append note by specifying single "-F"' '
+       test_when_finished "git notes remove" &&
+       cat >expect <<-EOF &&
+       initial-line
+
+       first-line
+
+       second-line
+       EOF
+
+       cat >note-file <<-EOF &&
+       ${LF}
+       first-line
+       ${MULTI_LF}
+       second-line
+       ${LF}
+       EOF
+
+       git notes add -m "initial-line" &&
+       git notes append -F note-file &&
+       git notes show >actual &&
+       test_cmp expect actual
+'
+
+test_expect_success 'append notes by specifying multiple "-F"' '
+       test_when_finished "git notes remove" &&
+       cat >expect <<-EOF &&
+       initial-line
+
+       file-1-first-line
+
+       file-1-second-line
+
+       file-2-first-line
+
+       file-2-second-line
+       EOF
+
+       cat >note-file-1 <<-EOF &&
+       ${LF}
+       file-1-first-line
+       ${MULTI_LF}
+       file-1-second-line
+       ${LF}
+       EOF
+
+       cat >note-file-2 <<-EOF &&
+       ${LF}
+       file-2-first-line
+       ${MULTI_LF}
+       file-2-second-line
+       ${LF}
+       EOF
+
+       git notes add -m "initial-line" &&
+       git notes append -F note-file-1 -F note-file-2 &&
+       git notes show >actual &&
+       test_cmp expect actual
+'
+
+test_expect_success 'add notes with empty messages' '
+       rev=$(git rev-parse HEAD) &&
+       git notes add -m "${LF}" \
+                     -m "${MULTI_LF}" \
+                     -m "${LF}" >actual 2>&1 &&
+       test_i18ngrep "Removing note for object" actual
+'
+
+test_expect_success 'add note by specifying "-C" , do not stripspace is the default behavior' '
+       test_when_finished "git notes remove" &&
+       cat >expect <<-EOF &&
+       ${LF}
+       first-line
+       ${MULTI_LF}
+       second-line
+       ${LF}
+       EOF
+
+       cat expect | git hash-object -w --stdin >blob &&
+       git notes add -C $(cat blob) &&
+       git notes show >actual &&
+       test_cmp expect actual
+'
+
+test_expect_success 'add notes with "-C" and "-m", "-m" will stripspace all together' '
+       test_when_finished "git notes remove" &&
+       cat >data <<-EOF &&
+       ${LF}
+       first-line
+       ${MULTI_LF}
+       second-line
+       ${LF}
+       EOF
+
+       cat >expect <<-EOF &&
+       first-line
+
+       second-line
+
+       third-line
+       EOF
+
+       cat data | git hash-object -w --stdin >blob &&
+       git notes add -C $(cat blob) -m "third-line" &&
+       git notes show >actual &&
+       test_cmp expect actual
+'
+
+test_expect_success 'add notes with "-m" and "-C", "-C" will not stripspace all together' '
+       test_when_finished "git notes remove" &&
+       cat >data <<-EOF &&
+
+       second-line
+       EOF
+
+       cat >expect <<-EOF &&
+       first-line
+       ${LF}
+       second-line
+       EOF
+
+       cat data | git hash-object -w --stdin >blob &&
+       git notes add -m "first-line" -C $(cat blob)  &&
+       git notes show >actual &&
+       test_cmp expect actual
+'
+
+test_done