]> git.ipfire.org Git - thirdparty/git.git/commitdiff
t7002: add tests for moving from in-cone to out-of-cone
authorShaoxuan Yuan <shaoxuan.yuan02@gmail.com>
Tue, 9 Aug 2022 12:09:02 +0000 (20:09 +0800)
committerJunio C Hamano <gitster@pobox.com>
Wed, 10 Aug 2022 20:57:48 +0000 (13:57 -0700)
Add corresponding tests to test that user can move an in-cone <source>
to out-of-cone <destination> when --sparse is supplied.

Such <source> can be either clean or dirty, and moving it results in
different behaviors:

A clean move should move <source> to <destination> in the index (do
*not* create <destination> in the worktree), then delete <source> from
the worktree.

A dirty move should move the <source> to the <destination>, both in the
working tree and the index, but should *not* remove the resulted path
from the working tree and should *not* turn on its CE_SKIP_WORKTREE bit.

Also make sure that if <destination> exists in the index (existing
check for if <destination> is in the worktree is not enough in
in-to-out moves), warn user against the overwrite. And Git should force
the overwrite when supplied with -f or --force.

Helped-by: Derrick Stolee <derrickstolee@github.com>
Helped-by: Victoria Dye <vdye@github.com>
Signed-off-by: Shaoxuan Yuan <shaoxuan.yuan02@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/t7002-mv-sparse-checkout.sh

index 71fe29690fd1226082f14df5b8e17cef76659e3f..1ac78edde6c2f64b6e88e6803f21487ecea914cc 100755 (executable)
@@ -290,4 +290,202 @@ test_expect_success 'move sparse file to existing destination with --force and -
        test_cmp expect sub/file1
 '
 
