2006-05-17 11:55:40 +02:00
|
|
|
#!/bin/sh
|
|
|
|
|
#
|
|
|
|
|
# Copyright (c) 2006 Shawn Pearce
|
|
|
|
|
#
|
|
|
|
|
|
2007-07-03 07:52:14 +02:00
|
|
|
test_description='Test git update-ref and basic ref logging'
|
2006-05-17 11:55:40 +02:00
|
|
|
. ./test-lib.sh
|
|
|
|
|
|
2018-05-13 04:24:13 +02:00
|
|
|
Z=$ZERO_OID
|
2008-01-16 00:50:17 +01:00
|
|
|
|
2017-01-27 11:09:48 +01:00
|
|
|
m=refs/heads/master
|
|
|
|
|
n_dir=refs/heads/gu
|
|
|
|
|
n=$n_dir/fixes
|
|
|
|
|
outside=refs/foo
|
|
|
|
|
bare=bare-repo
|
2008-01-16 00:50:17 +01:00
|
|
|
|
2017-01-27 11:09:48 +01:00
|
|
|
create_test_commits ()
|
|
|
|
|
{
|
|
|
|
|
prfx="$1"
|
2008-01-16 00:50:17 +01:00
|
|
|
for name in A B C D E F
|
|
|
|
|
do
|
|
|
|
|
test_tick &&
|
|
|
|
|
T=$(git write-tree) &&
|
|
|
|
|
sha1=$(echo $name | git commit-tree $T) &&
|
2017-01-27 11:09:48 +01:00
|
|
|
eval $prfx$name=$sha1
|
2008-01-16 00:50:17 +01:00
|
|
|
done
|
2017-01-27 11:09:48 +01:00
|
|
|
}
|
2008-01-16 00:50:17 +01:00
|
|
|
|
2017-01-27 11:09:48 +01:00
|
|
|
test_expect_success setup '
|
|
|
|
|
create_test_commits "" &&
|
|
|
|
|
mkdir $bare &&
|
|
|
|
|
cd $bare &&
|
|
|
|
|
git init --bare &&
|
|
|
|
|
create_test_commits "bare" &&
|
|
|
|
|
cd -
|
2008-01-16 00:50:17 +01:00
|
|
|
'
|
|
|
|
|
|
2017-04-16 04:31:02 +02:00
|
|
|
test_expect_success "create $m" '
|
|
|
|
|
git update-ref $m $A &&
|
|
|
|
|
test $A = $(cat .git/$m)
|
|
|
|
|
'
|
|
|
|
|
test_expect_success "create $m with oldvalue verification" '
|
|
|
|
|
git update-ref $m $B $A &&
|
|
|
|
|
test $B = $(cat .git/$m)
|
|
|
|
|
'
|
2008-05-25 18:14:29 +02:00
|
|
|
test_expect_success "fail to delete $m with stale ref" '
|
|
|
|
|
test_must_fail git update-ref -d $m $A &&
|
|
|
|
|
test $B = "$(cat .git/$m)"
|
|
|
|
|
'
|
|
|
|
|
test_expect_success "delete $m" '
|
2017-03-21 01:56:16 +01:00
|
|
|
test_when_finished "rm -f .git/$m" &&
|
2008-05-25 18:14:29 +02:00
|
|
|
git update-ref -d $m $B &&
|
2017-03-21 01:56:14 +01:00
|
|
|
test_path_is_missing .git/$m
|
2008-05-25 18:14:29 +02:00
|
|
|
'
|
2006-05-17 11:55:40 +02:00
|
|
|
|
2017-03-21 01:56:16 +01:00
|
|
|
test_expect_success "delete $m without oldvalue verification" '
|
|
|
|
|
test_when_finished "rm -f .git/$m" &&
|
2008-06-03 01:34:53 +02:00
|
|
|
git update-ref $m $A &&
|
2017-03-21 01:56:16 +01:00
|
|
|
test $A = $(cat .git/$m) &&
|
2008-06-03 01:34:53 +02:00
|
|
|
git update-ref -d $m &&
|
2017-03-21 01:56:14 +01:00
|
|
|
test_path_is_missing .git/$m
|
2017-03-21 01:56:16 +01:00
|
|
|
'
|
2008-06-03 01:34:53 +02:00
|
|
|
|
2017-03-21 01:56:16 +01:00
|
|
|
test_expect_success "fail to create $n" '
|
|
|
|
|
test_when_finished "rm -f .git/$n_dir" &&
|
|
|
|
|
touch .git/$n_dir &&
|
|
|
|
|
test_must_fail git update-ref $n $A
|
|
|
|
|
'
|
2006-07-29 05:44:51 +02:00
|
|
|
|
2017-04-16 04:31:02 +02:00
|
|
|
test_expect_success "create $m (by HEAD)" '
|
|
|
|
|
git update-ref HEAD $A &&
|
|
|
|
|
test $A = $(cat .git/$m)
|
|
|
|
|
'
|
|
|
|
|
test_expect_success "create $m (by HEAD) with oldvalue verification" '
|
|
|
|
|
git update-ref HEAD $B $A &&
|
|
|
|
|
test $B = $(cat .git/$m)
|
|
|
|
|
'
|
2008-05-25 18:14:29 +02:00
|
|
|
test_expect_success "fail to delete $m (by HEAD) with stale ref" '
|
|
|
|
|
test_must_fail git update-ref -d HEAD $A &&
|
|
|
|
|
test $B = $(cat .git/$m)
|
|
|
|
|
'
|
|
|
|
|
test_expect_success "delete $m (by HEAD)" '
|
2017-03-21 01:56:16 +01:00
|
|
|
test_when_finished "rm -f .git/$m" &&
|
2008-05-25 18:14:29 +02:00
|
|
|
git update-ref -d HEAD $B &&
|
2017-03-21 01:56:14 +01:00
|
|
|
test_path_is_missing .git/$m
|
2008-05-25 18:14:29 +02:00
|
|
|
'
|
2006-05-17 11:55:40 +02:00
|
|
|
|
2017-02-21 02:10:33 +01:00
|
|
|
test_expect_success "deleting current branch adds message to HEAD's log" '
|
2017-03-21 01:56:16 +01:00
|
|
|
test_when_finished "rm -f .git/$m" &&
|
2017-02-21 02:10:33 +01:00
|
|
|
git update-ref $m $A &&
|
|
|
|
|
git symbolic-ref HEAD $m &&
|
|
|
|
|
git update-ref -m delete-$m -d $m &&
|
2017-03-21 01:56:14 +01:00
|
|
|
test_path_is_missing .git/$m &&
|
2017-02-21 02:10:33 +01:00
|
|
|
grep "delete-$m$" .git/logs/HEAD
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success "deleting by HEAD adds message to HEAD's log" '
|
2017-03-21 01:56:16 +01:00
|
|
|
test_when_finished "rm -f .git/$m" &&
|
2017-02-21 02:10:33 +01:00
|
|
|
git update-ref $m $A &&
|
|
|
|
|
git symbolic-ref HEAD $m &&
|
|
|
|
|
git update-ref -m delete-by-head -d HEAD &&
|
2017-03-21 01:56:14 +01:00
|
|
|
test_path_is_missing .git/$m &&
|
2017-02-21 02:10:33 +01:00
|
|
|
grep "delete-by-head$" .git/logs/HEAD
|
|
|
|
|
'
|
|
|
|
|
|
2015-07-21 23:04:55 +02:00
|
|
|
test_expect_success 'update-ref does not create reflogs by default' '
|
|
|
|
|
test_when_finished "git update-ref -d $outside" &&
|
|
|
|
|
git update-ref $outside $A &&
|
|
|
|
|
git rev-parse $A >expect &&
|
|
|
|
|
git rev-parse $outside >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
test_must_fail git reflog exists $outside
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'update-ref creates reflogs with --create-reflog' '
|
|
|
|
|
test_when_finished "git update-ref -d $outside" &&
|
|
|
|
|
git update-ref --create-reflog $outside $A &&
|
|
|
|
|
git rev-parse $A >expect &&
|
|
|
|
|
git rev-parse $outside >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
git reflog exists $outside
|
|
|
|
|
'
|
|
|
|
|
|
2017-01-27 11:09:48 +01:00
|
|
|
test_expect_success 'creates no reflog in bare repository' '
|
|
|
|
|
git -C $bare update-ref $m $bareA &&
|
|
|
|
|
git -C $bare rev-parse $bareA >expect &&
|
|
|
|
|
git -C $bare rev-parse $m >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
test_must_fail git -C $bare reflog exists $m
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'core.logAllRefUpdates=true creates reflog in bare repository' '
|
|
|
|
|
test_when_finished "git -C $bare config --unset core.logAllRefUpdates && \
|
|
|
|
|
rm $bare/logs/$m" &&
|
|
|
|
|
git -C $bare config core.logAllRefUpdates true &&
|
|
|
|
|
git -C $bare update-ref $m $bareB &&
|
|
|
|
|
git -C $bare rev-parse $bareB >expect &&
|
|
|
|
|
git -C $bare rev-parse $m >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
git -C $bare reflog exists $m
|
|
|
|
|
'
|
|
|
|
|
|
2017-01-27 11:09:47 +01:00
|
|
|
test_expect_success 'core.logAllRefUpdates=true does not create reflog by default' '
|
|
|
|
|
test_config core.logAllRefUpdates true &&
|
|
|
|
|
test_when_finished "git update-ref -d $outside" &&
|
|
|
|
|
git update-ref $outside $A &&
|
|
|
|
|
git rev-parse $A >expect &&
|
|
|
|
|
git rev-parse $outside >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
test_must_fail git reflog exists $outside
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'core.logAllRefUpdates=always creates reflog by default' '
|
|
|
|
|
test_config core.logAllRefUpdates always &&
|
|
|
|
|
test_when_finished "git update-ref -d $outside" &&
|
|
|
|
|
git update-ref $outside $A &&
|
|
|
|
|
git rev-parse $A >expect &&
|
|
|
|
|
git rev-parse $outside >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
git reflog exists $outside
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'core.logAllRefUpdates=always creates no reflog for ORIG_HEAD' '
|
|
|
|
|
test_config core.logAllRefUpdates always &&
|
|
|
|
|
git update-ref ORIG_HEAD $A &&
|
|
|
|
|
test_must_fail git reflog exists ORIG_HEAD
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success '--no-create-reflog overrides core.logAllRefUpdates=always' '
|
|
|
|
|
test_config core.logAllRefUpdates true &&
|
|
|
|
|
test_when_finished "git update-ref -d $outside" &&
|
|
|
|
|
git update-ref --no-create-reflog $outside $A &&
|
|
|
|
|
git rev-parse $A >expect &&
|
|
|
|
|
git rev-parse $outside >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
test_must_fail git reflog exists $outside
|
|
|
|
|
'
|
|
|
|
|
|
2017-04-16 04:31:02 +02:00
|
|
|
test_expect_success "create $m (by HEAD)" '
|
|
|
|
|
git update-ref HEAD $A &&
|
|
|
|
|
test $A = $(cat .git/$m)
|
|
|
|
|
'
|
|
|
|
|
test_expect_success 'pack refs' '
|
|
|
|
|
git pack-refs --all
|
|
|
|
|
'
|
|
|
|
|
test_expect_success "move $m (by HEAD)" '
|
|
|
|
|
git update-ref HEAD $B $A &&
|
|
|
|
|
test $B = $(cat .git/$m)
|
|
|
|
|
'
|
2012-10-21 12:40:32 +02:00
|
|
|
test_expect_success "delete $m (by HEAD) should remove both packed and loose $m" '
|
2017-03-21 01:56:16 +01:00
|
|
|
test_when_finished "rm -f .git/$m" &&
|
2012-10-21 12:40:31 +02:00
|
|
|
git update-ref -d HEAD $B &&
|
|
|
|
|
! grep "$m" .git/packed-refs &&
|
2017-03-21 01:56:14 +01:00
|
|
|
test_path_is_missing .git/$m
|
2012-10-21 12:40:31 +02:00
|
|
|
'
|
|
|
|
|
|
2008-10-26 03:33:58 +01:00
|
|
|
cp -f .git/HEAD .git/HEAD.orig
|
2017-04-16 04:31:02 +02:00
|
|
|
test_expect_success 'delete symref without dereference' '
|
2017-03-21 01:56:16 +01:00
|
|
|
test_when_finished "cp -f .git/HEAD.orig .git/HEAD" &&
|
2008-10-26 03:33:58 +01:00
|
|
|
git update-ref --no-deref -d HEAD &&
|
2017-03-21 01:56:14 +01:00
|
|
|
test_path_is_missing .git/HEAD
|
2008-10-26 03:33:58 +01:00
|
|
|
'
|
|
|
|
|
|
2017-04-16 04:31:02 +02:00
|
|
|
test_expect_success 'delete symref without dereference when the referred ref is packed' '
|
2017-03-21 01:56:16 +01:00
|
|
|
test_when_finished "cp -f .git/HEAD.orig .git/HEAD" &&
|
2008-11-01 00:25:44 +01:00
|
|
|
echo foo >foo.c &&
|
|
|
|
|
git add foo.c &&
|
|
|
|
|
git commit -m foo &&
|
|
|
|
|
git pack-refs --all &&
|
|
|
|
|
git update-ref --no-deref -d HEAD &&
|
2017-03-21 01:56:14 +01:00
|
|
|
test_path_is_missing .git/HEAD
|
2008-11-01 00:25:44 +01:00
|
|
|
'
|
2017-03-21 01:56:16 +01:00
|
|
|
|
2008-11-01 00:25:44 +01:00
|
|
|
git update-ref -d $m
|
|
|
|
|
|
2014-09-11 03:22:48 +02:00
|
|
|
test_expect_success 'update-ref -d is not confused by self-reference' '
|
|
|
|
|
git symbolic-ref refs/heads/self refs/heads/self &&
|
|
|
|
|
test_when_finished "rm -f .git/refs/heads/self" &&
|
|
|
|
|
test_path_is_file .git/refs/heads/self &&
|
|
|
|
|
test_must_fail git update-ref -d refs/heads/self &&
|
|
|
|
|
test_path_is_file .git/refs/heads/self
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'update-ref --no-deref -d can delete self-reference' '
|
|
|
|
|
git symbolic-ref refs/heads/self refs/heads/self &&
|
|
|
|
|
test_when_finished "rm -f .git/refs/heads/self" &&
|
|
|
|
|
test_path_is_file .git/refs/heads/self &&
|
|
|
|
|
git update-ref --no-deref -d refs/heads/self &&
|
|
|
|
|
test_path_is_missing .git/refs/heads/self
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'update-ref --no-deref -d can delete reference to bad ref' '
|
|
|
|
|
>.git/refs/heads/bad &&
|
|
|
|
|
test_when_finished "rm -f .git/refs/heads/bad" &&
|
|
|
|
|
git symbolic-ref refs/heads/ref-to-bad refs/heads/bad &&
|
|
|
|
|
test_when_finished "rm -f .git/refs/heads/ref-to-bad" &&
|
|
|
|
|
test_path_is_file .git/refs/heads/ref-to-bad &&
|
|
|
|
|
git update-ref --no-deref -d refs/heads/ref-to-bad &&
|
|
|
|
|
test_path_is_missing .git/refs/heads/ref-to-bad
|
|
|
|
|
'
|
|
|
|
|
|
2017-04-16 04:31:02 +02:00
|
|
|
test_expect_success '(not) create HEAD with old sha1' '
|
2008-07-12 17:47:52 +02:00
|
|
|
test_must_fail git update-ref HEAD $A $B
|
2017-04-16 04:31:02 +02:00
|
|
|
'
|
2017-03-21 01:56:16 +01:00
|
|
|
test_expect_success "(not) prior created .git/$m" '
|
|
|
|
|
test_when_finished "rm -f .git/$m" &&
|
2017-03-21 01:56:14 +01:00
|
|
|
test_path_is_missing .git/$m
|
2017-03-21 01:56:16 +01:00
|
|
|
'
|
2006-05-17 11:55:40 +02:00
|
|
|
|
2017-04-16 04:31:02 +02:00
|
|
|
test_expect_success 'create HEAD' '
|
|
|
|
|
git update-ref HEAD $A
|
|
|
|
|
'
|
|
|
|
|
test_expect_success '(not) change HEAD with wrong SHA1' '
|
2008-07-12 17:47:52 +02:00
|
|
|
test_must_fail git update-ref HEAD $B $Z
|
2017-04-16 04:31:02 +02:00
|
|
|
'
|
2017-03-21 01:56:16 +01:00
|
|
|
test_expect_success "(not) changed .git/$m" '
|
|
|
|
|
test_when_finished "rm -f .git/$m" &&
|
|
|
|
|
! test $B = $(cat .git/$m)
|
2008-02-01 10:50:53 +01:00
|
|
|
'
|
2006-05-17 11:55:40 +02:00
|
|
|
|
2015-07-28 00:57:08 +02:00
|
|
|
rm -f .git/logs/refs/heads/master
|
2017-04-16 04:31:02 +02:00
|
|
|
test_expect_success "create $m (logged by touch)" '
|
|
|
|
|
test_config core.logAllRefUpdates false &&
|
|
|
|
|
GIT_COMMITTER_DATE="2005-05-26 23:30" \
|
|
|
|
|
git update-ref --create-reflog HEAD $A -m "Initial Creation" &&
|
|
|
|
|
test $A = $(cat .git/$m)
|
|
|
|
|
'
|
|
|
|
|
test_expect_success "update $m (logged by touch)" '
|
|
|
|
|
test_config core.logAllRefUpdates false &&
|
|
|
|
|
GIT_COMMITTER_DATE="2005-05-26 23:31" \
|
|
|
|
|
git update-ref HEAD $B $A -m "Switch" &&
|
|
|
|
|
test $B = $(cat .git/$m)
|
|
|
|
|
'
|
|
|
|
|
test_expect_success "set $m (logged by touch)" '
|
|
|
|
|
test_config core.logAllRefUpdates false &&
|
|
|
|
|
GIT_COMMITTER_DATE="2005-05-26 23:41" \
|
|
|
|
|
git update-ref HEAD $A &&
|
|
|
|
|
test $A = $(cat .git/$m)
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'empty directory removal' '
|
2017-01-06 17:22:43 +01:00
|
|
|
git branch d1/d2/r1 HEAD &&
|
|
|
|
|
git branch d1/r2 HEAD &&
|
2017-03-21 01:56:14 +01:00
|
|
|
test_path_is_file .git/refs/heads/d1/d2/r1 &&
|
|
|
|
|
test_path_is_file .git/logs/refs/heads/d1/d2/r1 &&
|
2017-01-06 17:22:43 +01:00
|
|
|
git branch -d d1/d2/r1 &&
|
2017-03-21 01:56:14 +01:00
|
|
|
test_path_is_missing .git/refs/heads/d1/d2 &&
|
|
|
|
|
test_path_is_missing .git/logs/refs/heads/d1/d2 &&
|
|
|
|
|
test_path_is_file .git/refs/heads/d1/r2 &&
|
|
|
|
|
test_path_is_file .git/logs/refs/heads/d1/r2
|
2017-01-06 17:22:43 +01:00
|
|
|
'
|
|
|
|
|
|
2017-04-16 04:31:02 +02:00
|
|
|
test_expect_success 'symref empty directory removal' '
|
2017-01-06 17:22:43 +01:00
|
|
|
git branch e1/e2/r1 HEAD &&
|
|
|
|
|
git branch e1/r2 HEAD &&
|
|
|
|
|
git checkout e1/e2/r1 &&
|
|
|
|
|
test_when_finished "git checkout master" &&
|
2017-03-21 01:56:14 +01:00
|
|
|
test_path_is_file .git/refs/heads/e1/e2/r1 &&
|
|
|
|
|
test_path_is_file .git/logs/refs/heads/e1/e2/r1 &&
|
2017-01-06 17:22:43 +01:00
|
|
|
git update-ref -d HEAD &&
|
2017-03-21 01:56:14 +01:00
|
|
|
test_path_is_missing .git/refs/heads/e1/e2 &&
|
|
|
|
|
test_path_is_missing .git/logs/refs/heads/e1/e2 &&
|
|
|
|
|
test_path_is_file .git/refs/heads/e1/r2 &&
|
|
|
|
|
test_path_is_file .git/logs/refs/heads/e1/r2 &&
|
|
|
|
|
test_path_is_file .git/logs/HEAD
|
2017-01-06 17:22:43 +01:00
|
|
|
'
|
|
|
|
|
|
2006-05-17 11:55:40 +02:00
|
|
|
cat >expect <<EOF
|
|
|
|
|
$Z $A $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150200 +0000 Initial Creation
|
|
|
|
|
$A $B $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150260 +0000 Switch
|
|
|
|
|
$B $A $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150860 +0000
|
|
|
|
|
EOF
|
2017-03-21 01:56:16 +01:00
|
|
|
test_expect_success "verifying $m's log (logged by touch)" '
|
|
|
|
|
test_when_finished "rm -rf .git/$m .git/logs expect" &&
|
|
|
|
|
test_cmp expect .git/logs/$m
|
|
|
|
|
'
|
2006-05-17 11:55:40 +02:00
|
|
|
|
2017-04-16 04:31:02 +02:00
|
|
|
test_expect_success "create $m (logged by config)" '
|
|
|
|
|
test_config core.logAllRefUpdates true &&
|
|
|
|
|
GIT_COMMITTER_DATE="2005-05-26 23:32" \
|
|
|
|
|
git update-ref HEAD $A -m "Initial Creation" &&
|
|
|
|
|
test $A = $(cat .git/$m)
|
|
|
|
|
'
|
|
|
|
|
test_expect_success "update $m (logged by config)" '
|
|
|
|
|
test_config core.logAllRefUpdates true &&
|
|
|
|
|
GIT_COMMITTER_DATE="2005-05-26 23:33" \
|
|
|
|
|
git update-ref HEAD'" $B $A "'-m "Switch" &&
|
|
|
|
|
test $B = $(cat .git/$m)
|
|
|
|
|
'
|
|
|
|
|
test_expect_success "set $m (logged by config)" '
|
|
|
|
|
test_config core.logAllRefUpdates true &&
|
|
|
|
|
GIT_COMMITTER_DATE="2005-05-26 23:43" \
|
|
|
|
|
git update-ref HEAD $A &&
|
|
|
|
|
test $A = $(cat .git/$m)
|
|
|
|
|
'
|
2006-05-17 11:55:40 +02:00
|
|
|
|
|
|
|
|
cat >expect <<EOF
|
|
|
|
|
$Z $A $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150320 +0000 Initial Creation
|
|
|
|
|
$A $B $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150380 +0000 Switch
|
|
|
|
|
$B $A $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150980 +0000
|
|
|
|
|
EOF
|
2017-04-16 04:31:02 +02:00
|
|
|
test_expect_success "verifying $m's log (logged by config)" '
|
|
|
|
|
test_when_finished "rm -f .git/$m .git/logs/$m expect" &&
|
|
|
|
|
test_cmp expect .git/logs/$m
|
|
|
|
|
'
|
2006-05-17 11:55:40 +02:00
|
|
|
|
2019-10-28 13:57:17 +01:00
|
|
|
test_expect_success 'set up for querying the reflog' '
|
|
|
|
|
git update-ref $m $D &&
|
|
|
|
|
cat >.git/logs/$m <<-EOF
|
|
|
|
|
$Z $C $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150320 -0500
|
|
|
|
|
$C $A $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150350 -0500
|
|
|
|
|
$A $B $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150380 -0500
|
|
|
|
|
$F $Z $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150680 -0500
|
|
|
|
|
$Z $E $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150980 -0500
|
|
|
|
|
EOF
|
|
|
|
|
'
|
2006-05-19 09:28:19 +02:00
|
|
|
|
|
|
|
|
ed="Thu, 26 May 2005 18:32:00 -0500"
|
|
|
|
|
gd="Thu, 26 May 2005 18:33:00 -0500"
|
|
|
|
|
ld="Thu, 26 May 2005 18:43:00 -0500"
|
2017-04-16 04:31:02 +02:00
|
|
|
test_expect_success 'Query "master@{May 25 2005}" (before history)' '
|
|
|
|
|
test_when_finished "rm -f o e" &&
|
|
|
|
|
git rev-parse --verify "master@{May 25 2005}" >o 2>e &&
|
|
|
|
|
test $C = $(cat o) &&
|
|
|
|
|
test "warning: Log for '\''master'\'' only goes back to $ed." = "$(cat e)"
|
|
|
|
|
'
|
|
|
|
|
test_expect_success 'Query master@{2005-05-25} (before history)' '
|
|
|
|
|
test_when_finished "rm -f o e" &&
|
|
|
|
|
git rev-parse --verify master@{2005-05-25} >o 2>e &&
|
|
|
|
|
test $C = $(cat o) &&
|
2018-09-28 17:43:59 +02:00
|
|
|
test "warning: Log for '\''master'\'' only goes back to $ed." = "$(cat e)"
|
2017-04-16 04:31:02 +02:00
|
|
|
'
|
|
|
|
|
test_expect_success 'Query "master@{May 26 2005 23:31:59}" (1 second before history)' '
|
|
|
|
|
test_when_finished "rm -f o e" &&
|
|
|
|
|
git rev-parse --verify "master@{May 26 2005 23:31:59}" >o 2>e &&
|
|
|
|
|
test $C = $(cat o) &&
|
|
|
|
|
test "warning: Log for '\''master'\'' only goes back to $ed." = "$(cat e)"
|
|
|
|
|
'
|
|
|
|
|
test_expect_success 'Query "master@{May 26 2005 23:32:00}" (exactly history start)' '
|
|
|
|
|
test_when_finished "rm -f o e" &&
|
|
|
|
|
git rev-parse --verify "master@{May 26 2005 23:32:00}" >o 2>e &&
|
|
|
|
|
test $C = $(cat o) &&
|
2019-11-26 20:41:57 +01:00
|
|
|
test_must_be_empty e
|
2017-04-16 04:31:02 +02:00
|
|
|
'
|
|
|
|
|
test_expect_success 'Query "master@{May 26 2005 23:32:30}" (first non-creation change)' '
|
|
|
|
|
test_when_finished "rm -f o e" &&
|
|
|
|
|
git rev-parse --verify "master@{May 26 2005 23:32:30}" >o 2>e &&
|
|
|
|
|
test $A = $(cat o) &&
|
2019-11-26 20:41:57 +01:00
|
|
|
test_must_be_empty e
|
2017-04-16 04:31:02 +02:00
|
|
|
'
|
|
|
|
|
test_expect_success 'Query "master@{2005-05-26 23:33:01}" (middle of history with gap)' '
|
|
|
|
|
test_when_finished "rm -f o e" &&
|
|
|
|
|
git rev-parse --verify "master@{2005-05-26 23:33:01}" >o 2>e &&
|
|
|
|
|
test $B = $(cat o) &&
|
2018-07-21 09:49:35 +02:00
|
|
|
test_i18ngrep -F "warning: log for ref $m has gap after $gd" e
|
2017-04-16 04:31:02 +02:00
|
|
|
'
|
|
|
|
|
test_expect_success 'Query "master@{2005-05-26 23:38:00}" (middle of history)' '
|
|
|
|
|
test_when_finished "rm -f o e" &&
|
|
|
|
|
git rev-parse --verify "master@{2005-05-26 23:38:00}" >o 2>e &&
|
|
|
|
|
test $Z = $(cat o) &&
|
2019-11-26 20:41:57 +01:00
|
|
|
test_must_be_empty e
|
2017-04-16 04:31:02 +02:00
|
|
|
'
|
|
|
|
|
test_expect_success 'Query "master@{2005-05-26 23:43:00}" (exact end of history)' '
|
|
|
|
|
test_when_finished "rm -f o e" &&
|
|
|
|
|
git rev-parse --verify "master@{2005-05-26 23:43:00}" >o 2>e &&
|
|
|
|
|
test $E = $(cat o) &&
|
2019-11-26 20:41:57 +01:00
|
|
|
test_must_be_empty e
|
2017-04-16 04:31:02 +02:00
|
|
|
'
|
|
|
|
|
test_expect_success 'Query "master@{2005-05-28}" (past end of history)' '
|
|
|
|
|
test_when_finished "rm -f o e" &&
|
|
|
|
|
git rev-parse --verify "master@{2005-05-28}" >o 2>e &&
|
|
|
|
|
test $D = $(cat o) &&
|
2018-07-21 09:49:35 +02:00
|
|
|
test_i18ngrep -F "warning: log for ref $m unexpectedly ended on $ld" e
|
2017-04-16 04:31:02 +02:00
|
|
|
'
|
2006-05-19 09:29:43 +02:00
|
|
|
|
|
|
|
|
rm -f .git/$m .git/logs/$m expect
|
|
|
|
|
|
2017-04-16 04:31:02 +02:00
|
|
|
test_expect_success 'creating initial files' '
|
|
|
|
|
test_when_finished rm -f M &&
|
|
|
|
|
echo TEST >F &&
|
|
|
|
|
git add F &&
|
|
|
|
|
GIT_AUTHOR_DATE="2005-05-26 23:30" \
|
|
|
|
|
GIT_COMMITTER_DATE="2005-05-26 23:30" git commit -m add -a &&
|
|
|
|
|
h_TEST=$(git rev-parse --verify HEAD) &&
|
|
|
|
|
echo The other day this did not work. >M &&
|
|
|
|
|
echo And then Bob told me how to fix it. >>M &&
|
|
|
|
|
echo OTHER >F &&
|
|
|
|
|
GIT_AUTHOR_DATE="2005-05-26 23:41" \
|
|
|
|
|
GIT_COMMITTER_DATE="2005-05-26 23:41" git commit -F M -a &&
|
|
|
|
|
h_OTHER=$(git rev-parse --verify HEAD) &&
|
|
|
|
|
GIT_AUTHOR_DATE="2005-05-26 23:44" \
|
|
|
|
|
GIT_COMMITTER_DATE="2005-05-26 23:44" git commit --amend &&
|
|
|
|
|
h_FIXED=$(git rev-parse --verify HEAD) &&
|
|
|
|
|
echo Merged initial commit and a later commit. >M &&
|
|
|
|
|
echo $h_TEST >.git/MERGE_HEAD &&
|
|
|
|
|
GIT_AUTHOR_DATE="2005-05-26 23:45" \
|
|
|
|
|
GIT_COMMITTER_DATE="2005-05-26 23:45" git commit -F M &&
|
|
|
|
|
h_MERGED=$(git rev-parse --verify HEAD)
|
|
|
|
|
'
|
2006-05-25 05:33:18 +02:00
|
|
|
|
|
|
|
|
cat >expect <<EOF
|
2006-07-11 04:48:47 +02:00
|
|
|
$Z $h_TEST $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150200 +0000 commit (initial): add
|
2006-05-25 05:33:18 +02:00
|
|
|
$h_TEST $h_OTHER $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150860 +0000 commit: The other day this did not work.
|
2006-07-11 04:48:47 +02:00
|
|
|
$h_OTHER $h_FIXED $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117151040 +0000 commit (amend): The other day this did not work.
|
|
|
|
|
$h_FIXED $h_MERGED $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117151100 +0000 commit (merge): Merged initial commit and a later commit.
|
2006-05-25 05:33:18 +02:00
|
|
|
EOF
|
2017-04-16 04:31:02 +02:00
|
|
|
test_expect_success 'git commit logged updates' '
|
|
|
|
|
test_cmp expect .git/logs/$m
|
|
|
|
|
'
|
2006-07-11 04:48:47 +02:00
|
|
|
unset h_TEST h_OTHER h_FIXED h_MERGED
|
2006-05-19 09:29:43 +02:00
|
|
|
|
2017-04-16 04:31:02 +02:00
|
|
|
test_expect_success 'git cat-file blob master:F (expect OTHER)' '
|
|
|
|
|
test OTHER = $(git cat-file blob master:F)
|
|
|
|
|
'
|
|
|
|
|
test_expect_success 'git cat-file blob master@{2005-05-26 23:30}:F (expect TEST)' '
|
|
|
|
|
test TEST = $(git cat-file blob "master@{2005-05-26 23:30}:F")
|
|
|
|
|
'
|
|
|
|
|
test_expect_success 'git cat-file blob master@{2005-05-26 23:42}:F (expect OTHER)' '
|
|
|
|
|
test OTHER = $(git cat-file blob "master@{2005-05-26 23:42}:F")
|
|
|
|
|
'
|
2006-05-19 09:29:43 +02:00
|
|
|
|
2018-05-10 21:29:55 +02:00
|
|
|
# Test adding and deleting pseudorefs
|
|
|
|
|
|
|
|
|
|
test_expect_success 'given old value for missing pseudoref, do not create' '
|
|
|
|
|
test_must_fail git update-ref PSEUDOREF $A $B 2>err &&
|
|
|
|
|
test_path_is_missing .git/PSEUDOREF &&
|
2018-07-21 09:49:35 +02:00
|
|
|
test_i18ngrep "could not read ref" err
|
2018-05-10 21:29:55 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'create pseudoref' '
|
|
|
|
|
git update-ref PSEUDOREF $A &&
|
|
|
|
|
test $A = $(cat .git/PSEUDOREF)
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'overwrite pseudoref with no old value given' '
|
|
|
|
|
git update-ref PSEUDOREF $B &&
|
|
|
|
|
test $B = $(cat .git/PSEUDOREF)
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'overwrite pseudoref with correct old value' '
|
|
|
|
|
git update-ref PSEUDOREF $C $B &&
|
|
|
|
|
test $C = $(cat .git/PSEUDOREF)
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'do not overwrite pseudoref with wrong old value' '
|
|
|
|
|
test_must_fail git update-ref PSEUDOREF $D $E 2>err &&
|
|
|
|
|
test $C = $(cat .git/PSEUDOREF) &&
|
2018-07-21 09:49:35 +02:00
|
|
|
test_i18ngrep "unexpected object ID" err
|
2018-05-10 21:29:55 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'delete pseudoref' '
|
|
|
|
|
git update-ref -d PSEUDOREF &&
|
|
|
|
|
test_path_is_missing .git/PSEUDOREF
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'do not delete pseudoref with wrong old value' '
|
|
|
|
|
git update-ref PSEUDOREF $A &&
|
|
|
|
|
test_must_fail git update-ref -d PSEUDOREF $B 2>err &&
|
|
|
|
|
test $A = $(cat .git/PSEUDOREF) &&
|
2018-07-21 09:49:35 +02:00
|
|
|
test_i18ngrep "unexpected object ID" err
|
2018-05-10 21:29:55 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'delete pseudoref with correct old value' '
|
|
|
|
|
git update-ref -d PSEUDOREF $A &&
|
|
|
|
|
test_path_is_missing .git/PSEUDOREF
|
|
|
|
|
'
|
|
|
|
|
|
2018-05-10 21:29:56 +02:00
|
|
|
test_expect_success 'create pseudoref with old OID zero' '
|
2018-05-10 21:29:55 +02:00
|
|
|
git update-ref PSEUDOREF $A $Z &&
|
|
|
|
|
test $A = $(cat .git/PSEUDOREF)
|
|
|
|
|
'
|
|
|
|
|
|
2018-05-10 21:29:56 +02:00
|
|
|
test_expect_success 'do not overwrite pseudoref with old OID zero' '
|
2018-05-10 21:29:55 +02:00
|
|
|
test_when_finished git update-ref -d PSEUDOREF &&
|
|
|
|
|
test_must_fail git update-ref PSEUDOREF $B $Z 2>err &&
|
|
|
|
|
test $A = $(cat .git/PSEUDOREF) &&
|
2018-07-21 09:49:35 +02:00
|
|
|
test_i18ngrep "already exists" err
|
2018-05-10 21:29:55 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
|
|
# Test --stdin
|
|
|
|
|
|
2013-09-11 14:46:18 +02:00
|
|
|
a=refs/heads/a
|
|
|
|
|
b=refs/heads/b
|
|
|
|
|
c=refs/heads/c
|
|
|
|
|
E='""'
|
|
|
|
|
F='%s\0'
|
|
|
|
|
pws='path with space'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin test setup' '
|
|
|
|
|
echo "$pws" >"$pws" &&
|
|
|
|
|
git add -- "$pws" &&
|
|
|
|
|
git commit -m "$pws"
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success '-z fails without --stdin' '
|
|
|
|
|
test_must_fail git update-ref -z $m $m $m 2>err &&
|
2016-06-17 22:21:07 +02:00
|
|
|
test_i18ngrep "usage: git update-ref" err
|
2013-09-11 14:46:18 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin works with no input' '
|
|
|
|
|
>stdin &&
|
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
|
git rev-parse --verify -q $m
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails on empty line' '
|
|
|
|
|
echo "" >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
|
|
|
|
grep "fatal: empty command in input" err
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails on only whitespace' '
|
|
|
|
|
echo " " >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
|
|
|
|
grep "fatal: whitespace before command: " err
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails on leading whitespace' '
|
|
|
|
|
echo " create $a $m" >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
|
|
|
|
grep "fatal: whitespace before command: create $a $m" err
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails on unknown command' '
|
|
|
|
|
echo "unknown $a" >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
|
|
|
|
grep "fatal: unknown command: unknown $a" err
|
|
|
|
|
'
|
|
|
|
|
|
2014-04-07 15:47:55 +02:00
|
|
|
test_expect_success 'stdin fails on unbalanced quotes' '
|
2013-09-11 14:46:18 +02:00
|
|
|
echo "create $a \"master" >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
|
|
|
|
grep "fatal: badly quoted argument: \\\"master" err
|
|
|
|
|
'
|
|
|
|
|
|
2014-04-07 15:47:55 +02:00
|
|
|
test_expect_success 'stdin fails on invalid escape' '
|
|
|
|
|
echo "create $a \"ma\zter\"" >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
|
|
|
|
grep "fatal: badly quoted argument: \\\"ma\\\\zter\\\"" err
|
|
|
|
|
'
|
|
|
|
|
|
2014-04-07 15:47:54 +02:00
|
|
|
test_expect_success 'stdin fails on junk after quoted argument' '
|
2013-09-11 14:46:18 +02:00
|
|
|
echo "create \"$a\"master" >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 15:47:54 +02:00
|
|
|
grep "fatal: unexpected character after quoted argument: \\\"$a\\\"master" err
|
2013-09-11 14:46:18 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails create with no ref' '
|
|
|
|
|
echo "create " >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 15:48:09 +02:00
|
|
|
grep "fatal: create: missing <ref>" err
|
2013-09-11 14:46:18 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails create with no new value' '
|
|
|
|
|
echo "create $a" >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 15:48:09 +02:00
|
|
|
grep "fatal: create $a: missing <newvalue>" err
|
2013-09-11 14:46:18 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails create with too many arguments' '
|
|
|
|
|
echo "create $a $m $m" >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 15:48:09 +02:00
|
|
|
grep "fatal: create $a: extra input: $m" err
|
2013-09-11 14:46:18 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails update with no ref' '
|
|
|
|
|
echo "update " >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 15:48:09 +02:00
|
|
|
grep "fatal: update: missing <ref>" err
|
2013-09-11 14:46:18 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails update with no new value' '
|
|
|
|
|
echo "update $a" >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 15:48:09 +02:00
|
|
|
grep "fatal: update $a: missing <newvalue>" err
|
2013-09-11 14:46:18 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails update with too many arguments' '
|
|
|
|
|
echo "update $a $m $m $m" >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 15:48:09 +02:00
|
|
|
grep "fatal: update $a: extra input: $m" err
|
2013-09-11 14:46:18 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails delete with no ref' '
|
|
|
|
|
echo "delete " >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 15:48:09 +02:00
|
|
|
grep "fatal: delete: missing <ref>" err
|
2013-09-11 14:46:18 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails delete with too many arguments' '
|
|
|
|
|
echo "delete $a $m $m" >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 15:48:09 +02:00
|
|
|
grep "fatal: delete $a: extra input: $m" err
|
2013-09-11 14:46:18 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails verify with too many arguments' '
|
|
|
|
|
echo "verify $a $m $m" >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 15:48:09 +02:00
|
|
|
grep "fatal: verify $a: extra input: $m" err
|
2013-09-11 14:46:18 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails option with unknown name' '
|
|
|
|
|
echo "option unknown" >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
|
|
|
|
grep "fatal: option unknown: unknown" err
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails with duplicate refs' '
|
|
|
|
|
cat >stdin <<-EOF &&
|
|
|
|
|
create $a $m
|
|
|
|
|
create $b $m
|
|
|
|
|
create $a $m
|
|
|
|
|
EOF
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2018-07-21 09:49:35 +02:00
|
|
|
test_i18ngrep "fatal: multiple updates for ref '"'"'$a'"'"' not allowed" err
|
2013-09-11 14:46:18 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin create ref works' '
|
|
|
|
|
echo "create $a $m" >stdin &&
|
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
2015-07-21 23:04:55 +02:00
|
|
|
test_expect_success 'stdin does not create reflogs by default' '
|
|
|
|
|
test_when_finished "git update-ref -d $outside" &&
|
|
|
|
|
echo "create $outside $m" >stdin &&
|
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
git rev-parse $outside >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
test_must_fail git reflog exists $outside
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin creates reflogs with --create-reflog' '
|
2017-01-27 11:09:47 +01:00
|
|
|
test_when_finished "git update-ref -d $outside" &&
|
2015-07-21 23:04:55 +02:00
|
|
|
echo "create $outside $m" >stdin &&
|
|
|
|
|
git update-ref --create-reflog --stdin <stdin &&
|
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
git rev-parse $outside >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
git reflog exists $outside
|
|
|
|
|
'
|
|
|
|
|
|
2014-04-07 15:47:55 +02:00
|
|
|
test_expect_success 'stdin succeeds with quoted argument' '
|
|
|
|
|
git update-ref -d $a &&
|
|
|
|
|
echo "create $a \"$m\"" >stdin &&
|
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin succeeds with escaped character' '
|
|
|
|
|
git update-ref -d $a &&
|
|
|
|
|
echo "create $a \"ma\\163ter\"" >stdin &&
|
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
2013-09-11 14:46:18 +02:00
|
|
|
test_expect_success 'stdin update ref creates with zero old value' '
|
|
|
|
|
echo "update $b $m $Z" >stdin &&
|
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
git update-ref -d $b
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin update ref creates with empty old value' '
|
|
|
|
|
echo "update $b $m $E" >stdin &&
|
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin create ref works with path with space to blob' '
|
|
|
|
|
echo "create refs/blobs/pws \"$m:$pws\"" >stdin &&
|
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
|
git rev-parse "$m:$pws" >expect &&
|
|
|
|
|
git rev-parse refs/blobs/pws >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
git update-ref -d refs/blobs/pws
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin update ref fails with wrong old value' '
|
|
|
|
|
echo "update $c $m $m~1" >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2015-05-23 01:34:57 +02:00
|
|
|
grep "fatal: cannot lock ref '"'"'$c'"'"'" err &&
|
2013-09-11 14:46:18 +02:00
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin update ref fails with bad old value' '
|
|
|
|
|
echo "update $c $m does-not-exist" >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 15:48:02 +02:00
|
|
|
grep "fatal: update $c: invalid <oldvalue>: does-not-exist" err &&
|
2013-09-11 14:46:18 +02:00
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin create ref fails with bad new value' '
|
|
|
|
|
echo "create $c does-not-exist" >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 15:48:02 +02:00
|
|
|
grep "fatal: create $c: invalid <newvalue>: does-not-exist" err &&
|
2013-09-11 14:46:18 +02:00
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin create ref fails with zero new value' '
|
|
|
|
|
echo "create $c " >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 15:48:09 +02:00
|
|
|
grep "fatal: create $c: zero <newvalue>" err &&
|
2013-09-11 14:46:18 +02:00
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin update ref works with right old value' '
|
|
|
|
|
echo "update $b $m~1 $m" >stdin &&
|
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
|
git rev-parse $m~1 >expect &&
|
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin delete ref fails with wrong old value' '
|
|
|
|
|
echo "delete $a $m~1" >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2015-05-23 01:34:57 +02:00
|
|
|
grep "fatal: cannot lock ref '"'"'$a'"'"'" err &&
|
2013-09-11 14:46:18 +02:00
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin delete ref fails with zero old value' '
|
|
|
|
|
echo "delete $a " >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 15:48:09 +02:00
|
|
|
grep "fatal: delete $a: zero <oldvalue>" err &&
|
2013-09-11 14:46:18 +02:00
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin update symref works option no-deref' '
|
|
|
|
|
git symbolic-ref TESTSYMREF $b &&
|
|
|
|
|
cat >stdin <<-EOF &&
|
|
|
|
|
option no-deref
|
|
|
|
|
update TESTSYMREF $a $b
|
|
|
|
|
EOF
|
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
|
git rev-parse TESTSYMREF >expect &&
|
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
git rev-parse $m~1 >expect &&
|
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin delete symref works option no-deref' '
|
|
|
|
|
git symbolic-ref TESTSYMREF $b &&
|
|
|
|
|
cat >stdin <<-EOF &&
|
|
|
|
|
option no-deref
|
|
|
|
|
delete TESTSYMREF $b
|
|
|
|
|
EOF
|
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
|
test_must_fail git rev-parse --verify -q TESTSYMREF &&
|
|
|
|
|
git rev-parse $m~1 >expect &&
|
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
2018-09-05 19:25:50 +02:00
|
|
|
test_expect_success 'stdin update symref works flag --no-deref' '
|
|
|
|
|
git symbolic-ref TESTSYMREFONE $b &&
|
|
|
|
|
git symbolic-ref TESTSYMREFTWO $b &&
|
|
|
|
|
cat >stdin <<-EOF &&
|
|
|
|
|
update TESTSYMREFONE $a $b
|
|
|
|
|
update TESTSYMREFTWO $a $b
|
|
|
|
|
EOF
|
|
|
|
|
git update-ref --no-deref --stdin <stdin &&
|
|
|
|
|
git rev-parse TESTSYMREFONE TESTSYMREFTWO >expect &&
|
|
|
|
|
git rev-parse $a $a >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
git rev-parse $m~1 >expect &&
|
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin delete symref works flag --no-deref' '
|
|
|
|
|
git symbolic-ref TESTSYMREFONE $b &&
|
|
|
|
|
git symbolic-ref TESTSYMREFTWO $b &&
|
|
|
|
|
cat >stdin <<-EOF &&
|
|
|
|
|
delete TESTSYMREFONE $b
|
|
|
|
|
delete TESTSYMREFTWO $b
|
|
|
|
|
EOF
|
|
|
|
|
git update-ref --no-deref --stdin <stdin &&
|
|
|
|
|
test_must_fail git rev-parse --verify -q TESTSYMREFONE &&
|
|
|
|
|
test_must_fail git rev-parse --verify -q TESTSYMREFTWO &&
|
|
|
|
|
git rev-parse $m~1 >expect &&
|
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
2013-09-11 14:46:18 +02:00
|
|
|
test_expect_success 'stdin delete ref works with right old value' '
|
|
|
|
|
echo "delete $b $m~1" >stdin &&
|
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
|
test_must_fail git rev-parse --verify -q $b
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin update/create/verify combination works' '
|
|
|
|
|
cat >stdin <<-EOF &&
|
|
|
|
|
update $a $m
|
|
|
|
|
create $b $m
|
|
|
|
|
verify $c
|
|
|
|
|
EOF
|
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
|
'
|
|
|
|
|
|
2014-12-11 00:47:51 +01:00
|
|
|
test_expect_success 'stdin verify succeeds for correct value' '
|
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
echo "verify $m $m" >stdin &&
|
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
|
git rev-parse $m >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin verify succeeds for missing reference' '
|
|
|
|
|
echo "verify refs/heads/missing $Z" >stdin &&
|
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
|
test_must_fail git rev-parse --verify -q refs/heads/missing
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin verify treats no value as missing' '
|
|
|
|
|
echo "verify refs/heads/missing" >stdin &&
|
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
|
test_must_fail git rev-parse --verify -q refs/heads/missing
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin verify fails for wrong value' '
|
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
echo "verify $m $m~1" >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin &&
|
|
|
|
|
git rev-parse $m >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin verify fails for mistaken null value' '
|
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
echo "verify $m $Z" >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin &&
|
|
|
|
|
git rev-parse $m >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
2014-12-11 00:47:52 +01:00
|
|
|
test_expect_success 'stdin verify fails for mistaken empty value' '
|
2014-12-11 00:47:51 +01:00
|
|
|
M=$(git rev-parse $m) &&
|
|
|
|
|
test_when_finished "git update-ref $m $M" &&
|
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
echo "verify $m" >stdin &&
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin &&
|
|
|
|
|
git rev-parse $m >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
2013-09-11 14:46:18 +02:00
|
|
|
test_expect_success 'stdin update refs works with identity updates' '
|
|
|
|
|
cat >stdin <<-EOF &&
|
|
|
|
|
update $a $m $m
|
|
|
|
|
update $b $m $m
|
|
|
|
|
update $c $Z $E
|
|
|
|
|
EOF
|
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin update refs fails with wrong old value' '
|
|
|
|
|
git update-ref $c $m &&
|
|
|
|
|
cat >stdin <<-EOF &&
|
|
|
|
|
update $a $m $m
|
|
|
|
|
update $b $m $m
|
|
|
|
|
update $c ''
|
|
|
|
|
EOF
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2015-05-23 01:34:57 +02:00
|
|
|
grep "fatal: cannot lock ref '"'"'$c'"'"'" err &&
|
2013-09-11 14:46:18 +02:00
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
git rev-parse $c >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin delete refs works with packed and loose refs' '
|
|
|
|
|
git pack-refs --all &&
|
|
|
|
|
git update-ref $c $m~1 &&
|
|
|
|
|
cat >stdin <<-EOF &&
|
|
|
|
|
delete $a $m
|
|
|
|
|
update $b $Z $m
|
|
|
|
|
update $c $E $m~1
|
|
|
|
|
EOF
|
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
|
test_must_fail git rev-parse --verify -q $a &&
|
|
|
|
|
test_must_fail git rev-parse --verify -q $b &&
|
|
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z works on empty input' '
|
|
|
|
|
>stdin &&
|
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
|
git rev-parse --verify -q $m
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails on empty line' '
|
|
|
|
|
echo "" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
|
|
|
|
grep "fatal: whitespace before command: " err
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails on empty command' '
|
|
|
|
|
printf $F "" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
|
|
|
|
grep "fatal: empty command in input" err
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails on only whitespace' '
|
|
|
|
|
printf $F " " >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
|
|
|
|
grep "fatal: whitespace before command: " err
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails on leading whitespace' '
|
|
|
|
|
printf $F " create $a" "$m" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
|
|
|
|
grep "fatal: whitespace before command: create $a" err
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails on unknown command' '
|
|
|
|
|
printf $F "unknown $a" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
|
|
|
|
grep "fatal: unknown command: unknown $a" err
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails create with no ref' '
|
|
|
|
|
printf $F "create " >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 15:48:09 +02:00
|
|
|
grep "fatal: create: missing <ref>" err
|
2013-09-11 14:46:18 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails create with no new value' '
|
|
|
|
|
printf $F "create $a" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 15:48:08 +02:00
|
|
|
grep "fatal: create $a: unexpected end of input when reading <newvalue>" err
|
2013-09-11 14:46:18 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails create with too many arguments' '
|
|
|
|
|
printf $F "create $a" "$m" "$m" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
|
|
|
|
grep "fatal: unknown command: $m" err
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails update with no ref' '
|
|
|
|
|
printf $F "update " >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 15:48:09 +02:00
|
|
|
grep "fatal: update: missing <ref>" err
|
2013-09-11 14:46:18 +02:00
|
|
|
'
|
|
|
|
|
|
2014-04-07 15:48:07 +02:00
|
|
|
test_expect_success 'stdin -z fails update with too few args' '
|
|
|
|
|
printf $F "update $a" "$m" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 15:48:08 +02:00
|
|
|
grep "fatal: update $a: unexpected end of input when reading <oldvalue>" err
|
2013-09-11 14:46:18 +02:00
|
|
|
'
|
|
|
|
|
|
2014-04-07 15:48:06 +02:00
|
|
|
test_expect_success 'stdin -z emits warning with empty new value' '
|
2014-04-07 15:48:04 +02:00
|
|
|
git update-ref $a $m &&
|
|
|
|
|
printf $F "update $a" "" "" >stdin &&
|
2014-04-07 15:48:06 +02:00
|
|
|
git update-ref -z --stdin <stdin 2>err &&
|
|
|
|
|
grep "warning: update $a: missing <newvalue>, treating as zero" err &&
|
2014-04-07 15:48:04 +02:00
|
|
|
test_must_fail git rev-parse --verify -q $a
|
|
|
|
|
'
|
|
|
|
|
|
2013-09-11 14:46:18 +02:00
|
|
|
test_expect_success 'stdin -z fails update with no new value' '
|
|
|
|
|
printf $F "update $a" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 15:48:08 +02:00
|
|
|
grep "fatal: update $a: unexpected end of input when reading <newvalue>" err
|
2013-09-11 14:46:18 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails update with no old value' '
|
|
|
|
|
printf $F "update $a" "$m" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 15:48:08 +02:00
|
|
|
grep "fatal: update $a: unexpected end of input when reading <oldvalue>" err
|
2013-09-11 14:46:18 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails update with too many arguments' '
|
|
|
|
|
printf $F "update $a" "$m" "$m" "$m" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
|
|
|
|
grep "fatal: unknown command: $m" err
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails delete with no ref' '
|
|
|
|
|
printf $F "delete " >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 15:48:09 +02:00
|
|
|
grep "fatal: delete: missing <ref>" err
|
2013-09-11 14:46:18 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails delete with no old value' '
|
|
|
|
|
printf $F "delete $a" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 15:48:08 +02:00
|
|
|
grep "fatal: delete $a: unexpected end of input when reading <oldvalue>" err
|
2013-09-11 14:46:18 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails delete with too many arguments' '
|
|
|
|
|
printf $F "delete $a" "$m" "$m" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
|
|
|
|
grep "fatal: unknown command: $m" err
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails verify with too many arguments' '
|
|
|
|
|
printf $F "verify $a" "$m" "$m" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
|
|
|
|
grep "fatal: unknown command: $m" err
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails verify with no old value' '
|
|
|
|
|
printf $F "verify $a" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 15:48:08 +02:00
|
|
|
grep "fatal: verify $a: unexpected end of input when reading <oldvalue>" err
|
2013-09-11 14:46:18 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails option with unknown name' '
|
|
|
|
|
printf $F "option unknown" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
|
|
|
|
grep "fatal: option unknown: unknown" err
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails with duplicate refs' '
|
|
|
|
|
printf $F "create $a" "$m" "create $b" "$m" "create $a" "$m" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2018-07-21 09:49:35 +02:00
|
|
|
test_i18ngrep "fatal: multiple updates for ref '"'"'$a'"'"' not allowed" err
|
2013-09-11 14:46:18 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z create ref works' '
|
|
|
|
|
printf $F "create $a" "$m" >stdin &&
|
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z update ref creates with zero old value' '
|
|
|
|
|
printf $F "update $b" "$m" "$Z" >stdin &&
|
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
git update-ref -d $b
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z update ref creates with empty old value' '
|
|
|
|
|
printf $F "update $b" "$m" "" >stdin &&
|
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z create ref works with path with space to blob' '
|
|
|
|
|
printf $F "create refs/blobs/pws" "$m:$pws" >stdin &&
|
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
|
git rev-parse "$m:$pws" >expect &&
|
|
|
|
|
git rev-parse refs/blobs/pws >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
git update-ref -d refs/blobs/pws
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z update ref fails with wrong old value' '
|
|
|
|
|
printf $F "update $c" "$m" "$m~1" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2015-05-23 01:34:57 +02:00
|
|
|
grep "fatal: cannot lock ref '"'"'$c'"'"'" err &&
|
2013-09-11 14:46:18 +02:00
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z update ref fails with bad old value' '
|
|
|
|
|
printf $F "update $c" "$m" "does-not-exist" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 15:48:02 +02:00
|
|
|
grep "fatal: update $c: invalid <oldvalue>: does-not-exist" err &&
|
2013-09-11 14:46:18 +02:00
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
|
'
|
|
|
|
|
|
2014-04-02 10:09:54 +02:00
|
|
|
test_expect_success 'stdin -z create ref fails when ref exists' '
|
|
|
|
|
git update-ref $c $m &&
|
|
|
|
|
git rev-parse "$c" >expect &&
|
|
|
|
|
printf $F "create $c" "$m~1" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2015-05-23 01:34:57 +02:00
|
|
|
grep "fatal: cannot lock ref '"'"'$c'"'"'" err &&
|
2014-04-02 10:09:54 +02:00
|
|
|
git rev-parse "$c" >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
2013-09-11 14:46:18 +02:00
|
|
|
test_expect_success 'stdin -z create ref fails with bad new value' '
|
2014-04-02 10:09:54 +02:00
|
|
|
git update-ref -d "$c" &&
|
2013-09-11 14:46:18 +02:00
|
|
|
printf $F "create $c" "does-not-exist" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 15:48:02 +02:00
|
|
|
grep "fatal: create $c: invalid <newvalue>: does-not-exist" err &&
|
2013-09-11 14:46:18 +02:00
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
|
'
|
|
|
|
|
|
update-ref.c: extract a new function, parse_next_sha1()
Replace three functions, update_store_new_sha1(),
update_store_old_sha1(), and parse_next_arg(), with a single function,
parse_next_sha1(). The new function takes care of a whole argument,
including checking whether it is there, converting it to an SHA-1, and
emitting errors on EOF or for invalid values. The return value
indicates whether the argument was present or absent, which requires
a bit of intelligence because absent values are represented
differently depending on whether "-z" was used.
The new interface means that the calling functions, parse_cmd_*(),
don't have to interpret the result differently based on the
line_termination mode that is in effect. It also means that
parse_cmd_create() can distinguish unambiguously between an empty new
value and a zeros new value, which fixes a failure in t1400.
Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2014-04-07 15:48:05 +02:00
|
|
|
test_expect_success 'stdin -z create ref fails with empty new value' '
|
2013-09-11 14:46:18 +02:00
|
|
|
printf $F "create $c" "" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 15:48:09 +02:00
|
|
|
grep "fatal: create $c: missing <newvalue>" err &&
|
2013-09-11 14:46:18 +02:00
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z update ref works with right old value' '
|
|
|
|
|
printf $F "update $b" "$m~1" "$m" >stdin &&
|
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
|
git rev-parse $m~1 >expect &&
|
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z delete ref fails with wrong old value' '
|
|
|
|
|
printf $F "delete $a" "$m~1" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2015-05-23 01:34:57 +02:00
|
|
|
grep "fatal: cannot lock ref '"'"'$a'"'"'" err &&
|
2013-09-11 14:46:18 +02:00
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z delete ref fails with zero old value' '
|
|
|
|
|
printf $F "delete $a" "$Z" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 15:48:09 +02:00
|
|
|
grep "fatal: delete $a: zero <oldvalue>" err &&
|
2013-09-11 14:46:18 +02:00
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z update symref works option no-deref' '
|
|
|
|
|
git symbolic-ref TESTSYMREF $b &&
|
|
|
|
|
printf $F "option no-deref" "update TESTSYMREF" "$a" "$b" >stdin &&
|
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
|
git rev-parse TESTSYMREF >expect &&
|
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
git rev-parse $m~1 >expect &&
|
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z delete symref works option no-deref' '
|
|
|
|
|
git symbolic-ref TESTSYMREF $b &&
|
|
|
|
|
printf $F "option no-deref" "delete TESTSYMREF" "$b" >stdin &&
|
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
|
test_must_fail git rev-parse --verify -q TESTSYMREF &&
|
|
|
|
|
git rev-parse $m~1 >expect &&
|
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z delete ref works with right old value' '
|
|
|
|
|
printf $F "delete $b" "$m~1" >stdin &&
|
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
|
test_must_fail git rev-parse --verify -q $b
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z update/create/verify combination works' '
|
|
|
|
|
printf $F "update $a" "$m" "" "create $b" "$m" "verify $c" "" >stdin &&
|
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
|
'
|
|
|
|
|
|
2014-12-11 00:47:51 +01:00
|
|
|
test_expect_success 'stdin -z verify succeeds for correct value' '
|
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
printf $F "verify $m" "$m" >stdin &&
|
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
|
git rev-parse $m >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z verify succeeds for missing reference' '
|
|
|
|
|
printf $F "verify refs/heads/missing" "$Z" >stdin &&
|
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
|
test_must_fail git rev-parse --verify -q refs/heads/missing
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z verify treats no value as missing' '
|
|
|
|
|
printf $F "verify refs/heads/missing" "" >stdin &&
|
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
|
test_must_fail git rev-parse --verify -q refs/heads/missing
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z verify fails for wrong value' '
|
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
printf $F "verify $m" "$m~1" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin &&
|
|
|
|
|
git rev-parse $m >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z verify fails for mistaken null value' '
|
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
printf $F "verify $m" "$Z" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin &&
|
|
|
|
|
git rev-parse $m >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
2014-12-11 00:47:52 +01:00
|
|
|
test_expect_success 'stdin -z verify fails for mistaken empty value' '
|
2014-12-11 00:47:51 +01:00
|
|
|
M=$(git rev-parse $m) &&
|
|
|
|
|
test_when_finished "git update-ref $m $M" &&
|
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
printf $F "verify $m" "" >stdin &&
|
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin &&
|
|
|
|
|
git rev-parse $m >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
2013-09-11 14:46:18 +02:00
|
|
|
test_expect_success 'stdin -z update refs works with identity updates' '
|
|
|
|
|
printf $F "update $a" "$m" "$m" "update $b" "$m" "$m" "update $c" "$Z" "" >stdin &&
|
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z update refs fails with wrong old value' '
|
|
|
|
|
git update-ref $c $m &&
|
2014-04-07 15:47:53 +02:00
|
|
|
printf $F "update $a" "$m" "$m" "update $b" "$m" "$m" "update $c" "$m" "$Z" >stdin &&
|
2013-09-11 14:46:18 +02:00
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2015-05-23 01:34:57 +02:00
|
|
|
grep "fatal: cannot lock ref '"'"'$c'"'"'" err &&
|
2013-09-11 14:46:18 +02:00
|
|
|
git rev-parse $m >expect &&
|
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
git rev-parse $c >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z delete refs works with packed and loose refs' '
|
|
|
|
|
git pack-refs --all &&
|
|
|
|
|
git update-ref $c $m~1 &&
|
|
|
|
|
printf $F "delete $a" "$m" "update $b" "$Z" "$m" "update $c" "" "$m~1" >stdin &&
|
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
|
test_must_fail git rev-parse --verify -q $a &&
|
|
|
|
|
test_must_fail git rev-parse --verify -q $b &&
|
|
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
|
'
|
|
|
|
|
|
refs: resolve symbolic refs first
Before committing ref updates, split symbolic ref updates into two
parts: an update to the underlying ref, and a log-only update to the
symbolic ref. This ensures that both references are locked correctly
during the transaction, including while their reflogs are updated.
Similarly, if the reference pointed to by HEAD is modified directly, add
a separate log-only update to HEAD, rather than leaving the job of
updating HEAD's reflog to commit_ref_update(). This change ensures that
HEAD is locked correctly while its reflog is being modified, as well as
being cheaper (HEAD only needs to be resolved once).
This makes use of a new function, lock_raw_ref(), which is analogous to
read_raw_ref(), but acquires a lock on the reference before reading it.
This change still has two problems:
* There are redundant read_ref_full() reference lookups.
* It is still possible to get incorrect reflogs for symbolic references
if there is a concurrent update by another process, since the old_oid
of a symref is determined before the lock on the pointed-to ref is
held.
Both problems will soon be fixed.
Signed-off-by: David Turner <dturner@twopensource.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>
WIP
2016-04-25 15:56:07 +02:00
|
|
|
test_expect_success 'fails with duplicate HEAD update' '
|
|
|
|
|
git branch target1 $A &&
|
|
|
|
|
git checkout target1 &&
|
|
|
|
|
cat >stdin <<-EOF &&
|
|
|
|
|
update refs/heads/target1 $C
|
|
|
|
|
option no-deref
|
|
|
|
|
update HEAD $B
|
|
|
|
|
EOF
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2018-07-21 09:49:35 +02:00
|
|
|
test_i18ngrep "fatal: multiple updates for '\''HEAD'\'' (including one via its referent .refs/heads/target1.) are not allowed" err &&
|
refs: resolve symbolic refs first
Before committing ref updates, split symbolic ref updates into two
parts: an update to the underlying ref, and a log-only update to the
symbolic ref. This ensures that both references are locked correctly
during the transaction, including while their reflogs are updated.
Similarly, if the reference pointed to by HEAD is modified directly, add
a separate log-only update to HEAD, rather than leaving the job of
updating HEAD's reflog to commit_ref_update(). This change ensures that
HEAD is locked correctly while its reflog is being modified, as well as
being cheaper (HEAD only needs to be resolved once).
This makes use of a new function, lock_raw_ref(), which is analogous to
read_raw_ref(), but acquires a lock on the reference before reading it.
This change still has two problems:
* There are redundant read_ref_full() reference lookups.
* It is still possible to get incorrect reflogs for symbolic references
if there is a concurrent update by another process, since the old_oid
of a symref is determined before the lock on the pointed-to ref is
held.
Both problems will soon be fixed.
Signed-off-by: David Turner <dturner@twopensource.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>
WIP
2016-04-25 15:56:07 +02:00
|
|
|
echo "refs/heads/target1" >expect &&
|
|
|
|
|
git symbolic-ref HEAD >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
echo "$A" >expect &&
|
|
|
|
|
git rev-parse refs/heads/target1 >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
|
|
|
|
test_expect_success 'fails with duplicate ref update via symref' '
|
|
|
|
|
git branch target2 $A &&
|
|
|
|
|
git symbolic-ref refs/heads/symref2 refs/heads/target2 &&
|
|
|
|
|
cat >stdin <<-EOF &&
|
|
|
|
|
update refs/heads/target2 $C
|
|
|
|
|
update refs/heads/symref2 $B
|
|
|
|
|
EOF
|
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2018-07-21 09:49:35 +02:00
|
|
|
test_i18ngrep "fatal: multiple updates for '\''refs/heads/target2'\'' (including one via symref .refs/heads/symref2.) are not allowed" err &&
|
refs: resolve symbolic refs first
Before committing ref updates, split symbolic ref updates into two
parts: an update to the underlying ref, and a log-only update to the
symbolic ref. This ensures that both references are locked correctly
during the transaction, including while their reflogs are updated.
Similarly, if the reference pointed to by HEAD is modified directly, add
a separate log-only update to HEAD, rather than leaving the job of
updating HEAD's reflog to commit_ref_update(). This change ensures that
HEAD is locked correctly while its reflog is being modified, as well as
being cheaper (HEAD only needs to be resolved once).
This makes use of a new function, lock_raw_ref(), which is analogous to
read_raw_ref(), but acquires a lock on the reference before reading it.
This change still has two problems:
* There are redundant read_ref_full() reference lookups.
* It is still possible to get incorrect reflogs for symbolic references
if there is a concurrent update by another process, since the old_oid
of a symref is determined before the lock on the pointed-to ref is
held.
Both problems will soon be fixed.
Signed-off-by: David Turner <dturner@twopensource.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>
WIP
2016-04-25 15:56:07 +02:00
|
|
|
echo "refs/heads/target2" >expect &&
|
|
|
|
|
git symbolic-ref refs/heads/symref2 >actual &&
|
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
echo "$A" >expect &&
|
|
|
|
|
git rev-parse refs/heads/target2 >actual &&
|
|
|
|
|
test_cmp expect actual
|
|
|
|
|
'
|
|
|
|
|
|
2015-04-15 00:25:06 +02:00
|
|
|
run_with_limited_open_files () {
|
|
|
|
|
(ulimit -n 32 && "$@")
|
|
|
|
|
}
|
|
|
|
|
|
2017-09-14 19:24:41 +02:00
|
|
|
test_lazy_prereq ULIMIT_FILE_DESCRIPTORS '
|
|
|
|
|
test_have_prereq !MINGW,!CYGWIN &&
|
|
|
|
|
run_with_limited_open_files true
|
|
|
|
|
'
|
2015-04-15 00:25:06 +02:00
|
|
|
|
ref_transaction_commit(): fix atomicity and avoid fd exhaustion
The old code was roughly
for update in updates:
acquire locks and check old_sha
for update in updates:
if changing value:
write_ref_to_lockfile()
commit_ref_update()
for update in updates:
if deleting value:
unlink()
rewrite packed-refs file
for update in updates:
if reference still locked:
unlock_ref()
This has two problems.
Non-atomic updates
==================
The atomicity of the reference transaction depends on all pre-checks
being done in the first loop, before any changes have started being
committed in the second loop. The problem is that
write_ref_to_lockfile() (previously part of write_ref_sha1()), which
is called from the second loop, contains two more checks:
* It verifies that new_sha1 is a valid object
* If the reference being updated is a branch, it verifies that
new_sha1 points at a commit object (as opposed to a tag, tree, or
blob).
If either of these checks fails, the "transaction" is aborted during
the second loop. But this might happen after some reference updates
have already been permanently committed. In other words, the
all-or-nothing promise of "git update-ref --stdin" could be violated.
So these checks have to be moved to the first loop.
File descriptor exhaustion
==========================
The old code locked all of the references in the first loop, leaving
all of the lockfiles open until later loops. Since we might be
updating a lot of references, this could result in file descriptor
exhaustion.
The solution
============
After this patch, the code looks like
for update in updates:
acquire locks and check old_sha
if changing value:
write_ref_to_lockfile()
else:
close_ref()
for update in updates:
if changing value:
commit_ref_update()
for update in updates:
if deleting value:
unlink()
rewrite packed-refs file
for update in updates:
if reference still locked:
unlock_ref()
This fixes both problems:
1. The pre-checks in write_ref_to_lockfile() are now done in the first
loop, before any changes have been committed. If any of the checks
fails, the whole transaction can now be rolled back correctly.
2. All lockfiles are closed in the first loop immediately after they
are created (either by write_ref_to_lockfile() or by close_ref()).
This means that there is never more than one open lockfile at a
time, preventing file descriptor exhaustion.
To simplify the bookkeeping across loops, add a new REF_NEEDS_COMMIT
bit to update->flags, which keeps track of whether the corresponding
lockfile needs to be committed, as opposed to just unlocked. (Since
"struct ref_update" is internal to the refs module, this change is not
visible to external callers.)
This change fixes two tests in t1400.
Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2015-04-24 13:35:49 +02:00
|
|
|
test_expect_success ULIMIT_FILE_DESCRIPTORS 'large transaction creating branches does not burst open file limit' '
|
2015-04-15 00:25:06 +02:00
|
|
|
(
|
|
|
|
|
for i in $(test_seq 33)
|
|
|
|
|
do
|
|
|
|
|
echo "create refs/heads/$i HEAD"
|
|
|
|
|
done >large_input &&
|
|
|
|
|
run_with_limited_open_files git update-ref --stdin <large_input &&
|
|
|
|
|
git rev-parse --verify -q refs/heads/33
|
|
|
|
|
)
|
|
|
|
|
'
|
|
|
|
|
|
ref_transaction_commit(): fix atomicity and avoid fd exhaustion
The old code was roughly
for update in updates:
acquire locks and check old_sha
for update in updates:
if changing value:
write_ref_to_lockfile()
commit_ref_update()
for update in updates:
if deleting value:
unlink()
rewrite packed-refs file
for update in updates:
if reference still locked:
unlock_ref()
This has two problems.
Non-atomic updates
==================
The atomicity of the reference transaction depends on all pre-checks
being done in the first loop, before any changes have started being
committed in the second loop. The problem is that
write_ref_to_lockfile() (previously part of write_ref_sha1()), which
is called from the second loop, contains two more checks:
* It verifies that new_sha1 is a valid object
* If the reference being updated is a branch, it verifies that
new_sha1 points at a commit object (as opposed to a tag, tree, or
blob).
If either of these checks fails, the "transaction" is aborted during
the second loop. But this might happen after some reference updates
have already been permanently committed. In other words, the
all-or-nothing promise of "git update-ref --stdin" could be violated.
So these checks have to be moved to the first loop.
File descriptor exhaustion
==========================
The old code locked all of the references in the first loop, leaving
all of the lockfiles open until later loops. Since we might be
updating a lot of references, this could result in file descriptor
exhaustion.
The solution
============
After this patch, the code looks like
for update in updates:
acquire locks and check old_sha
if changing value:
write_ref_to_lockfile()
else:
close_ref()
for update in updates:
if changing value:
commit_ref_update()
for update in updates:
if deleting value:
unlink()
rewrite packed-refs file
for update in updates:
if reference still locked:
unlock_ref()
This fixes both problems:
1. The pre-checks in write_ref_to_lockfile() are now done in the first
loop, before any changes have been committed. If any of the checks
fails, the whole transaction can now be rolled back correctly.
2. All lockfiles are closed in the first loop immediately after they
are created (either by write_ref_to_lockfile() or by close_ref()).
This means that there is never more than one open lockfile at a
time, preventing file descriptor exhaustion.
To simplify the bookkeeping across loops, add a new REF_NEEDS_COMMIT
bit to update->flags, which keeps track of whether the corresponding
lockfile needs to be committed, as opposed to just unlocked. (Since
"struct ref_update" is internal to the refs module, this change is not
visible to external callers.)
This change fixes two tests in t1400.
Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2015-04-24 13:35:49 +02:00
|
|
|
test_expect_success ULIMIT_FILE_DESCRIPTORS 'large transaction deleting branches does not burst open file limit' '
|
2015-04-15 00:25:06 +02:00
|
|
|
(
|
|
|
|
|
for i in $(test_seq 33)
|
|
|
|
|
do
|
|
|
|
|
echo "delete refs/heads/$i HEAD"
|
|
|
|
|
done >large_input &&
|
|
|
|
|
run_with_limited_open_files git update-ref --stdin <large_input &&
|
|
|
|
|
test_must_fail git rev-parse --verify -q refs/heads/33
|
|
|
|
|
)
|
|
|
|
|
'
|
|
|
|
|
|
2015-09-01 04:13:11 +02:00
|
|
|
test_expect_success 'handle per-worktree refs in refs/bisect' '
|
|
|
|
|
git commit --allow-empty -m "initial commit" &&
|
|
|
|
|
git worktree add -b branch worktree &&
|
|
|
|
|
(
|
|
|
|
|
cd worktree &&
|
|
|
|
|
git commit --allow-empty -m "test commit" &&
|
|
|
|
|
git for-each-ref >for-each-ref.out &&
|
|
|
|
|
! grep refs/bisect for-each-ref.out &&
|
|
|
|
|
git update-ref refs/bisect/something HEAD &&
|
|
|
|
|
git rev-parse refs/bisect/something >../worktree-head &&
|
|
|
|
|
git for-each-ref | grep refs/bisect/something
|
|
|
|
|
) &&
|
|
|
|
|
test_path_is_missing .git/refs/bisect &&
|
|
|
|
|
test_must_fail git rev-parse refs/bisect/something &&
|
|
|
|
|
git update-ref refs/bisect/something HEAD &&
|
|
|
|
|
git rev-parse refs/bisect/something >main-head &&
|
|
|
|
|
! test_cmp main-head worktree-head
|
|
|
|
|
'
|
|
|
|
|
|
2006-05-17 11:55:40 +02:00
|
|
|
test_done
|