summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAlex Brainman <alex.brainman@gmail.com>2013-02-15 15:52:12 +1100
committerAlex Brainman <alex.brainman@gmail.com>2013-02-15 15:52:12 +1100
commitd7d3175331fdfa9fbc13282fef0cee4676993d59 (patch)
tree87858a1806f35d0980fe892854a845625cc0426a
parentf42fb9962b7e786587c1772a73a2bdcfc8a797e7 (diff)
downloadgo-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.go73
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)
- }
-}