Skip to content

Commit 67d6a51

Browse files
author
Bryan C. Mills
committed
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 <[email protected]> Reviewed-by: Emmanuel Odeke <[email protected]> Trust: Bryan Mills <[email protected]> Run-TryBot: Bryan Mills <[email protected]> TryBot-Result: Gopher Robot <[email protected]>
1 parent 690ac40 commit 67d6a51

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

src/net/net_test.go

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -549,9 +549,7 @@ func TestNotTemporaryRead(t *testing.T) {
549549
if runtime.GOOS == "plan9" {
550550
return
551551
}
552-
// TODO: during an open development cycle, try making this a failure
553-
// and see whether it causes the test to become flaky anywhere else.
554-
return
552+
t.Fatal("Read unexpectedly returned io.EOF after socket was abruptly closed")
555553
}
556554
if ne, ok := err.(Error); !ok {
557555
t.Errorf("Read error does not implement net.Error: %v", err)

0 commit comments

Comments
 (0)