From 3eabdd291d3f53c88fdd01aca30158d0c06420b6 Mon Sep 17 00:00:00 2001 From: Ian Lance Taylor Date: Wed, 6 Nov 2019 13:47:19 -0800 Subject: [PATCH] net: skip TestDialCancel if Dial fails with "connection refused" Fixes #15191 Change-Id: I86214ede619400acd44f21138b5ddf6cef4649a3 Reviewed-on: https://go-review.googlesource.com/c/go/+/205698 Run-TryBot: Ian Lance Taylor Reviewed-by: Brad Fitzpatrick --- src/net/dial_test.go | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/net/dial_test.go b/src/net/dial_test.go index 4569703d12..2eddac8284 100644 --- a/src/net/dial_test.go +++ b/src/net/dial_test.go @@ -14,6 +14,7 @@ import ( "io" "os" "runtime" + "strings" "sync" "testing" "time" @@ -759,10 +760,6 @@ func TestDialerKeepAlive(t *testing.T) { } func TestDialCancel(t *testing.T) { - switch testenv.Builder() { - case "linux-arm64-buildlet": - t.Skip("skipping on linux-arm64-buildlet; incompatible network config? issue 15191") - } mustHaveExternalNetwork(t) blackholeIPPort := JoinHostPort(slowDst4, "1234") @@ -807,6 +804,11 @@ func TestDialCancel(t *testing.T) { t.Error(perr) } if ticks < cancelTick { + // Using strings.Contains is ugly but + // may work on plan9 and windows. + if strings.Contains(err.Error(), "connection refused") { + t.Skipf("connection to %v failed fast with %v", blackholeIPPort, err) + } t.Fatalf("dial error after %d ticks (%d before cancel sent): %v", ticks, cancelTick-ticks, err) } -- 2.50.0