summaryrefslogtreecommitdiff
path: root/t
diff options
context:
space:
mode:
authorLibravatar Junio C Hamano <gitster@pobox.com>2016-11-10 15:55:13 -0800
committerLibravatar Junio C Hamano <gitster@pobox.com>2016-11-10 15:55:13 -0800
commitb36b716cf611dea0054afef55241c88b99399571 (patch)
treefd2371d2b1b8d7de6905dd197d14f8c6d6f0dad2 /t
parentRevert "t6026-merge-attr: ensure that the merge driver was called" (diff)
downloadtgif-b36b716cf611dea0054afef55241c88b99399571.tar.xz
Revert "t6026-merge-attr: don't fail if sleep exits early"
This reverts commit 734fde2d7167e4b20d2ff6062ade3846949b0741. The point of the test is that the stray process was still running when 'git merge' did its thing through its completion, so a failure to "kill" it means we didn't give a condition to the test to trigger a possible future breakage. Appending "|| :" to the "kill" is sweeping a test-bug under the rug.
Diffstat (limited to 't')
-rwxr-xr-xt/t6026-merge-attr.sh2
1 files changed, 1 insertions, 1 deletions
diff --git a/t/t6026-merge-attr.sh b/t/t6026-merge-attr.sh
index 2672b15aa3..7a6e33e673 100755
--- a/t/t6026-merge-attr.sh
+++ b/t/t6026-merge-attr.sh
@@ -187,7 +187,7 @@ test_expect_success 'custom merge does not lock index' '
sleep 1 &
echo $! >sleep.pid
EOF
- test_when_finished "kill \$(cat sleep.pid) || :" &&
+ test_when_finished "kill \$(cat sleep.pid)" &&
test_write_lines >.gitattributes \
"* merge=ours" "text merge=sleep-one-second" &&