From eb84d34b8df923851fb9c9630a21bf19c3e8b533 Mon Sep 17 00:00:00 2001 From: Olaoluwa Osuntokun Date: Wed, 1 Nov 2017 15:12:02 -0700 Subject: [PATCH] brontide: don't run tests that use establishTestConnection in parallel --- brontide/noise_test.go | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/brontide/noise_test.go b/brontide/noise_test.go index de2fa078..094d1bac 100644 --- a/brontide/noise_test.go +++ b/brontide/noise_test.go @@ -85,8 +85,6 @@ func establishTestConnection() (net.Conn, net.Conn, func(), error) { } func TestConnectionCorrectness(t *testing.T) { - t.Parallel() - // Create a test connection, grabbing either side of the connection // into local variables. If the initial crypto handshake fails, then // we'll get a non-nil error here. @@ -142,7 +140,7 @@ func TestMaxPayloadLength(t *testing.T) { b := Machine{} b.split() - // Create a payload that's juust over the maximum alloted payload + // Create a payload that's juust over the maximum allotted payload // length. payloadToReject := make([]byte, math.MaxUint16+1) @@ -177,8 +175,6 @@ func TestMaxPayloadLength(t *testing.T) { } func TestWriteMessageChunking(t *testing.T) { - t.Parallel() - // Create a test connection, grabbing either side of the connection // into local variables. If the initial crypto handshake fails, then // we'll get a non-nil error here.