]> Cypherpunks repositories - gostls13.git/commitdiff
net/rpc: call client.Close() when test exit
authorChaiShushan <chaishushan@gmail.com>
Mon, 24 Jun 2013 20:18:50 +0000 (13:18 -0700)
committerRob Pike <r@golang.org>
Mon, 24 Jun 2013 20:18:50 +0000 (13:18 -0700)
Fixes #5768.

R=golang-dev
CC=golang-dev
https://golang.org/cl/10503043

src/pkg/net/rpc/server_test.go

index eb17210abc91157e209cfbee443de183e7148cdc..82773240069e14520264fdd6ee3ac4356d20f49d 100644 (file)
@@ -125,6 +125,7 @@ func testRPC(t *testing.T, addr string) {
        if err != nil {
                t.Fatal("dialing", err)
        }
+       defer client.Close()
 
        // Synchronous calls
        args := &Args{7, 8}
@@ -253,6 +254,7 @@ func testHTTPRPC(t *testing.T, path string) {
        if err != nil {
                t.Fatal("dialing", err)
        }
+       defer client.Close()
 
        // Synchronous calls
        args := &Args{7, 8}
@@ -329,6 +331,7 @@ func TestServeRequest(t *testing.T) {
 
 func testServeRequest(t *testing.T, server *Server) {
        client := CodecEmulator{server: server}
+       defer client.Close()
 
        args := &Args{7, 8}
        reply := new(Reply)
@@ -411,6 +414,7 @@ func (WriteFailCodec) Close() error {
 
 func TestSendDeadlock(t *testing.T) {
        client := NewClientWithCodec(WriteFailCodec(0))
+       defer client.Close()
 
        done := make(chan bool)
        go func() {
@@ -449,6 +453,8 @@ func countMallocs(dial func() (*Client, error), t *testing.T) float64 {
        if err != nil {
                t.Fatal("error dialing", err)
        }
+       defer client.Close()
+
        args := &Args{7, 8}
        reply := new(Reply)
        return testing.AllocsPerRun(100, func() {
@@ -496,6 +502,8 @@ func (writeCrasher) Write(p []byte) (int, error) {
 func TestClientWriteError(t *testing.T) {
        w := &writeCrasher{done: make(chan bool)}
        c := NewClient(w)
+       defer c.Close()
+
        res := false
        err := c.Call("foo", 1, &res)
        if err == nil {
@@ -552,6 +560,7 @@ func benchmarkEndToEnd(dial func() (*Client, error), b *testing.B) {
        if err != nil {
                b.Fatal("error dialing:", err)
        }
+       defer client.Close()
 
        // Synchronous calls
        args := &Args{7, 8}
@@ -587,6 +596,7 @@ func benchmarkEndToEndAsync(dial func() (*Client, error), b *testing.B) {
        if err != nil {
                b.Fatal("error dialing:", err)
        }
+       defer client.Close()
 
        // Asynchronous calls
        args := &Args{7, 8}