From f9bbfe4a093144e48a4774d1388c1e99be64673b Mon Sep 17 00:00:00 2001 From: David du Colombier <0intro@gmail.com> Date: Thu, 20 Oct 2016 16:51:03 +0200 Subject: [PATCH] net/http: remove workaround in TestTransportClosesBodyOnError on Plan 9 This issue has been fixed in CL 31390. Fixes #9554. Change-Id: Ib8ff4cb1ffcb7cdbf117510b98b4a7e13e4efd2b Reviewed-on: https://go-review.googlesource.com/31520 TryBot-Result: Gobot Gobot Run-TryBot: David du Colombier <0intro@gmail.com> Reviewed-by: Brad Fitzpatrick --- src/net/http/transport_test.go | 18 +----------------- 1 file changed, 1 insertion(+), 17 deletions(-) diff --git a/src/net/http/transport_test.go b/src/net/http/transport_test.go index 147b468e78..083fedefe0 100644 --- a/src/net/http/transport_test.go +++ b/src/net/http/transport_test.go @@ -2485,22 +2485,6 @@ type errorReader struct { func (e errorReader) Read(p []byte) (int, error) { return 0, e.err } -type plan9SleepReader struct{} - -func (plan9SleepReader) Read(p []byte) (int, error) { - if runtime.GOOS == "plan9" { - // After the fix to unblock TCP Reads in - // https://golang.org/cl/15941, this sleep is required - // on plan9 to make sure TCP Writes before an - // immediate TCP close go out on the wire. On Plan 9, - // it seems that a hangup of a TCP connection with - // queued data doesn't send the queued data first. - // https://golang.org/issue/9554 - time.Sleep(50 * time.Millisecond) - } - return 0, io.EOF -} - type closerFunc func() error func (f closerFunc) Close() error { return f() } @@ -2595,7 +2579,7 @@ func TestTransportClosesBodyOnError(t *testing.T) { io.Reader io.Closer }{ - io.MultiReader(io.LimitReader(neverEnding('x'), 1<<20), plan9SleepReader{}, errorReader{fakeErr}), + io.MultiReader(io.LimitReader(neverEnding('x'), 1<<20), errorReader{fakeErr}), closerFunc(func() error { select { case didClose <- true: -- 2.48.1