mirror of
https://github.com/git/git
synced 2024-11-05 18:59:29 +00:00
0a0416a34a
The only way to safely quote arbitrary text in a pretty-print user format is to replace instances of "%" with "%x25". This is slightly unreadable, and many users would expect "%%" to produce a single "%", as that is what printf format specifiers do. This patch converts "%%" to "%" for all users of strbuf_expand(): (1) git-daemon interpolated paths (2) pretty-print user formats (3) merge driver command lines Case (1) was already doing the conversion itself outside of strbuf_expand(). Case (2) is the intended beneficiary of this patch. Case (3) users probably won't notice, but as this is user-facing behavior, consistently providing the quoting mechanism makes sense. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
212 lines
5.5 KiB
Bash
Executable file
212 lines
5.5 KiB
Bash
Executable file
#!/bin/sh
|
||
|
||
test_description='git rev-list --pretty=format test'
|
||
|
||
. ./test-lib.sh
|
||
|
||
test_tick
|
||
test_expect_success 'setup' '
|
||
touch foo && git add foo && git commit -m "added foo" &&
|
||
echo changed >foo && git commit -a -m "changed foo"
|
||
'
|
||
|
||
# usage: test_format name format_string <expected_output
|
||
test_format() {
|
||
cat >expect.$1
|
||
test_expect_success "format $1" "
|
||
git rev-list --pretty=format:'$2' master >output.$1 &&
|
||
test_cmp expect.$1 output.$1
|
||
"
|
||
}
|
||
|
||
test_format percent %%h <<'EOF'
|
||
commit 131a310eb913d107dd3c09a65d1651175898735d
|
||
%h
|
||
commit 86c75cfd708a0e5868dc876ed5b8bb66c80b4873
|
||
%h
|
||
EOF
|
||
|
||
test_format hash %H%n%h <<'EOF'
|
||
commit 131a310eb913d107dd3c09a65d1651175898735d
|
||
131a310eb913d107dd3c09a65d1651175898735d
|
||
131a310
|
||
commit 86c75cfd708a0e5868dc876ed5b8bb66c80b4873
|
||
86c75cfd708a0e5868dc876ed5b8bb66c80b4873
|
||
86c75cf
|
||
EOF
|
||
|
||
test_format tree %T%n%t <<'EOF'
|
||
commit 131a310eb913d107dd3c09a65d1651175898735d
|
||
fe722612f26da5064c32ca3843aa154bdb0b08a0
|
||
fe72261
|
||
commit 86c75cfd708a0e5868dc876ed5b8bb66c80b4873
|
||
4d5fcadc293a348e88f777dc0920f11e7d71441c
|
||
4d5fcad
|
||
EOF
|
||
|
||
test_format parents %P%n%p <<'EOF'
|
||
commit 131a310eb913d107dd3c09a65d1651175898735d
|
||
86c75cfd708a0e5868dc876ed5b8bb66c80b4873
|
||
86c75cf
|
||
commit 86c75cfd708a0e5868dc876ed5b8bb66c80b4873
|
||
|
||
|
||
EOF
|
||
|
||
# we don't test relative here
|
||
test_format author %an%n%ae%n%ad%n%aD%n%at <<'EOF'
|
||
commit 131a310eb913d107dd3c09a65d1651175898735d
|
||
A U Thor
|
||
author@example.com
|
||
Thu Apr 7 15:13:13 2005 -0700
|
||
Thu, 7 Apr 2005 15:13:13 -0700
|
||
1112911993
|
||
commit 86c75cfd708a0e5868dc876ed5b8bb66c80b4873
|
||
A U Thor
|
||
author@example.com
|
||
Thu Apr 7 15:13:13 2005 -0700
|
||
Thu, 7 Apr 2005 15:13:13 -0700
|
||
1112911993
|
||
EOF
|
||
|
||
test_format committer %cn%n%ce%n%cd%n%cD%n%ct <<'EOF'
|
||
commit 131a310eb913d107dd3c09a65d1651175898735d
|
||
C O Mitter
|
||
committer@example.com
|
||
Thu Apr 7 15:13:13 2005 -0700
|
||
Thu, 7 Apr 2005 15:13:13 -0700
|
||
1112911993
|
||
commit 86c75cfd708a0e5868dc876ed5b8bb66c80b4873
|
||
C O Mitter
|
||
committer@example.com
|
||
Thu Apr 7 15:13:13 2005 -0700
|
||
Thu, 7 Apr 2005 15:13:13 -0700
|
||
1112911993
|
||
EOF
|
||
|
||
test_format encoding %e <<'EOF'
|
||
commit 131a310eb913d107dd3c09a65d1651175898735d
|
||
commit 86c75cfd708a0e5868dc876ed5b8bb66c80b4873
|
||
EOF
|
||
|
||
test_format subject %s <<'EOF'
|
||
commit 131a310eb913d107dd3c09a65d1651175898735d
|
||
changed foo
|
||
commit 86c75cfd708a0e5868dc876ed5b8bb66c80b4873
|
||
added foo
|
||
EOF
|
||
|
||
test_format body %b <<'EOF'
|
||
commit 131a310eb913d107dd3c09a65d1651175898735d
|
||
commit 86c75cfd708a0e5868dc876ed5b8bb66c80b4873
|
||
EOF
|
||
|
||
test_format colors %Credfoo%Cgreenbar%Cbluebaz%Cresetxyzzy <<'EOF'
|
||
commit 131a310eb913d107dd3c09a65d1651175898735d
|
||
[31mfoo[32mbar[34mbaz[mxyzzy
|
||
commit 86c75cfd708a0e5868dc876ed5b8bb66c80b4873
|
||
[31mfoo[32mbar[34mbaz[mxyzzy
|
||
EOF
|
||
|
||
test_format advanced-colors '%C(red yellow bold)foo%C(reset)' <<'EOF'
|
||
commit 131a310eb913d107dd3c09a65d1651175898735d
|
||
[1;31;43mfoo[m
|
||
commit 86c75cfd708a0e5868dc876ed5b8bb66c80b4873
|
||
[1;31;43mfoo[m
|
||
EOF
|
||
|
||
cat >commit-msg <<'EOF'
|
||
Test printing of complex bodies
|
||
|
||
This commit message is much longer than the others,
|
||
and it will be encoded in iso8859-1. We should therefore
|
||
include an iso8859 character: ¡bueno!
|
||
EOF
|
||
test_expect_success 'setup complex body' '
|
||
git config i18n.commitencoding iso8859-1 &&
|
||
echo change2 >foo && git commit -a -F commit-msg
|
||
'
|
||
|
||
test_format complex-encoding %e <<'EOF'
|
||
commit f58db70b055c5718631e5c61528b28b12090cdea
|
||
iso8859-1
|
||
commit 131a310eb913d107dd3c09a65d1651175898735d
|
||
commit 86c75cfd708a0e5868dc876ed5b8bb66c80b4873
|
||
EOF
|
||
|
||
test_format complex-subject %s <<'EOF'
|
||
commit f58db70b055c5718631e5c61528b28b12090cdea
|
||
Test printing of complex bodies
|
||
commit 131a310eb913d107dd3c09a65d1651175898735d
|
||
changed foo
|
||
commit 86c75cfd708a0e5868dc876ed5b8bb66c80b4873
|
||
added foo
|
||
EOF
|
||
|
||
test_format complex-body %b <<'EOF'
|
||
commit f58db70b055c5718631e5c61528b28b12090cdea
|
||
This commit message is much longer than the others,
|
||
and it will be encoded in iso8859-1. We should therefore
|
||
include an iso8859 character: ¡bueno!
|
||
|
||
commit 131a310eb913d107dd3c09a65d1651175898735d
|
||
commit 86c75cfd708a0e5868dc876ed5b8bb66c80b4873
|
||
EOF
|
||
|
||
test_expect_success '%ad respects --date=' '
|
||
echo 2005-04-07 >expect.ad-short &&
|
||
git log -1 --date=short --pretty=tformat:%ad >output.ad-short master &&
|
||
test_cmp expect.ad-short output.ad-short
|
||
'
|
||
|
||
test_expect_success 'empty email' '
|
||
test_tick &&
|
||
C=$(GIT_AUTHOR_EMAIL= git commit-tree HEAD^{tree} </dev/null) &&
|
||
A=$(git show --pretty=format:%an,%ae,%ad%n -s $C) &&
|
||
test "$A" = "A U Thor,,Thu Apr 7 15:14:13 2005 -0700" || {
|
||
echo "Eh? $A" >failure
|
||
false
|
||
}
|
||
'
|
||
|
||
test_expect_success 'del LF before empty (1)' '
|
||
git show -s --pretty=format:"%s%n%-b%nThanks%n" HEAD^^ >actual &&
|
||
test $(wc -l <actual) = 2
|
||
'
|
||
|
||
test_expect_success 'del LF before empty (2)' '
|
||
git show -s --pretty=format:"%s%n%-b%nThanks%n" HEAD >actual &&
|
||
test $(wc -l <actual) = 6 &&
|
||
grep "^$" actual
|
||
'
|
||
|
||
test_expect_success 'add LF before non-empty (1)' '
|
||
git show -s --pretty=format:"%s%+b%nThanks%n" HEAD^^ >actual &&
|
||
test $(wc -l <actual) = 2
|
||
'
|
||
|
||
test_expect_success 'add LF before non-empty (2)' '
|
||
git show -s --pretty=format:"%s%+b%nThanks%n" HEAD >actual &&
|
||
test $(wc -l <actual) = 6 &&
|
||
grep "^$" actual
|
||
'
|
||
|
||
test_expect_success '"%h %gD: %gs" is same as git-reflog' '
|
||
git reflog >expect &&
|
||
git log -g --format="%h %gD: %gs" >actual &&
|
||
test_cmp expect actual
|
||
'
|
||
|
||
test_expect_success '"%h %gD: %gs" is same as git-reflog (with date)' '
|
||
git reflog --date=raw >expect &&
|
||
git log -g --format="%h %gD: %gs" --date=raw >actual &&
|
||
test_cmp expect actual
|
||
'
|
||
|
||
test_expect_success '%gd shortens ref name' '
|
||
echo "master@{0}" >expect.gd-short &&
|
||
git log -g -1 --format=%gd refs/heads/master >actual.gd-short &&
|
||
test_cmp expect.gd-short actual.gd-short
|
||
'
|
||
|
||
test_done
|