From: David du Colombier <0intro@gmail.com>
Date: Sat, 14 Jan 2017 13:13:58 +0000 (+0100)
Subject: net/http: skip TestServerHijackGetsBackgroundByte on Plan 9
X-Git-Tag: go1.8rc2~1^2~13
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=76f981c8d86ec122da50a140f8843eb636ac8240;p=gostls13.git

net/http: skip TestServerHijackGetsBackgroundByte on Plan 9

CL 5232 added TestServerHijackGetsBackgroundByte, which is failing
on Plan 9, because CloseWrite is not implemented on Plan 9 yet.

Updates #17906.
Updates #18657.

Change-Id: I3c2f73760b0f767f3f9ed2698c855372170e0481
Reviewed-on: https://go-review.googlesource.com/35178
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
---

diff --git a/src/net/http/serve_test.go b/src/net/http/serve_test.go
index bc76407b38..681dff193a 100644
--- a/src/net/http/serve_test.go
+++ b/src/net/http/serve_test.go
@@ -5180,6 +5180,9 @@ func TestServerDuplicateBackgroundRead(t *testing.T) {
 // bufio.Reader.Buffered(), without resorting to Reading it
 // (potentially blocking) to get at it.
 func TestServerHijackGetsBackgroundByte(t *testing.T) {
+	if runtime.GOOS == "plan9" {
+		t.Skip("skipping test; see https://golang.org/issue/18657")
+	}
 	setParallel(t)
 	defer afterTest(t)
 	done := make(chan struct{})