mirror of
https://github.com/git/git
synced 2024-08-24 18:26:02 +00:00
Revert "t0081 (line-buffer): add buffering tests"
This (morally) reverts commit d280f68313
,
which added some tests that are a pain to maintain and are not likely
to find bugs in git.
Helped-by: Johannes Sixt <j6t@kdbg.org>
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
Acked-by: Jeff King <peff@peff.net>
This commit is contained in:
parent
a892a2ddfe
commit
6908e99946
|
@ -2,74 +2,14 @@
|
|||
|
||||
test_description="Test the svn importer's input handling routines.
|
||||
|
||||
These tests exercise the line_buffer library, but their real purpose
|
||||
is to check the assumptions that library makes of the platform's input
|
||||
routines. Processes engaged in bi-directional communication would
|
||||
hang if fread or fgets is too greedy.
|
||||
These tests provide some simple checks that the line_buffer API
|
||||
behaves as advertised.
|
||||
|
||||
While at it, check that input of newlines and null bytes are handled
|
||||
correctly.
|
||||
"
|
||||
. ./test-lib.sh
|
||||
|
||||
test -n "$GIT_REMOTE_SVN_TEST_BIG_FILES" && test_set_prereq EXPENSIVE
|
||||
|
||||
generate_tens_of_lines () {
|
||||
tens=$1 &&
|
||||
line=$2 &&
|
||||
|
||||
i=0 &&
|
||||
while test $i -lt "$tens"
|
||||
do
|
||||
for j in a b c d e f g h i j
|
||||
do
|
||||
echo "$line"
|
||||
done &&
|
||||
: $((i = $i + 1)) ||
|
||||
return
|
||||
done
|
||||
}
|
||||
|
||||
long_read_test () {
|
||||
: each line is 10 bytes, including newline &&
|
||||
line=abcdefghi &&
|
||||
echo "$line" >expect &&
|
||||
|
||||
if ! test_declared_prereq PIPE
|
||||
then
|
||||
echo >&4 "long_read_test: need to declare PIPE prerequisite"
|
||||
return 127
|
||||
fi &&
|
||||
tens_of_lines=$(($1 / 100 + 1)) &&
|
||||
lines=$(($tens_of_lines * 10)) &&
|
||||
readsize=$((($lines - 1) * 10 + 3)) &&
|
||||
copysize=7 &&
|
||||
rm -f input &&
|
||||
mkfifo input &&
|
||||
{
|
||||
(
|
||||
generate_tens_of_lines $tens_of_lines "$line" &&
|
||||
exec sleep 100
|
||||
) >input &
|
||||
} &&
|
||||
test-line-buffer input <<-EOF >output &&
|
||||
binary $readsize
|
||||
copy $copysize
|
||||
EOF
|
||||
kill $! &&
|
||||
test_line_count = $lines output &&
|
||||
tail -n 1 <output >actual &&
|
||||
test_cmp expect actual
|
||||
}
|
||||
|
||||
test_expect_success 'setup: have pipes?' '
|
||||
rm -f frob &&
|
||||
if mkfifo frob
|
||||
then
|
||||
test_set_prereq PIPE
|
||||
fi
|
||||
'
|
||||
|
||||
test_expect_success 'hello world' '
|
||||
echo ">HELLO" >expect &&
|
||||
test-line-buffer <<-\EOF >actual &&
|
||||
|
@ -79,21 +19,6 @@ test_expect_success 'hello world' '
|
|||
test_cmp expect actual
|
||||
'
|
||||
|
||||
test_expect_success PIPE '0-length read, no input available' '
|
||||
printf ">" >expect &&
|
||||
rm -f input &&
|
||||
mkfifo input &&
|
||||
{
|
||||
sleep 100 >input &
|
||||
} &&
|
||||
test-line-buffer input <<-\EOF >actual &&
|
||||
binary 0
|
||||
copy 0
|
||||
EOF
|
||||
kill $! &&
|
||||
test_cmp expect actual
|
||||
'
|
||||
|
||||
test_expect_success '0-length read, send along greeting' '
|
||||
echo ">HELLO" >expect &&
|
||||
test-line-buffer <<-\EOF >actual &&
|
||||
|
@ -104,33 +29,6 @@ test_expect_success '0-length read, send along greeting' '
|
|||
test_cmp expect actual
|
||||
'
|
||||
|
||||
test_expect_success PIPE '1-byte read, no input available' '
|
||||
printf ">%s" ab >expect &&
|
||||
rm -f input &&
|
||||
mkfifo input &&
|
||||
{
|
||||
(
|
||||
printf "%s" a &&
|
||||
printf "%s" b &&
|
||||
exec sleep 100
|
||||
) >input &
|
||||
} &&
|
||||
test-line-buffer input <<-\EOF >actual &&
|
||||
binary 1
|
||||
copy 1
|
||||
EOF
|
||||
kill $! &&
|
||||
test_cmp expect actual
|
||||
'
|
||||
|
||||
test_expect_success PIPE 'long read (around 8192 bytes)' '
|
||||
long_read_test 8192
|
||||
'
|
||||
|
||||
test_expect_success PIPE,EXPENSIVE 'longer read (around 65536 bytes)' '
|
||||
long_read_test 65536
|
||||
'
|
||||
|
||||
test_expect_success 'read from file descriptor' '
|
||||
rm -f input &&
|
||||
echo hello >expect &&
|
||||
|
|
Loading…
Reference in a new issue