Skip to content

Commit 9462ac7

Browse files
committed
Merge branch 'tg/t5570-drop-racy-test'
An inherently racy test that caused intermittent failures has been removed. * tg/t5570-drop-racy-test: Revert "t/lib-git-daemon: record daemon log" t5570: drop racy test
2 parents 74ae065 + 3c78e97 commit 9462ac7

File tree

2 files changed

+3
-24
lines changed

2 files changed

+3
-24
lines changed

t/lib-git-daemon.sh

Lines changed: 3 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -54,19 +54,11 @@ start_git_daemon() {
5454
"$@" "$GIT_DAEMON_DOCUMENT_ROOT_PATH" \
5555
>&3 2>git_daemon_output &
5656
GIT_DAEMON_PID=$!
57-
>daemon.log
5857
{
5958
read -r line <&7
60-
printf "%s\n" "$line"
61-
printf >&4 "%s\n" "$line"
62-
(
63-
while read -r line <&7
64-
do
65-
printf "%s\n" "$line"
66-
printf >&4 "%s\n" "$line"
67-
done
68-
) &
69-
} 7<git_daemon_output >>"$TRASH_DIRECTORY/daemon.log" &&
59+
printf "%s\n" "$line" >&4
60+
cat <&7 >&4 &
61+
} 7<git_daemon_output &&
7062

7163
# Check expected output
7264
if test x"$(expr "$line" : "\[[0-9]*\] \(.*\)")" != x"Ready to rumble"

t/t5570-git-daemon.sh

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -183,19 +183,6 @@ test_expect_success 'hostname cannot break out of directory' '
183183
git ls-remote "$GIT_DAEMON_URL/escape.git"
184184
'
185185

186-
test_expect_success 'daemon log records all attributes' '
187-
cat >expect <<-\EOF &&
188-
Extended attribute "host": localhost
189-
Extended attribute "protocol": version=1
190-
EOF
191-
>daemon.log &&
192-
GIT_OVERRIDE_VIRTUAL_HOST=localhost \
193-
git -c protocol.version=1 \
194-
ls-remote "$GIT_DAEMON_URL/interp.git" &&
195-
grep -i extended.attribute daemon.log | cut -d" " -f2- >actual &&
196-
test_cmp expect actual
197-
'
198-
199186
test_expect_success FAKENC 'hostname interpolation works after LF-stripping' '
200187
{
201188
printf "git-upload-pack /interp.git\n\0host=localhost" | packetize

0 commit comments

Comments
 (0)