diff options
| author | Shawn Pearce <spearce@spearce.org> | 2006-07-28 23:44:51 -0400 |
|---|---|---|
| committer | Junio C Hamano <junkio@cox.net> | 2006-07-28 20:53:53 -0700 |
| commit | 818f477c40fee62ab8ea5b493c51bb357f38957c (patch) | |
| tree | 5e03cb93f7518b99b86a9c66246f0e14952266c5 /t | |
| parent | 1b03dfed182a1dc47cc0eb1047a34cd914440ce6 (diff) | |
| download | git-818f477c40fee62ab8ea5b493c51bb357f38957c.tar.gz git-818f477c40fee62ab8ea5b493c51bb357f38957c.tar.xz | |
Display an error from update-ref if target ref name is invalid.
Alex Riesen (raa.lkml@gmail.com) recently observed that git branch
would fail with no error message due to unexpected situations with
regards to refs. For example, if .git/refs/heads/gu is a file but
"git branch -b refs/heads/gu/fixa HEAD" was invoked by the user
it would fail silently due to refs/heads/gu being a file and not
a directory.
This change adds a test for trying to create a ref within a directory
that is actually currently a file, and adds error printing within
the ref locking routine should the resolve operation fail.
The error printing code probably belongs at this level of the library
as other failures within the ref locking, writing and logging code
are also currently at this level of the code.
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
Diffstat (limited to 't')
| -rwxr-xr-x | t/t1400-update-ref.sh | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/t/t1400-update-ref.sh b/t/t1400-update-ref.sh index 04fab2662..ddc80bbea 100755 --- a/t/t1400-update-ref.sh +++ b/t/t1400-update-ref.sh @@ -14,6 +14,8 @@ D=4444444444444444444444444444444444444444 E=5555555555555555555555555555555555555555 F=6666666666666666666666666666666666666666 m=refs/heads/master +n_dir=refs/heads/gu +n=$n_dir/fixes test_expect_success \ "create $m" \ @@ -26,6 +28,16 @@ test_expect_success \ rm -f .git/$m test_expect_success \ + "fail to create $n" \ + 'touch .git/$n_dir + git-update-ref $n $A >out 2>err + test $? = 1 && + test "" = "$(cat out)" && + grep "error: unable to resolve reference" err && + grep $n err' +rm -f .git/$n_dir out err + +test_expect_success \ "create $m (by HEAD)" \ 'git-update-ref HEAD $A && test $A = $(cat .git/$m)' |