Merge branch 'as/t4012-style-updates'
* as/t4012-style-updates: t4012: Use test_must_fail instead of if-else t4012: use 'printf' instead of 'dd' to generate a binary file t4012: Re-indent test snippets t4012: Make --shortstat test more robust t4012: Break up pipe into serial redirections t4012: Actually quote the sed script t4012: Unquote git command fragment in test title t4012: modernize style for quoting
This commit is contained in:
commit
19f13d29a2
@ -15,13 +15,14 @@ cat >expect.binary-numstat <<\EOF
|
||||
- - d
|
||||
EOF
|
||||
|
||||
test_expect_success 'prepare repository' \
|
||||
'echo AIT >a && echo BIT >b && echo CIT >c && echo DIT >d &&
|
||||
test_expect_success 'prepare repository' '
|
||||
echo AIT >a && echo BIT >b && echo CIT >c && echo DIT >d &&
|
||||
git update-index --add a b c d &&
|
||||
echo git >a &&
|
||||
cat "$TEST_DIRECTORY"/test-binary-1.png >b &&
|
||||
echo git >c &&
|
||||
cat b b >d'
|
||||
cat b b >d
|
||||
'
|
||||
|
||||
cat > expected <<\EOF
|
||||
a | 2 +-
|
||||
@ -30,16 +31,16 @@ cat > expected <<\EOF
|
||||
d | Bin
|
||||
4 files changed, 2 insertions(+), 2 deletions(-)
|
||||
EOF
|
||||
test_expect_success '"apply --stat" output for binary file change' '
|
||||
test_expect_success 'apply --stat output for binary file change' '
|
||||
git diff >diff &&
|
||||
git apply --stat --summary <diff >current &&
|
||||
test_i18ncmp expected current
|
||||
'
|
||||
|
||||
test_expect_success 'diff --shortstat output for binary file change' '
|
||||
echo " 4 files changed, 2 insertions(+), 2 deletions(-)" >expected &&
|
||||
tail -n 1 expected >expect &&
|
||||
git diff --shortstat >current &&
|
||||
test_i18ncmp expected current
|
||||
test_i18ncmp expect current
|
||||
'
|
||||
|
||||
test_expect_success 'diff --shortstat output for binary file change only' '
|
||||
@ -62,49 +63,42 @@ test_expect_success 'apply --numstat understands diff --binary format' '
|
||||
|
||||
# apply needs to be able to skip the binary material correctly
|
||||
# in order to report the line number of a corrupt patch.
|
||||
test_expect_success 'apply detecting corrupt patch correctly' \
|
||||
'git diff | sed -e 's/-CIT/xCIT/' >broken &&
|
||||
if git apply --stat --summary broken 2>detected
|
||||
then
|
||||
echo unhappy - should have detected an error
|
||||
(exit 1)
|
||||
else
|
||||
echo happy
|
||||
fi &&
|
||||
test_expect_success 'apply detecting corrupt patch correctly' '
|
||||
git diff >output &&
|
||||
sed -e "s/-CIT/xCIT/" <output >broken &&
|
||||
test_must_fail git apply --stat --summary broken 2>detected &&
|
||||
detected=`cat detected` &&
|
||||
detected=`expr "$detected" : "fatal.*at line \\([0-9]*\\)\$"` &&
|
||||
detected=`sed -ne "${detected}p" broken` &&
|
||||
test "$detected" = xCIT'
|
||||
test "$detected" = xCIT
|
||||
'
|
||||
|
||||
test_expect_success 'apply detecting corrupt patch correctly' \
|
||||
'git diff --binary | sed -e 's/-CIT/xCIT/' >broken &&
|
||||
if git apply --stat --summary broken 2>detected
|
||||
then
|
||||
echo unhappy - should have detected an error
|
||||
(exit 1)
|
||||
else
|
||||
echo happy
|
||||
fi &&
|
||||
test_expect_success 'apply detecting corrupt patch correctly' '
|
||||
git diff --binary | sed -e "s/-CIT/xCIT/" >broken &&
|
||||
test_must_fail git apply --stat --summary broken 2>detected &&
|
||||
detected=`cat detected` &&
|
||||
detected=`expr "$detected" : "fatal.*at line \\([0-9]*\\)\$"` &&
|
||||
detected=`sed -ne "${detected}p" broken` &&
|
||||
test "$detected" = xCIT'
|
||||
test "$detected" = xCIT
|
||||
'
|
||||
|
||||
test_expect_success 'initial commit' 'git commit -a -m initial'
|
||||
|
||||
# Try removal (b), modification (d), and creation (e).
|
||||
test_expect_success 'diff-index with --binary' \
|
||||
'echo AIT >a && mv b e && echo CIT >c && cat e >d &&
|
||||
test_expect_success 'diff-index with --binary' '
|
||||
echo AIT >a && mv b e && echo CIT >c && cat e >d &&
|
||||
git update-index --add --remove a b c d e &&
|
||||
tree0=`git write-tree` &&
|
||||
git diff --cached --binary >current &&
|
||||
git apply --stat --summary current'
|
||||
git apply --stat --summary current
|
||||
'
|
||||
|
||||
test_expect_success 'apply binary patch' \
|
||||
'git reset --hard &&
|
||||
test_expect_success 'apply binary patch' '
|
||||
git reset --hard &&
|
||||
git apply --binary --index <current &&
|
||||
tree1=`git write-tree` &&
|
||||
test "$tree1" = "$tree0"'
|
||||
test "$tree1" = "$tree0"
|
||||
'
|
||||
|
||||
test_expect_success 'diff --no-index with binary creation' '
|
||||
echo Q | q_to_nul >binary &&
|
||||
@ -125,7 +119,7 @@ cat >expect <<EOF
|
||||
EOF
|
||||
|
||||
test_expect_success 'diff --stat with binary files and big change count' '
|
||||
echo X | dd of=binfile bs=1k seek=1 &&
|
||||
printf "\01\00%1024d" 1 >binfile &&
|
||||
git add binfile &&
|
||||
i=0 &&
|
||||
while test $i -lt 10000; do
|
||||
|
Loading…
Reference in New Issue
Block a user