unpack-objects: fix --strict handling

diff --git a/t/t4103-apply-binary.sh b/t/t4103-apply-binary.sh
old mode 100644 (file)
new mode 100755 (executable)
index 948d5b5..7c25634
--- a/t/t4103-apply-binary.sh
+++ b/t/t4103-apply-binary.sh
@@ -3,7 +3,7 @@
 # Copyright (c) 2005 Junio C Hamano
 #
 
-test_description='git-apply handling binary patches
+test_description='git apply handling binary patches
 
 '
 . ./test-lib.sh
@@ -20,59 +20,100 @@ EOF
 cat file1 >file2
 cat file1 >file4
 
-git-update-index --add --remove file1 file2 file4
+git update-index --add --remove file1 file2 file4
 git-commit -m 'Initial Version' 2>/dev/null
 
 git-checkout -b binary
-tr 'x' '\0' <file1 >file3
+tr 'x' '\000' <file1 >file3
 cat file3 >file4
-git-add file2
-tr '\0' 'v' <file3 >file1
+git add file2
+tr '\000' 'v' <file3 >file1
 rm -f file2
-git-update-index --add --remove file1 file2 file3 file4
+git update-index --add --remove file1 file2 file3 file4
 git-commit -m 'Second Version'
 
-git-diff-tree -p master binary >B.diff
-git-diff-tree -p -C master binary >C.diff
+git diff-tree -p master binary >B.diff
+git diff-tree -p -C master binary >C.diff
+
+git diff-tree -p --binary master binary >BF.diff
+git diff-tree -p --binary -C master binary >CF.diff
 
 test_expect_success 'stat binary diff -- should not fail.' \
        'git-checkout master
-        git-apply --stat --summary B.diff'
+        git apply --stat --summary B.diff'
 
 test_expect_success 'stat binary diff (copy) -- should not fail.' \
        'git-checkout master
-        git-apply --stat --summary C.diff'
-
-test_expect_failure 'check binary diff -- should fail.' \
-       'git-checkout master
-        git-apply --check B.diff'
+        git apply --stat --summary C.diff'
 
-test_expect_failure 'check binary diff (copy) -- should fail.' \
-       'git-checkout master
-        git-apply --check C.diff'
+test_expect_success 'check binary diff -- should fail.' \
+       'git-checkout master &&
+        ! git apply --check B.diff'
 
-# Now we start applying them.
+test_expect_success 'check binary diff (copy) -- should fail.' \
+       'git-checkout master &&
+        ! git apply --check C.diff'
 
-test_expect_failure 'apply binary diff -- should fail.' \
-       'git-checkout master
-        git-apply B.diff'
+test_expect_success \
+       'check incomplete binary diff with replacement -- should fail.' '
+       git-checkout master &&
+       ! git apply --check --allow-binary-replacement B.diff
+'
 
-git-reset --hard
+test_expect_success \
+    'check incomplete binary diff with replacement (copy) -- should fail.' '
+        git-checkout master &&
+        ! git apply --check --allow-binary-replacement C.diff
+'
 
-test_expect_failure 'apply binary diff -- should fail.' \
+test_expect_success 'check binary diff with replacement.' \
        'git-checkout master
-        git-apply --index B.diff'
-
-git-reset --hard
+        git apply --check --allow-binary-replacement BF.diff'
 
-test_expect_failure 'apply binary diff (copy) -- should fail.' \
+test_expect_success 'check binary diff with replacement (copy).' \
        'git-checkout master
-        git-apply C.diff'
+        git apply --check --allow-binary-replacement CF.diff'
 
-git-reset --hard
+# Now we start applying them.
 
-test_expect_failure 'apply binary diff (copy) -- should fail.' \
-       'git-checkout master
-        git-apply --index C.diff'
+do_reset () {
+       rm -f file? &&
+       git-reset --hard &&
+       git-checkout -f master
+}
+
+test_expect_success 'apply binary diff -- should fail.' \
+       'do_reset &&
+        ! git apply B.diff'
+
+test_expect_success 'apply binary diff -- should fail.' \
+       'do_reset &&
+        ! git apply --index B.diff'
+
+test_expect_success 'apply binary diff (copy) -- should fail.' \
+       'do_reset &&
+        ! git apply C.diff'
+
+test_expect_success 'apply binary diff (copy) -- should fail.' \
+       'do_reset &&
+        ! git apply --index C.diff'
+
+test_expect_success 'apply binary diff without replacement.' \
+       'do_reset &&
+        git apply BF.diff'
+
+test_expect_success 'apply binary diff without replacement (copy).' \
+       'do_reset &&
+        git apply CF.diff'
+
+test_expect_success 'apply binary diff.' \
+       'do_reset &&
+        git apply --allow-binary-replacement --index BF.diff &&
+        test -z "$(git diff --name-status binary)"'
+
+test_expect_success 'apply binary diff (copy).' \
+       'do_reset &&
+        git apply --allow-binary-replacement --index CF.diff &&
+        test -z "$(git diff --name-status binary)"'
 
 test_done