mirror of
https://github.com/git/git
synced 2024-11-04 16:17:49 +00:00
4c844c2f49
In add_pattern_to_hashsets(), we remove entries from the recursive_hashmap when adding similar ones to the parent_hashmap. I won't pretend to understand all of what's going on here, but there's an obvious leak: whatever we removed from recursive_hashmap is not referenced anywhere else, and is never free()d. We can easily fix this by asking the hashmap to return a pointer to the old entry. This makes t7002 now completely leak-free. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
518 lines
14 KiB
Bash
Executable file
518 lines
14 KiB
Bash
Executable file
#!/bin/sh
|
|
|
|
test_description='git mv in sparse working trees'
|
|
|
|
TEST_PASSES_SANITIZE_LEAK=true
|
|
. ./test-lib.sh
|
|
|
|
setup_sparse_checkout () {
|
|
mkdir folder1 &&
|
|
touch folder1/file1 &&
|
|
git add folder1 &&
|
|
git sparse-checkout set --cone sub
|
|
}
|
|
|
|
cleanup_sparse_checkout () {
|
|
git sparse-checkout disable &&
|
|
git reset --hard
|
|
}
|
|
|
|
test_expect_success 'setup' "
|
|
mkdir -p sub/dir sub/dir2 &&
|
|
touch a b c sub/d sub/dir/e sub/dir2/e &&
|
|
git add -A &&
|
|
git commit -m files &&
|
|
|
|
cat >sparse_error_header <<-EOF &&
|
|
The following paths and/or pathspecs matched paths that exist
|
|
outside of your sparse-checkout definition, so will not be
|
|
updated in the index:
|
|
EOF
|
|
|
|
cat >sparse_hint <<-EOF &&
|
|
hint: If you intend to update such entries, try one of the following:
|
|
hint: * Use the --sparse option.
|
|
hint: * Disable or modify the sparsity rules.
|
|
hint: Disable this message with \"git config advice.updateSparsePath false\"
|
|
EOF
|
|
|
|
cat >dirty_error_header <<-EOF &&
|
|
The following paths have been moved outside the
|
|
sparse-checkout definition but are not sparse due to local
|
|
modifications.
|
|
EOF
|
|
|
|
cat >dirty_hint <<-EOF
|
|
hint: To correct the sparsity of these paths, do the following:
|
|
hint: * Use \"git add --sparse <paths>\" to update the index
|
|
hint: * Use \"git sparse-checkout reapply\" to apply the sparsity rules
|
|
hint: Disable this message with \"git config advice.updateSparsePath false\"
|
|
EOF
|
|
"
|
|
|
|
test_expect_success 'mv refuses to move sparse-to-sparse' '
|
|
test_when_finished rm -f e &&
|
|
git reset --hard &&
|
|
git sparse-checkout set --no-cone a &&
|
|
touch b &&
|
|
test_must_fail git mv b e 2>stderr &&
|
|
cat sparse_error_header >expect &&
|
|
echo b >>expect &&
|
|
echo e >>expect &&
|
|
cat sparse_hint >>expect &&
|
|
test_cmp expect stderr &&
|
|
git mv --sparse b e 2>stderr &&
|
|
test_must_be_empty stderr
|
|
'
|
|
|
|
test_expect_success 'mv refuses to move sparse-to-sparse, ignores failure' '
|
|
test_when_finished rm -f b c e &&
|
|
git reset --hard &&
|
|
git sparse-checkout set a &&
|
|
|
|
# tracked-to-untracked
|
|
touch b &&
|
|
git mv -k b e 2>stderr &&
|
|
test_path_exists b &&
|
|
test_path_is_missing e &&
|
|
cat sparse_error_header >expect &&
|
|
echo b >>expect &&
|
|
echo e >>expect &&
|
|
cat sparse_hint >>expect &&
|
|
test_cmp expect stderr &&
|
|
|
|
git mv --sparse b e 2>stderr &&
|
|
test_must_be_empty stderr &&
|
|
test_path_is_missing b &&
|
|
test_path_exists e &&
|
|
|
|
# tracked-to-tracked
|
|
git reset --hard &&
|
|
touch b &&
|
|
git mv -k b c 2>stderr &&
|
|
test_path_exists b &&
|
|
test_path_is_missing c &&
|
|
cat sparse_error_header >expect &&
|
|
echo b >>expect &&
|
|
echo c >>expect &&
|
|
cat sparse_hint >>expect &&
|
|
test_cmp expect stderr &&
|
|
|
|
git mv --sparse b c 2>stderr &&
|
|
test_must_be_empty stderr &&
|
|
test_path_is_missing b &&
|
|
test_path_exists c
|
|
'
|
|
|
|
test_expect_success 'mv refuses to move non-sparse-to-sparse' '
|
|
test_when_finished rm -f b c e &&
|
|
git reset --hard &&
|
|
git sparse-checkout set a &&
|
|
|
|
# tracked-to-untracked
|
|
test_must_fail git mv a e 2>stderr &&
|
|
test_path_exists a &&
|
|
test_path_is_missing e &&
|
|
cat sparse_error_header >expect &&
|
|
echo e >>expect &&
|
|
cat sparse_hint >>expect &&
|
|
test_cmp expect stderr &&
|
|
git mv --sparse a e 2>stderr &&
|
|
test_must_be_empty stderr &&
|
|
test_path_is_missing a &&
|
|
test_path_exists e &&
|
|
|
|
# tracked-to-tracked
|
|
rm e &&
|
|
git reset --hard &&
|
|
test_must_fail git mv a c 2>stderr &&
|
|
test_path_exists a &&
|
|
test_path_is_missing c &&
|
|
cat sparse_error_header >expect &&
|
|
echo c >>expect &&
|
|
cat sparse_hint >>expect &&
|
|
test_cmp expect stderr &&
|
|
git mv --sparse a c 2>stderr &&
|
|
test_must_be_empty stderr &&
|
|
test_path_is_missing a &&
|
|
test_path_exists c
|
|
'
|
|
|
|
test_expect_success 'mv refuses to move sparse-to-non-sparse' '
|
|
test_when_finished rm -f b c e &&
|
|
git reset --hard &&
|
|
git sparse-checkout set a e &&
|
|
|
|
# tracked-to-untracked
|
|
touch b &&
|
|
test_must_fail git mv b e 2>stderr &&
|
|
cat sparse_error_header >expect &&
|
|
echo b >>expect &&
|
|
cat sparse_hint >>expect &&
|
|
test_cmp expect stderr &&
|
|
git mv --sparse b e 2>stderr &&
|
|
test_must_be_empty stderr
|
|
'
|
|
|
|
test_expect_success 'recursive mv refuses to move (possible) sparse' '
|
|
test_when_finished rm -rf b c e sub2 &&
|
|
git reset --hard &&
|
|
# Without cone mode, "sub" and "sub2" do not match
|
|
git sparse-checkout set sub/dir sub2/dir &&
|
|
|
|
# Add contained contents to ensure we avoid non-existence errors
|
|
mkdir sub/dir2 &&
|
|
touch sub/d sub/dir2/e &&
|
|
|
|
test_must_fail git mv sub sub2 2>stderr &&
|
|
cat sparse_error_header >expect &&
|
|
cat >>expect <<-\EOF &&
|
|
sub/d
|
|
sub2/d
|
|
sub/dir2/e
|
|
sub2/dir2/e
|
|
EOF
|
|
cat sparse_hint >>expect &&
|
|
test_cmp expect stderr &&
|
|
git mv --sparse sub sub2 2>stderr &&
|
|
test_must_be_empty stderr &&
|
|
git commit -m "moved sub to sub2" &&
|
|
git rev-parse HEAD~1:sub >expect &&
|
|
git rev-parse HEAD:sub2 >actual &&
|
|
test_cmp expect actual &&
|
|
git reset --hard HEAD~1
|
|
'
|
|
|
|
test_expect_success 'recursive mv refuses to move sparse' '
|
|
git reset --hard &&
|
|
# Use cone mode so "sub/" matches the sparse-checkout patterns
|
|
git sparse-checkout init --cone &&
|
|
git sparse-checkout set sub/dir sub2/dir &&
|
|
|
|
# Add contained contents to ensure we avoid non-existence errors
|
|
mkdir sub/dir2 &&
|
|
touch sub/dir2/e &&
|
|
|
|
test_must_fail git mv sub sub2 2>stderr &&
|
|
cat sparse_error_header >expect &&
|
|
cat >>expect <<-\EOF &&
|
|
sub/dir2/e
|
|
sub2/dir2/e
|
|
EOF
|
|
cat sparse_hint >>expect &&
|
|
test_cmp expect stderr &&
|
|
git mv --sparse sub sub2 2>stderr &&
|
|
test_must_be_empty stderr &&
|
|
git commit -m "moved sub to sub2" &&
|
|
git rev-parse HEAD~1:sub >expect &&
|
|
git rev-parse HEAD:sub2 >actual &&
|
|
test_cmp expect actual &&
|
|
git reset --hard HEAD~1
|
|
'
|
|
|
|
test_expect_success 'can move files to non-sparse dir' '
|
|
git reset --hard &&
|
|
git sparse-checkout init --no-cone &&
|
|
git sparse-checkout set a b c w !/x y/ &&
|
|
mkdir -p w x/y &&
|
|
|
|
git mv a w/new-a 2>stderr &&
|
|
git mv b x/y/new-b 2>stderr &&
|
|
test_must_be_empty stderr
|
|
'
|
|
|
|
test_expect_success 'refuse to move file to non-skip-worktree sparse path' '
|
|
test_when_finished "cleanup_sparse_checkout" &&
|
|
git reset --hard &&
|
|
git sparse-checkout init --no-cone &&
|
|
git sparse-checkout set a !/x y/ !x/y/z &&
|
|
mkdir -p x/y/z &&
|
|
|
|
test_must_fail git mv a x/y/z/new-a 2>stderr &&
|
|
echo x/y/z/new-a | cat sparse_error_header - sparse_hint >expect &&
|
|
test_cmp expect stderr
|
|
'
|
|
|
|
test_expect_success 'refuse to move out-of-cone directory without --sparse' '
|
|
test_when_finished "cleanup_sparse_checkout" &&
|
|
setup_sparse_checkout &&
|
|
|
|
test_must_fail git mv folder1 sub 2>stderr &&
|
|
cat sparse_error_header >expect &&
|
|
echo folder1/file1 >>expect &&
|
|
cat sparse_hint >>expect &&
|
|
test_cmp expect stderr
|
|
'
|
|
|
|
test_expect_success 'can move out-of-cone directory with --sparse' '
|
|
test_when_finished "cleanup_sparse_checkout" &&
|
|
setup_sparse_checkout &&
|
|
|
|
git mv --sparse folder1 sub 2>stderr &&
|
|
test_must_be_empty stderr &&
|
|
|
|
test_path_is_dir sub/folder1 &&
|
|
test_path_is_file sub/folder1/file1
|
|
'
|
|
|
|
test_expect_success 'refuse to move out-of-cone file without --sparse' '
|
|
test_when_finished "cleanup_sparse_checkout" &&
|
|
setup_sparse_checkout &&
|
|
|
|
test_must_fail git mv folder1/file1 sub 2>stderr &&
|
|
cat sparse_error_header >expect &&
|
|
echo folder1/file1 >>expect &&
|
|
cat sparse_hint >>expect &&
|
|
test_cmp expect stderr
|
|
'
|
|
|
|
test_expect_success 'can move out-of-cone file with --sparse' '
|
|
test_when_finished "cleanup_sparse_checkout" &&
|
|
setup_sparse_checkout &&
|
|
|
|
git mv --sparse folder1/file1 sub 2>stderr &&
|
|
test_must_be_empty stderr &&
|
|
|
|
test_path_is_file sub/file1
|
|
'
|
|
|
|
test_expect_success 'refuse to move sparse file to existing destination' '
|
|
test_when_finished "cleanup_sparse_checkout" &&
|
|
mkdir folder1 &&
|
|
touch folder1/file1 &&
|
|
touch sub/file1 &&
|
|
git add folder1 sub/file1 &&
|
|
git sparse-checkout set --cone sub &&
|
|
|
|
test_must_fail git mv --sparse folder1/file1 sub 2>stderr &&
|
|
echo "fatal: destination exists, source=folder1/file1, destination=sub/file1" >expect &&
|
|
test_cmp expect stderr
|
|
'
|
|
|
|
test_expect_success 'move sparse file to existing destination with --force and --sparse' '
|
|
test_when_finished "cleanup_sparse_checkout" &&
|
|
mkdir folder1 &&
|
|
touch folder1/file1 &&
|
|
touch sub/file1 &&
|
|
echo "overwrite" >folder1/file1 &&
|
|
git add folder1 sub/file1 &&
|
|
git sparse-checkout set --cone sub &&
|
|
|
|
git mv --sparse --force folder1/file1 sub 2>stderr &&
|
|
test_must_be_empty stderr &&
|
|
echo "overwrite" >expect &&
|
|
test_cmp expect sub/file1
|
|
'
|
|
|
|
test_expect_success '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_success '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_success '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_success '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_success '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 &&
|
|
cat dirty_error_header >expect &&
|
|
echo "folder1/d" >>expect &&
|
|
cat dirty_hint >>expect &&
|
|
test_cmp expect 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_success 'move dir from in-cone to out-of-cone' '
|
|
test_when_finished "cleanup_sparse_checkout" &&
|
|
setup_sparse_checkout &&
|
|
mkdir sub/dir/deep &&
|
|
|
|
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 sub/dir &&
|
|
test_path_is_missing folder1 &&
|
|
git ls-files -t >actual &&
|
|
! grep "H sub/dir/e" actual &&
|
|
grep "S folder1/dir/e" actual
|
|
'
|
|
|
|
test_expect_success 'move partially-dirty dir from in-cone to out-of-cone' '
|
|
test_when_finished "cleanup_sparse_checkout" &&
|
|
setup_sparse_checkout &&
|
|
mkdir sub/dir/deep &&
|
|
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 &&
|
|
cat dirty_error_header >expect &&
|
|
echo "folder1/dir/e2" >>expect &&
|
|
echo "folder1/dir/e3" >>expect &&
|
|
cat dirty_hint >>expect &&
|
|
test_cmp expect stderr &&
|
|
|
|
test_path_is_missing sub/dir &&
|
|
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
|