+test_expect_failure 'move clean path from in-cone to out-of-cone' '
+       test_when_finished "cleanup_sparse_checkout" &&
+       setup_sparse_checkout &&
+
+       test_must_fail git mv sub/d folder1 2>stderr &&
+       cat sparse_error_header >expect &&
+       echo "folder1/d" >>expect &&
+       cat sparse_hint >>expect &&
+       test_cmp expect stderr &&
+
+       git mv --sparse sub/d folder1 2>stderr &&
+       test_must_be_empty stderr &&
+
+       test_path_is_missing sub/d &&
+       test_path_is_missing folder1/d &&
+       git ls-files -t >actual &&
+       ! grep "^H sub/d\$" actual &&
+       grep "S folder1/d" actual
+'
+
+test_expect_failure 'move clean path from in-cone to out-of-cone overwrite' '
+       test_when_finished "cleanup_sparse_checkout" &&
+       setup_sparse_checkout &&
+       echo "sub/file1 overwrite" >sub/file1 &&
+       git add sub/file1 &&
+
+       test_must_fail git mv sub/file1 folder1 2>stderr &&
+       cat sparse_error_header >expect &&
+       echo "folder1/file1" >>expect &&
+       cat sparse_hint >>expect &&
+       test_cmp expect stderr &&
+
+       test_must_fail git mv --sparse sub/file1 folder1 2>stderr &&
+       echo "fatal: destination exists in the index, source=sub/file1, destination=folder1/file1" \
+       >expect &&
+       test_cmp expect stderr &&
+
+       git mv --sparse -f sub/file1 folder1 2>stderr &&
+       test_must_be_empty stderr &&
+
+       test_path_is_missing sub/file1 &&
+       test_path_is_missing folder1/file1 &&
+       git ls-files -t >actual &&
+       ! grep "H sub/file1" actual &&
+       grep "S folder1/file1" actual &&
+
+       # compare file content before move and after move
+       echo "sub/file1 overwrite" >expect &&
+       git ls-files -s -- folder1/file1 | awk "{print \$2}" >oid &&
+       git cat-file blob $(cat oid) >actual &&
+       test_cmp expect actual
+'
+
+# This test is testing the same behavior as the
+# "move clean path from in-cone to out-of-cone overwrite" above.
+# The only difference is the <destination> changes from "folder1" to "folder1/file1"
+test_expect_failure 'move clean path from in-cone to out-of-cone file overwrite' '
+       test_when_finished "cleanup_sparse_checkout" &&
+       setup_sparse_checkout &&
+       echo "sub/file1 overwrite" >sub/file1 &&
+       git add sub/file1 &&
+
+       test_must_fail git mv sub/file1 folder1/file1 2>stderr &&
+       cat sparse_error_header >expect &&
+       echo "folder1/file1" >>expect &&
+       cat sparse_hint >>expect &&
+       test_cmp expect stderr &&
+
+       test_must_fail git mv --sparse sub/file1 folder1/file1 2>stderr &&
+       echo "fatal: destination exists in the index, source=sub/file1, destination=folder1/file1" \
+       >expect &&
+       test_cmp expect stderr &&
+
+       git mv --sparse -f sub/file1 folder1/file1 2>stderr &&
+       test_must_be_empty stderr &&
+
+       test_path_is_missing sub/file1 &&
+       test_path_is_missing folder1/file1 &&
+       git ls-files -t >actual &&
+       ! grep "H sub/file1" actual &&
+       grep "S folder1/file1" actual &&
+
+       # compare file content before move and after move
+       echo "sub/file1 overwrite" >expect &&
+       git ls-files -s -- folder1/file1 | awk "{print \$2}" >oid &&
+       git cat-file blob $(cat oid) >actual &&
+       test_cmp expect actual
+'
+
+test_expect_failure 'move directory with one of the files overwrite' '
+       test_when_finished "cleanup_sparse_checkout" &&
+       mkdir -p folder1/dir &&
+       touch folder1/dir/file1 &&
+       git add folder1 &&
+       git sparse-checkout set --cone sub &&
+
+       echo test >sub/dir/file1 &&
+       git add sub/dir/file1 &&
+
+       test_must_fail git mv sub/dir folder1 2>stderr &&
+       cat sparse_error_header >expect &&
+       echo "folder1/dir/e" >>expect &&
+       echo "folder1/dir/file1" >>expect &&
+       cat sparse_hint >>expect &&
+       test_cmp expect stderr &&
+
+       test_must_fail git mv --sparse sub/dir folder1 2>stderr &&
+       echo "fatal: destination exists in the index, source=sub/dir/file1, destination=folder1/dir/file1" \
+       >expect &&
+       test_cmp expect stderr &&
+
+       git mv --sparse -f sub/dir folder1 2>stderr &&
+       test_must_be_empty stderr &&
+
+       test_path_is_missing sub/dir/file1 &&
+       test_path_is_missing sub/dir/e &&
+       test_path_is_missing folder1/file1 &&
+       git ls-files -t >actual &&
+       ! grep "H sub/dir/file1" actual &&
+       ! grep "H sub/dir/e" actual &&
+       grep "S folder1/dir/file1" actual &&
+
+       # compare file content before move and after move
+       echo test >expect &&
+       git ls-files -s -- folder1/dir/file1 | awk "{print \$2}" >oid &&
+       git cat-file blob $(cat oid) >actual &&
+       test_cmp expect actual
+'
+
+test_expect_failure 'move dirty path from in-cone to out-of-cone' '
+       test_when_finished "cleanup_sparse_checkout" &&
+       setup_sparse_checkout &&
+       echo "modified" >>sub/d &&
+
+       test_must_fail git mv sub/d folder1 2>stderr &&
+       cat sparse_error_header >expect &&
+       echo "folder1/d" >>expect &&
+       cat sparse_hint >>expect &&
+       test_cmp expect stderr &&
+
+       git mv --sparse sub/d folder1 2>stderr &&
+
+       test_path_is_missing sub/d &&
+       test_path_is_file folder1/d &&
+       git ls-files -t >actual &&
+       ! grep "^H sub/d\$" actual &&
+       grep "H folder1/d" actual
+'
+
+test_expect_failure 'move dir from in-cone to out-of-cone' '
+       test_when_finished "cleanup_sparse_checkout" &&
+       setup_sparse_checkout &&
+
+       test_must_fail git mv sub/dir folder1 2>stderr &&
+       cat sparse_error_header >expect &&
+       echo "folder1/dir/e" >>expect &&
+       cat sparse_hint >>expect &&
+       test_cmp expect stderr &&
+
+       git mv --sparse sub/dir folder1 2>stderr &&
+       test_must_be_empty stderr &&
+
+       test_path_is_missing folder1 &&
+       git ls-files -t >actual &&
+       ! grep "H sub/dir/e" actual &&
+       grep "S folder1/dir/e" actual
+'
+
+test_expect_failure 'move partially-dirty dir from in-cone to out-of-cone' '
+       test_when_finished "cleanup_sparse_checkout" &&
+       setup_sparse_checkout &&
+       touch sub/dir/e2 sub/dir/e3 &&
+       git add sub/dir/e2 sub/dir/e3 &&
+       echo "modified" >>sub/dir/e2 &&
+       echo "modified" >>sub/dir/e3 &&
+
+       test_must_fail git mv sub/dir folder1 2>stderr &&
+       cat sparse_error_header >expect &&
+       echo "folder1/dir/e" >>expect &&
+       echo "folder1/dir/e2" >>expect &&
+       echo "folder1/dir/e3" >>expect &&
+       cat sparse_hint >>expect &&
+       test_cmp expect stderr &&
+
+       git mv --sparse sub/dir folder1 2>stderr &&
+
+       test_path_is_missing folder1/dir/e &&
+       test_path_is_file folder1/dir/e2 &&
+       test_path_is_file folder1/dir/e3 &&
+       git ls-files -t >actual &&
+       ! grep "H sub/dir/e" actual &&
+       ! grep "H sub/dir/e2" actual &&
+       ! grep "H sub/dir/e3" actual &&
+       grep "S folder1/dir/e" actual &&
+       grep "H folder1/dir/e2" actual &&
+       grep "H folder1/dir/e3" actual
+'
+
 test_done