add t.Parallel() everywhere

Author: aslilac

.golangci.yml

@@ -18,6 +18,7 @@ linters:
     - makezero
     - misspell
     - nilerr
+    - paralleltest
     - predeclared
     - staticcheck
     - unconvert

integration/integration_test.go

@@ -22,6 +22,7 @@ import (
 // - Runs the `main.tf` specified in the given test directory against the Coder deployment
 // - Asserts the state of the deployment via `codersdk`.
 func TestIntegration(t *testing.T) {
+	t.Parallel()
 	if os.Getenv("TF_ACC") == "1" {
 		t.Skip("Skipping integration tests during tf acceptance tests")
 	}

internal/provider/group_data_source_test.go

@@ -16,6 +16,7 @@ import (
 )
 
 func TestAccGroupDataSource(t *testing.T) {
+	t.Parallel()
 	if os.Getenv("TF_ACC") == "" {
 		t.Skip("Acceptance tests are disabled.")
 	}

internal/provider/group_resource_test.go

@@ -16,6 +16,7 @@ import (
 )
 
 func TestAccGroupResource(t *testing.T) {
+	t.Parallel()
 	if os.Getenv("TF_ACC") == "" {
 		t.Skip("Acceptance tests are disabled.")
 	}
@@ -133,6 +134,7 @@ func TestAccGroupResource(t *testing.T) {
 }
 
 func TestAccGroupResourceAGPL(t *testing.T) {
+	t.Parallel()
 	if os.Getenv("TF_ACC") == "" {
 		t.Skip("Acceptance tests are disabled.")
 	}

internal/provider/license_resource_test.go

@@ -13,6 +13,7 @@ import (
 )
 
 func TestAccLicenseResource(t *testing.T) {
+	t.Parallel()
 	if os.Getenv("TF_ACC") == "" {
 		t.Skip("Acceptance tests are disabled.")
 	}

internal/provider/organization_data_source_test.go

@@ -16,6 +16,7 @@ import (
 )
 
 func TestAccOrganizationDataSource(t *testing.T) {
+	t.Parallel()
 	if os.Getenv("TF_ACC") == "" {
 		t.Skip("Acceptance tests are disabled.")
 	}

internal/provider/organization_resource_test.go

@@ -19,6 +19,7 @@ import (
 )
 
 func TestAccOrganizationResource(t *testing.T) {
+	t.Parallel()
 	if os.Getenv("TF_ACC") == "" {
 		t.Skip("Acceptance tests are disabled.")
 	}

internal/provider/organization_sync_settings_resource_test.go

@@ -18,6 +18,7 @@ import (
 )
 
 func TestAccOrganizationSyncSettingsResource(t *testing.T) {
+	t.Parallel()
 	if os.Getenv("TF_ACC") == "" {
 		t.Skip("Acceptance tests are disabled.")
 	}

internal/provider/provider_test.go

@@ -16,6 +16,7 @@ var testAccProtoV6ProviderFactories = map[string]func() (tfprotov6.ProviderServe
 }
 
 func testAccPreCheck(t *testing.T) {
+	t.Parallel()
 	// You can add code here to run prior to any test case execution, for example assertions
 	// about the appropriate environment variables being set are common to see in a pre-check
 	// function.

internal/provider/provisioner_key_resource_test.go

@@ -18,6 +18,7 @@ import (
 )
 
 func TestAccProvisionerKeyResource(t *testing.T) {
+	t.Parallel()
 	if os.Getenv("TF_ACC") == "" {
 		t.Skip("Acceptance tests are disabled.")
 	}

internal/provider/template_data_source_test.go

@@ -20,6 +20,7 @@ import (
 )
 
 func TestAccTemplateDataSource(t *testing.T) {
+	t.Parallel()
 	if os.Getenv("TF_ACC") == "" {
 		t.Skip("Acceptance tests are disabled.")
 	}

internal/provider/template_resource_test.go

@@ -23,6 +23,7 @@ import (
 )
 
 func TestAccTemplateResource(t *testing.T) {
+	t.Parallel()
 	if os.Getenv("TF_ACC") == "" {
 		t.Skip("Acceptance tests are disabled.")
 	}
@@ -637,6 +638,7 @@ func TestAccTemplateResource(t *testing.T) {
 }
 
 func TestAccTemplateResourceEnterprise(t *testing.T) {
+	t.Parallel()
 	if os.Getenv("TF_ACC") == "" {
 		t.Skip("Acceptance tests are disabled.")
 	}
@@ -782,6 +784,7 @@ func TestAccTemplateResourceEnterprise(t *testing.T) {
 }
 
 func TestAccTemplateResourceAGPL(t *testing.T) {
+	t.Parallel()
 	if os.Getenv("TF_ACC") == "" {
 		t.Skip("Acceptance tests are disabled.")
 	}
@@ -875,6 +878,7 @@ func TestAccTemplateResourceAGPL(t *testing.T) {
 }
 
 func TestAccTemplateResourceVariables(t *testing.T) {
+	t.Parallel()
 	cfg := `
 provider coderd {
 	url   = %q
@@ -1137,6 +1141,7 @@ func testAccCheckNumTemplateVersions(ctx context.Context, client *codersdk.Clien
 }
 
 func TestReconcileVersionIDs(t *testing.T) {
+	t.Parallel()
 	aUUID := uuid.New()
 	bUUID := uuid.New()
 	cases := []struct {

internal/provider/user_data_source_test.go

@@ -16,6 +16,7 @@ import (
 )
 
 func TestAccUserDataSource(t *testing.T) {
+	t.Parallel()
 	if os.Getenv("TF_ACC") == "" {
 		t.Skip("Acceptance tests are disabled.")
 	}

internal/provider/user_resource_test.go

@@ -14,6 +14,7 @@ import (
 )
 
 func TestAccUserResource(t *testing.T) {
+	t.Parallel()
 	if os.Getenv("TF_ACC") == "" {
 		t.Skip("Acceptance tests are disabled.")
 	}

internal/provider/uuid_internal_test.go

@@ -50,7 +50,6 @@ func TestUUIDTypeValueFromTerraform(t *testing.T) {
 	for _, test := range tests {
 		test := test
 		t.Run(test.name, func(t *testing.T) {
-			t.Parallel()
 			ctx := context.Background()
 
 			actual, err := uuidType.ValueFromTerraform(UUIDType, ctx, test.input)
@@ -85,7 +84,6 @@ func TestUUIDToStringValue(t *testing.T) {
 	for name, test := range tests {
 		name, test := name, test
 		t.Run(name, func(t *testing.T) {
-			t.Parallel()
 			ctx := context.Background()
 
 			s, _ := test.uuid.ToStringValue(ctx)

internal/provider/workspace_proxy_resource_test.go

@@ -15,6 +15,7 @@ import (
 )
 
 func TestAccWorkspaceProxyResource(t *testing.T) {
+	t.Parallel()
 	if os.Getenv("TF_ACC") == "" {
 		t.Skip("Acceptance tests are disabled.")
 	}
@@ -56,6 +57,7 @@ func TestAccWorkspaceProxyResource(t *testing.T) {
 }
 
 func TestAccWorkspaceProxyResourceAGPL(t *testing.T) {
+	t.Parallel()
 	if os.Getenv("TF_ACC") == "" {
 		t.Skip("Acceptance tests are disabled.")
 	}