Skip to content

Commit

Permalink
nettest: skip testConcurrentMethods on Plan 9
Browse files Browse the repository at this point in the history
This test is running out of memory on Plan 9.

Updates golang/go#20489.

Change-Id: I79cdea9c31f396b6ef5c47474674ad736362b232
Reviewed-on: https://go-review.googlesource.com/44130
Reviewed-by: Brad Fitzpatrick <[email protected]>
  • Loading branch information
0intro committed May 25, 2017
1 parent 5b58a9c commit 7dcfb80
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions nettest/conntest.go
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ import (
"io/ioutil"
"math/rand"
"net"
"runtime"
"sync"
"testing"
"time"
Expand Down Expand Up @@ -341,6 +342,9 @@ func testCloseTimeout(t *testing.T, c1, c2 net.Conn) {
// testConcurrentMethods tests that the methods of net.Conn can safely
// be called concurrently.
func testConcurrentMethods(t *testing.T, c1, c2 net.Conn) {
if runtime.GOOS == "plan9" {
t.Skip("skipping on plan9; see https://golang.org/issue/20489")
}
go chunkedCopy(c2, c2)

// The results of the calls may be nonsensical, but this should
Expand Down

0 comments on commit 7dcfb80

Please sign in to comment.