Fix flaky agentprotocol test (#538)

Both testing threads called the Kill() function at the same time which
caused them to deadlock as they both were trying to send the termination
packet to each-other at the same time. In a real situation this would
not cause a deadlock as the communication between the two sides would be
buffered.

Signed-off-by: Nikos Tsipinakis <nikos@tsipinakis.com>

Author: tsipinakis

agentprotocol/server_test.go

@@ -95,6 +95,6 @@ func TestConnectionSetup(t *testing.T) {
 		t.Fatalf("Expected to read 'Message to server' instead got %s", string(buf[:nBytes]))
 	}
 	_ = testConClient.Close()
-	close(closeChan)
 	clientCtx.Kill()
+	close(closeChan)
 }