This is an archive of the discontinued Mercurial Phabricator instance.

Differential D12060

test-http-bad-server: move on test from bytes-count to pattern
ClosedPublic

Authored by marmoute on Jan 24 2022, 9:47 AM.

Details

Reviewers
Alphare
Group Reviewers
hg-reviewers
Commits
rHG177ef11a4a58: test-http-bad-server: move on test from bytes-count to pattern
Summary

This specific test case is now using the new "close when a pattern is seen"
approach compared to the more fragile "that many bytes were sent" approach.

Since such change are still a bit noisy, we split each change in there own
changesets.

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

marmoute created this revision.Jan 24 2022, 9:47 AM
Alphare accepted this revision.Jan 24 2022, 10:33 AM
This revision is now accepted and ready to land.Jan 24 2022, 10:33 AM
baymax updated this revision to Diff 31884.Jan 24 2022, 1:18 PM

✅ refresh by Heptapod after a successful CI run (🐙 💚)

Revision Contents
Changeset List

PathPackages
Mtests/test-http-bad-server.t (26 lines)

Diff 31935

tests/test-http-bad-server.t

#endif #endif
$ rm -f error.log $ rm -f error.log
Servers stops after bundle2 stream params header Servers stops after bundle2 stream params header
------------------------------------------------ ------------------------------------------------
$ hg serve --config badserver.close-after-send-bytes=992 -p $HGPORT -d --pid-file=hg.pid -E error.log $ hg serve \
> --config badserver.close-after-send-patterns='4\r\n\0\0\0\0\r\n' \
> -p $HGPORT -d --pid-file=hg.pid -E error.log
$ cat hg.pid > $DAEMON_PIDS $ cat hg.pid > $DAEMON_PIDS
$ hg clone http://localhost:$HGPORT/ clone $ hg clone http://localhost:$HGPORT/ clone
requesting all changes requesting all changes
abort: HTTP request error (incomplete response) abort: HTTP request error (incomplete response)
(this may be an intermittent network failure; if the error persists, consider contacting the network or server operator) (this may be an intermittent network failure; if the error persists, consider contacting the network or server operator)
[255] [255]
$ killdaemons.py $DAEMON_PIDS $ killdaemons.py $DAEMON_PIDS
#if py36 #if py36
$ "$PYTHON" $TESTDIR/filtertraceback.py < error.log | tail -10 $ "$PYTHON" $TESTDIR/filtertraceback.py < error.log | tail -10
sendall(167 from 167) -> (33) HTTP/1.1 200 Script output follows\r\nServer: badhttpserver\r\nDate: $HTTP_DATE$\r\nContent-Type: application/mercurial-0.2\r\nTransfer-Encoding: chunked\r\n\r\n sendall(167) -> HTTP/1.1 200 Script output follows\r\nServer: badhttpserver\r\nDate: $HTTP_DATE$\r\nContent-Type: application/mercurial-0.2\r\nTransfer-Encoding: chunked\r\n\r\n
sendall(6 from 6) -> (27) 1\\r\\n\x04\\r\\n (esc) sendall(6) -> 1\\r\\n\x04\\r\\n (esc)
sendall(9 from 9) -> (18) 4\r\nnone\r\n sendall(9) -> 4\r\nnone\r\n
sendall(9 from 9) -> (9) 4\r\nHG20\r\n sendall(9) -> 4\r\nHG20\r\n
sendall(9 from 9) -> (0) 4\\r\\n\x00\x00\x00\x00\\r\\n (esc) sendall(9 from 9) -> (0) 4\\r\\n\x00\x00\x00\x00\\r\\n (esc)
write limit reached; closing socket write limit reached; closing socket
$LOCALIP - - [$ERRDATE$] Exception happened during processing request '/?cmd=getbundle': (glob) $LOCALIP - - [$ERRDATE$] Exception happened during processing request '/?cmd=getbundle': (glob)
Traceback (most recent call last): Traceback (most recent call last):
Exception: connection closed after sending N bytes Exception: connection closed after sending N bytes
#else #else
$ "$PYTHON" $TESTDIR/filtertraceback.py < error.log | tail -12 $ "$PYTHON" $TESTDIR/filtertraceback.py < error.log | tail -12
readline(~) -> (2) \r\n (py3 !) readline(~) -> (2) \r\n (py3 !)
write(167 from 167) -> (33) HTTP/1.1 200 Script output follows\r\nServer: badhttpserver\r\nDate: $HTTP_DATE$\r\nContent-Type: application/mercurial-0.2\r\nTransfer-Encoding: chunked\r\n\r\n (py3 !) write(167) -> HTTP/1.1 200 Script output follows\r\nServer: badhttpserver\r\nDate: $HTTP_DATE$\r\nContent-Type: application/mercurial-0.2\r\nTransfer-Encoding: chunked\r\n\r\n (py3 !)
write(41 from 41) -> (63) Content-Type: application/mercurial-0.2\r\n write(41) -> Content-Type: application/mercurial-0.2\r\n
write(28 from 28) -> (35) Transfer-Encoding: chunked\r\n write(28) -> Transfer-Encoding: chunked\r\n (no-py3 !)
write(2 from 2) -> (33) \r\n write(2) -> \r\n (no-py3 !)
write(6 from 6) -> (27) 1\\r\\n\x04\\r\\n (esc) write(6) -> 1\\r\\n\x04\\r\\n (esc)
write(9 from 9) -> (18) 4\r\nnone\r\n write(9) -> 4\r\nnone\r\n
write(9 from 9) -> (9) 4\r\nHG20\r\n write(9) -> 4\r\nHG20\r\n
write(9 from 9) -> (0) 4\\r\\n\x00\x00\x00\x00\\r\\n (esc) write(9 from 9) -> (0) 4\\r\\n\x00\x00\x00\x00\\r\\n (esc)
write limit reached; closing socket write limit reached; closing socket
$LOCALIP - - [$ERRDATE$] Exception happened during processing request '/?cmd=getbundle': (glob) $LOCALIP - - [$ERRDATE$] Exception happened during processing request '/?cmd=getbundle': (glob)
Traceback (most recent call last): Traceback (most recent call last):
Exception: connection closed after sending N bytes Exception: connection closed after sending N bytes
#endif #endif