Skip to content

Commit

Permalink
t4126: fix test that happened to work due to timing
Browse files Browse the repository at this point in the history
The test did "reset --hard" (where the HEAD commit has an empty
blob at path "empty") followed by "> empty", expecting that
the index does not notice the file _changed_ since git wrote
it out upon "reset" if the redirection is done quickly enough.

There was no need to do the emptying, and it gave a wrong result
if "reset --hard" happened on time T and then ">empty" happened on
the next second T+1.  This fixes it.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
Junio C Hamano committed Jun 14, 2008
1 parent dc92cc2 commit d88593f
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions t/t4126-apply-empty.sh
Original file line number Diff line number Diff line change
Expand Up @@ -26,15 +26,13 @@ test_expect_success setup '

test_expect_success 'apply empty' '
git reset --hard &&
>empty &&
rm -f missing &&
git apply patch0 &&
test_cmp expect empty
'

test_expect_success 'apply --index empty' '
git reset --hard &&
>empty &&
rm -f missing &&
git apply --index patch0 &&
test_cmp expect empty &&
Expand All @@ -43,15 +41,13 @@ test_expect_success 'apply --index empty' '

test_expect_success 'apply create' '
git reset --hard &&
>empty &&
rm -f missing &&
git apply patch1 &&
test_cmp expect missing
'

test_expect_success 'apply --index create' '
git reset --hard &&
>empty &&
rm -f missing &&
git apply --index patch1 &&
test_cmp expect missing &&
Expand Down

0 comments on commit d88593f

Please sign in to comment.