From: Philippe Blain <levraiphilippeblain@xxxxxxxxx>
A following commit will fix a bug in the ref-filter API that causes
commit and tag messages containing CRLF to be incorrectly parsed and
displayed.
Add a test library (t/lib-crlf-messages.sh) that creates refs with such
commit messages, so that we can easily test that this bug does not
appear in other commands in the future.
The function test_crlf_subject_body_and_contents can be used to test
that the `--format` option of `branch`, `tag`, `for-each-ref` and
`log` correctly displays the subject, body and raw body of commits and
tag messages.
The commits are created using `commit-tree` such that the current branch
in the test repository is not affected when `test_create_crlf_refs` is
called in a test. This is to done so that the CRLF tests can be inserted
anywhere in a test script where it makes sense to do so, without having
to potentially modify further test that depend on output that would be
modified if the current branch gained new commits.
Signed-off-by: Philippe Blain <levraiphilippeblain@xxxxxxxxx>
---
t/lib-crlf-messages.sh | 73 ++++++++++++++++++++++++++++++++++++++++++
1 file changed, 73 insertions(+)
create mode 100644 t/lib-crlf-messages.sh
diff --git a/t/lib-crlf-messages.sh b/t/lib-crlf-messages.sh
new file mode 100644
index 00000000000..64d2ad12019
--- /dev/null
+++ b/t/lib-crlf-messages.sh
@@ -0,0 +1,73 @@
+# Setup refs with commit and tag messages containing CRLF
+
+create_crlf_ref () {
+ message="$1" &&
+ subject="$2" &&
+ body="$3" &&
+ branch="$4" &&
+ printf "${message}" >.crlf-message-${branch}.txt &&
+ printf "${subject}" >.crlf-subject-${branch}.txt &&
+ printf "${body}" >.crlf-body-${branch}.txt &&
+ test_tick &&
+ hash=$(git commit-tree HEAD^{tree} -p HEAD -F .crlf-message-${branch}.txt) &&
+ git branch ${branch} ${hash} &&
+ git tag tag-${branch} ${branch} -F .crlf-message-${branch}.txt --cleanup=verbatim
+}
+
+create_crlf_refs () {
+ message="Subject first line\r\n\r\nBody first line\r\nBody second line\r\n" &&
+ body="Body first line\r\nBody second line\r\n" &&
+ subject="Subject first line" &&
+ create_crlf_ref "${message}" "${subject}" "${body}" "crlf" &&
+ message="Subject first line\r\n\r\n\r\nBody first line\r\nBody second line\r\n" &&
+ create_crlf_ref "${message}" "${subject}" "${body}" "crlf-empty-lines-after-subject" &&
+ message="Subject first line\r\nSubject second line\r\n\r\nBody first line\r\nBody second line\r\n" &&
+ subject="Subject first line Subject second line" &&
+ create_crlf_ref "${message}" "${subject}" "${body}" "crlf-two-line-subject"
+}
+
+test_create_crlf_refs () {
+ test_expect_success 'setup refs with CRLF commit messages' '
+ create_crlf_refs
+ '
+}
+
+cleanup_crlf_refs () {
+ for branch in crlf crlf-empty-lines-after-subject crlf-two-line-subject; do
+ git branch -D ${branch} &&
+ git tag -d tag-${branch} &&
+ rm .crlf-message-${branch}.txt &&
+ rm .crlf-subject-${branch}.txt &&
+ rm .crlf-body-${branch}.txt
+ done
+}
+
+test_cleanup_crlf_refs () {
+ test_expect_success 'clenup refs with CRLF commit messages' '
+ cleanup_crlf_refs
+ '
+}
+
+test_crlf_subject_body_and_contents() {
+ command_and_args="$@" &&
+ command=$1 &&
+ if [ ${command} = "branch" ] || [ ${command} = "for-each-ref" ] || [ ${command} = "tag" ]; then
+ atoms="(contents:subject) (contents:body) (contents)"
+ elif [ ${command} = "log" ]; then
+ atoms="s b B"
+ fi &&
+ files="subject body message" &&
+ while [ -n "${atoms}" ]; do
+ set ${atoms} && atom=$1 && shift && atoms="$*" &&
+ set ${files} && file=$1 && shift && files="$*" &&
+ test_expect_success "${command}: --format='%${atom}' works with CRLF input" "
+ rm -f expect &&
+ for ref in crlf crlf-empty-lines-after-subject crlf-two-line-subject; do
+ cat .crlf-${file}-\"\${ref}\".txt >>expect &&
+ printf \"\n\" >>expect
+ done &&
+ git $command_and_args --format=\"%${atom}\" >actual &&
+ test_cmp expect actual
+ "
+ done
+}
--
gitgitgadget
