0b294c0abf
If git attempts to delete a ref, but the unlink of the ref file fails, we print a message to stderr. This is usually a good thing, but if the error is ENOENT, then it indicates that the ref has _already_ been deleted. And since that's our goal, it doesn't make sense to complain to the user. This harmonizes the error reporting behavior for the unpacked and packed cases; the packed case already printed nothing on ENOENT, but the unpacked printed unconditionally. Additionally, send-pack would, when deleting the tracking ref corresponding to a remote delete, print "Failed to delete" on any failure. This can be a misleading message, since we actually _did_ delete at the remote side, but we failed to delete locally. Rather than make the message more precise, let's just eliminate it entirely; the delete_ref routine already takes care of printing out a much more specific message about what went wrong. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
61 lines
1.4 KiB
Bash
Executable File
61 lines
1.4 KiB
Bash
Executable File
#!/bin/sh
|
|
|
|
test_description='tracking branch update checks for git push'
|
|
|
|
. ./test-lib.sh
|
|
|
|
test_expect_success 'setup' '
|
|
echo 1 >file &&
|
|
git add file &&
|
|
git commit -m 1 &&
|
|
git branch b1 &&
|
|
git branch b2 &&
|
|
git branch b3 &&
|
|
git clone . aa &&
|
|
git checkout b1 &&
|
|
echo b1 >>file &&
|
|
git commit -a -m b1 &&
|
|
git checkout b2 &&
|
|
echo b2 >>file &&
|
|
git commit -a -m b2
|
|
'
|
|
|
|
test_expect_success 'prepare pushable branches' '
|
|
cd aa &&
|
|
b1=$(git rev-parse origin/b1) &&
|
|
b2=$(git rev-parse origin/b2) &&
|
|
git checkout -b b1 origin/b1 &&
|
|
echo aa-b1 >>file &&
|
|
git commit -a -m aa-b1 &&
|
|
git checkout -b b2 origin/b2 &&
|
|
echo aa-b2 >>file &&
|
|
git commit -a -m aa-b2 &&
|
|
git checkout master &&
|
|
echo aa-master >>file &&
|
|
git commit -a -m aa-master
|
|
'
|
|
|
|
test_expect_success 'mixed-success push returns error' '! git push'
|
|
|
|
test_expect_success 'check tracking branches updated correctly after push' '
|
|
test "$(git rev-parse origin/master)" = "$(git rev-parse master)"
|
|
'
|
|
|
|
test_expect_success 'check tracking branches not updated for failed refs' '
|
|
test "$(git rev-parse origin/b1)" = "$b1" &&
|
|
test "$(git rev-parse origin/b2)" = "$b2"
|
|
'
|
|
|
|
test_expect_success 'deleted branches have their tracking branches removed' '
|
|
git push origin :b1 &&
|
|
test "$(git rev-parse origin/b1)" = "origin/b1"
|
|
'
|
|
|
|
test_expect_success 'already deleted tracking branches ignored' '
|
|
git branch -d -r origin/b3 &&
|
|
git push origin :b3 >output 2>&1 &&
|
|
! grep error output
|
|
'
|
|
|
|
test_done
|