From 67d6a510653c8dcb2e2a4eca5a0e7caf4fe38677 Mon Sep 17 00:00:00 2001 From: "Bryan C. Mills" Date: Mon, 14 Feb 2022 16:07:05 -0500 Subject: [PATCH] net: in TestNotTemporaryRead, reject io.EOF on platforms other than plan9 Updates #29685 Change-Id: Id8dca078213942666871ac8ded663326e98427fe Reviewed-on: https://go-review.googlesource.com/c/go/+/385754 Reviewed-by: Ian Lance Taylor Reviewed-by: Emmanuel Odeke Trust: Bryan Mills Run-TryBot: Bryan Mills TryBot-Result: Gopher Robot --- src/net/net_test.go | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/net/net_test.go b/src/net/net_test.go index 76a9c8b151..fa5ad632bb 100644 --- a/src/net/net_test.go +++ b/src/net/net_test.go @@ -549,9 +549,7 @@ func TestNotTemporaryRead(t *testing.T) { if runtime.GOOS == "plan9" { return } - // TODO: during an open development cycle, try making this a failure - // and see whether it causes the test to become flaky anywhere else. - return + t.Fatal("Read unexpectedly returned io.EOF after socket was abruptly closed") } if ne, ok := err.(Error); !ok { t.Errorf("Read error does not implement net.Error: %v", err) -- 2.50.0