Tests: Fix t.Parallel() errors in netdeploy package

.golangci.yml

@@ -149,9 +149,6 @@ issues:
     - path: ^logging.*_test\.go
       linters:
         - paralleltest
-    - path: ^netdeploy.*_test\.go
-      linters:
-        - paralleltest
     - path: ^network.*_test\.go
       linters:
         - paralleltest

netdeploy/networkTemplates_test.go

@@ -31,6 +31,7 @@ import (
 
 func TestLoadConfig(t *testing.T) {
 	partitiontest.PartitionTest(t)
+	t.Parallel()
 
 	a := require.New(t)
 
@@ -44,6 +45,7 @@ func TestLoadConfig(t *testing.T) {
 
 func TestLoadMissingConfig(t *testing.T) {
 	partitiontest.PartitionTest(t)
+	t.Parallel()
 
 	a := require.New(t)
 
@@ -55,6 +57,7 @@ func TestLoadMissingConfig(t *testing.T) {
 
 func TestGenerateGenesis(t *testing.T) {
 	partitiontest.PartitionTest(t)
+	t.Parallel()
 
 	a := require.New(t)
 
@@ -74,6 +77,7 @@ func TestGenerateGenesis(t *testing.T) {
 
 func TestValidate(t *testing.T) {
 	partitiontest.PartitionTest(t)
+	t.Parallel()
 
 	a := require.New(t)
 
@@ -106,6 +110,8 @@ type overlayTestStruct struct {
 // TestJsonOverlay ensures that encoding/json will only clobber fields present in the json and leave other fields unchanged
 func TestJsonOverlay(t *testing.T) {
 	partitiontest.PartitionTest(t)
+	t.Parallel()
+
 	before := overlayTestStruct{A: "one", B: "two"}
 	setB := "{\"B\":\"other\"}"
 	dec := json.NewDecoder(strings.NewReader(setB))

netdeploy/network_test.go

@@ -29,6 +29,7 @@ import (
 
 func TestSaveNetworkCfg(t *testing.T) {
 	partitiontest.PartitionTest(t)
+	t.Parallel()
 
 	a := require.New(t)
 
@@ -48,6 +49,7 @@ func TestSaveNetworkCfg(t *testing.T) {
 
 func TestSaveConsensus(t *testing.T) {
 	partitiontest.PartitionTest(t)
+	t.Parallel()
 
 	a := require.New(t)
 

netdeploy/remote/bootstrappedNetwork_test.go

@@ -26,6 +26,7 @@ import (
 
 func TestLoadBootstrappedData(t *testing.T) {
 	partitiontest.PartitionTest(t)
+	t.Parallel()
 
 	badSpecPath := filepath.Join("./../../test", "testdata/deployednettemplates/networks/bootstrapped/badSpec.json")
 	_, err := LoadBootstrappedData(badSpecPath)

netdeploy/remote/deployedNetwork_test.go

@@ -30,6 +30,7 @@ import (
 
 func TestCreateSignedTxBasic(t *testing.T) {
 	partitiontest.PartitionTest(t)
+	t.Parallel()
 
 	var networkState netState
 	networkState.nApplications = 2
@@ -90,6 +91,7 @@ func TestCreateSignedTxBasic(t *testing.T) {
 func TestCreateSignedTxAssets(t *testing.T) {
 	//	assets per account should not exceed limit
 	partitiontest.PartitionTest(t)
+	t.Parallel()
 
 	params := config.Consensus[protocol.ConsensusCurrentVersion]
 	secretDst := keypair()
@@ -127,6 +129,7 @@ func TestCreateSignedTxAssets(t *testing.T) {
 
 func TestAccountsNeeded(t *testing.T) {
 	partitiontest.PartitionTest(t)
+	t.Parallel()
 
 	params := config.Consensus[protocol.ConsensusCurrentVersion]
 	params.MaxAppsCreated = 10