chore: add t.Parallel() to speed up tests (#184)

This makes the test suite run in under a minute, compared to several
minutes per run before.

Author: aslilac

.golangci.yml

@@ -18,9 +18,16 @@ linters:
     - makezero
     - misspell
     - nilerr
+    - paralleltest
     - predeclared
     - staticcheck
     - unconvert
     - unparam
     - unused
     - usetesting
+
+linters-settings:
+  paralleltest:
+    # Terraform acceptance subtests all share a Coder instance, and cannot run
+    # in parallel.
+    ignore-missing-subtests: true

Makefile

@@ -4,8 +4,8 @@ fmt:
 	go fmt ./...
 	terraform fmt -recursive
 
-vet:
-	go vet ./...
+lint:
+	golangci-lint run ./...
 
 gen:
 	go generate ./...

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")
 	}
@@ -145,6 +146,8 @@ func TestIntegration(t *testing.T) {
 		},
 	} {
 		t.Run(tt.name, func(t *testing.T) {
+			t.Parallel()
+
 			client := StartCoder(ctx, t, tt.name, true)
 			wd, err := os.Getwd()
 			require.NoError(t, err)

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.Helper()
 	// 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.