Skip to content

Commit

Permalink
os: enable TestClosedPipeRace* on FreeBSD
Browse files Browse the repository at this point in the history
This test has worked since CL 165801 (committed March 12, 2019), so
stop skipping it. With this, we check that Close makes concurrent I/O
operations on pipes return Errclosed on all platforms.

Updates #19093.

Change-Id: Ic090c70996c115abf80d8f9b93ca2aeaf347c9d8
Reviewed-on: https://go-review.googlesource.com/c/go/+/371016
Trust: Austin Clements <austin@google.com>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Run-TryBot: Austin Clements <austin@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
  • Loading branch information
aclements committed Dec 14, 2021
1 parent d407a8c commit 0f05ed3
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions src/os/pipe_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -150,11 +150,6 @@ func TestStdPipeHelper(t *testing.T) {
}

func testClosedPipeRace(t *testing.T, read bool) {
switch runtime.GOOS {
case "freebsd":
t.Skip("FreeBSD does not use the poller; issue 19093")
}

limit := 1
if !read {
// Get the amount we have to write to overload a pipe
Expand Down

0 comments on commit 0f05ed3

Please sign in to comment.