Skip to content

Commit

Permalink
t5551: fix the 50,000 tag test
Browse files Browse the repository at this point in the history
The first version of test 23 did simply check that no output was
sent to the standard error stream.  With 5e2c7cd (t5551: do not use
unportable sed '\+', 2013-05-12), we started to also verify that the
expected tags were actually cloned.

Since 68b939b (clone: send diagnostic messages to stderr,
2013-09-18), "git clone" shows "Cloning into 'too-many-refs'" to the
standard error stream (it used to do so to the standard output),
causing the test to fail.

Signed-off-by: Torsten Bögershausen <tboegi@web.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
Torsten Bögershausen authored and Junio C Hamano committed Jun 9, 2014
1 parent 68b939b commit 9e10e0b
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions t/t5551-http-fetch.sh
Original file line number Diff line number Diff line change
Expand Up @@ -214,8 +214,7 @@ test_expect_success EXPENSIVE 'create 50,000 tags in the repo' '
'

test_expect_success EXPENSIVE 'clone the 50,000 tag repo to check OS command line overflow' '
git clone $HTTPD_URL/smart/repo.git too-many-refs 2>err &&
test_line_count = 0 err &&
git clone $HTTPD_URL/smart/repo.git too-many-refs &&
(
cd too-many-refs &&
test $(git for-each-ref refs/tags | wc -l) = 50000
Expand Down

0 comments on commit 9e10e0b

Please sign in to comment.