diff options
author | Alex Brainman <alex.brainman@gmail.com> | 2013-02-15 15:52:12 +1100 |
---|---|---|
committer | Alex Brainman <alex.brainman@gmail.com> | 2013-02-15 15:52:12 +1100 |
commit | d7d3175331fdfa9fbc13282fef0cee4676993d59 (patch) | |
tree | 87858a1806f35d0980fe892854a845625cc0426a | |
parent | f42fb9962b7e786587c1772a73a2bdcfc8a797e7 (diff) | |
download | go-d7d3175331fdfa9fbc13282fef0cee4676993d59.tar.gz |
net: delete TestDialTimeoutHandleLeak
It is too flaky. Tried to make it more reliable,
but that affects other tests (they run too long),
because we do unusual things here, like attempting
to connect to non-existing address and interrupt.
R=golang-dev, bradfitz, mikioh.mikioh
CC=golang-dev
https://codereview.appspot.com/7314097
-rw-r--r-- | src/pkg/net/dial_windows_test.go | 73 |
1 files changed, 0 insertions, 73 deletions
diff --git a/src/pkg/net/dial_windows_test.go b/src/pkg/net/dial_windows_test.go deleted file mode 100644 index aabd4d136..000000000 --- a/src/pkg/net/dial_windows_test.go +++ /dev/null @@ -1,73 +0,0 @@ -// Copyright 2013 The Go Authors. All rights reserved. -// Use of this source code is governed by a BSD-style -// license that can be found in the LICENSE file. - -package net - -import ( - "sync" - "syscall" - "testing" - "time" - "unsafe" -) - -var handleCounter struct { - once sync.Once - proc *syscall.Proc -} - -func numHandles(t *testing.T) int { - - handleCounter.once.Do(func() { - d, err := syscall.LoadDLL("kernel32.dll") - if err != nil { - t.Fatalf("LoadDLL: %v\n", err) - } - handleCounter.proc, err = d.FindProc("GetProcessHandleCount") - if err != nil { - t.Fatalf("FindProc: %v\n", err) - } - }) - - cp, err := syscall.GetCurrentProcess() - if err != nil { - t.Fatalf("GetCurrentProcess: %v\n", err) - } - var n uint32 - r, _, err := handleCounter.proc.Call(uintptr(cp), uintptr(unsafe.Pointer(&n))) - if r == 0 { - t.Fatalf("GetProcessHandleCount: %v\n", error(err)) - } - return int(n) -} - -func testDialTimeoutHandleLeak(t *testing.T) (before, after int) { - before = numHandles(t) - // See comment in TestDialTimeout about why we use this address. - c, err := DialTimeout("tcp", "127.0.71.111:49151", 200*time.Millisecond) - after = numHandles(t) - if err == nil { - c.Close() - t.Fatalf("unexpected: connected to %s", c.RemoteAddr()) - } - terr, ok := err.(timeout) - if !ok { - t.Fatalf("got error %q; want error with timeout interface", err) - } - if !terr.Timeout() { - t.Fatalf("got error %q; not a timeout", err) - } - return -} - -func TestDialTimeoutHandleLeak(t *testing.T) { - if !canUseConnectEx("tcp") { - t.Skip("skipping test; no ConnectEx found.") - } - testDialTimeoutHandleLeak(t) // ignore first call results - before, after := testDialTimeoutHandleLeak(t) - if before != after { - t.Fatalf("handle count is different before=%d and after=%d", before, after) - } -} |