status-im
/
consul
mirror of https://github.com/status-im/consul.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

connect/proxy: remove t.Parallel from tests 

These tests run in under 10ms, t.Parallel provides no value and makes debuging failures
more difficult.
This commit is contained in:
Daniel Nephin 2021-04-28 13:46:53 -04:00
parent 0c9555ec0e
commit 146e1d3878
2 changed files with 2 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
connect/proxy/config_test.go
View File

@ -14,8 +14,6 @@ import (
) )
func TestUpstreamResolverFuncFromClient(t *testing.T) { func TestUpstreamResolverFuncFromClient(t *testing.T) {
t.Parallel()
tests := []struct { tests := []struct {
name string name string
cfg UpstreamConfig cfg UpstreamConfig

9
connect/proxy/conn_test.go
View File

@ -7,9 +7,10 @@ import (
"testing" "testing"
"time" "time"
"github.com/hashicorp/consul/sdk/testutil/retry"
"github.com/stretchr/testify/assert" "github.com/stretchr/testify/assert"
"github.com/stretchr/testify/require" "github.com/stretchr/testify/require"
"github.com/hashicorp/consul/sdk/testutil/retry"
) )
// Assert io.Closer implementation // Assert io.Closer implementation
@ -65,8 +66,6 @@ func testConnPipelineSetup(t *testing.T) (net.Conn, net.Conn, *Conn, func()) {
} }
func TestConn(t *testing.T) { func TestConn(t *testing.T) {
t.Parallel()
src, dst, c, stop := testConnPipelineSetup(t) src, dst, c, stop := testConnPipelineSetup(t)
defer stop() defer stop()
@ -124,8 +123,6 @@ func TestConn(t *testing.T) {
} }
func TestConnSrcClosing(t *testing.T) { func TestConnSrcClosing(t *testing.T) {
t.Parallel()
src, dst, c, stop := testConnPipelineSetup(t) src, dst, c, stop := testConnPipelineSetup(t)
defer stop() defer stop()
@ -164,8 +161,6 @@ func TestConnSrcClosing(t *testing.T) {
} }
func TestConnDstClosing(t *testing.T) { func TestConnDstClosing(t *testing.T) {
t.Parallel()
src, dst, c, stop := testConnPipelineSetup(t) src, dst, c, stop := testConnPipelineSetup(t)
defer stop() defer stop()