From f5d27562151b136bc4e4840f1abb70f00e62b7b0 Mon Sep 17 00:00:00 2001 From: Tim Pepper Date: Wed, 17 Aug 2016 11:49:59 -0700 Subject: [PATCH] testutil: connect/disconnect make 1 result on event channel There's an event result write to the events channel in both the EventNotify() and in the ConnectNotify()/DisconnectNotify() code paths, but there will only be one read. These writers will block and leak goroutines. Signed-off-by: Tim Pepper --- testutil/server.go | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/testutil/server.go b/testutil/server.go index c6507e1af..a89ef3fc3 100644 --- a/testutil/server.go +++ b/testutil/server.go @@ -404,8 +404,12 @@ func (server *SsntpTestServer) EventNotify(uuid string, event ssntp.Event, frame payload := frame.Payload switch event { - // case ssntp.NodeConnected: handled by ConnectNotify() - // case ssntp.NodeDisconnected: handled by DisconnectNotify() + case ssntp.NodeConnected: + //handled by ConnectNotify() + return + case ssntp.NodeDisconnected: + //handled by DisconnectNotify() + return case ssntp.TraceReport: var traceEvent payloads.Trace