( )⚙ D12042 test-http-bad-server: unify log printing for `sendall` and `write`

This is an archive of the discontinued Mercurial Phabricator instance.

test-http-bad-server: unify log printing for `sendall` and `write`
ClosedPublic

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

Details

Summary

The write function was logging before calling the function while the sendall
function were logging after calling the function.

We align the write behavior on the sendall behavior before unifying the
code.

Diff Detail

Repository
rHG Mercurial
Branch
default
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

marmoute created this revision.Jan 24 2022, 9:45 AM
Alphare accepted this revision.Jan 24 2022, 10:00 AM
This revision is now accepted and ready to land.Jan 24 2022, 10:00 AM
baymax retitled this revision from test-http-bad-server: unify log printing for `sendall` and `write`o to test-http-bad-server: unify log printing for `sendall` and `write`.Jan 24 2022, 1:16 PM
baymax updated this revision to Diff 31866.

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