Movatterモバイル変換

Skip to content

coder/terraform-provider-coderdPublic

NotificationsYou must be signed in to change notification settings
Fork4
Star21

Commitaa3dbcd

aslilac

authored

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

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

1 parentc440681 commitaa3dbcdCopy full SHA for aa3dbcd

File tree

17 files changed

+36

-7

lines changed

17 files changed

+36

-7

lines changed

`‎.golangci.yml‎`

Lines changed: 7 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -18,9 +18,16 @@ linters:`
`18`	`18`	`-makezero`
`19`	`19`	`-misspell`
`20`	`20`	`-nilerr`
	`21`	`+ -paralleltest`
`21`	`22`	`-predeclared`
`22`	`23`	`-staticcheck`
`23`	`24`	`-unconvert`
`24`	`25`	`-unparam`
`25`	`26`	`-unused`
`26`	`27`	`-usetesting`
	`28`	`+`
	`29`	`+linters-settings:`
	`30`	`+paralleltest:`
	`31`	`+# Terraform acceptance subtests all share a Coder instance, and cannot run`
	`32`	`+# in parallel.`
	`33`	`+ignore-missing-subtests:true`

`‎Makefile‎`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -4,8 +4,8 @@ fmt:`
`4`	`4`	`go fmt ./...`
`5`	`5`	`terraform fmt -recursive`
`6`	`6`
`7`		`-vet:`
`8`		`-go vet ./...`
	`7`	`+lint:`
	`8`	`+golangci-lint run ./...`
`9`	`9`
`10`	`10`	`gen:`
`11`	`11`	`go generate ./...`

`‎integration/integration_test.go‎`

Lines changed: 3 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -22,6 +22,7 @@ import (`
`22`	`22`	// - Runs the `main.tf` specified in the given test directory against the Coder deployment
`23`	`23`	// - Asserts the state of the deployment via `codersdk`.
`24`	`24`	`funcTestIntegration(t*testing.T) {`
	`25`	`+t.Parallel()`
`25`	`26`	`ifos.Getenv("TF_ACC")=="1" {`
`26`	`27`	`t.Skip("Skipping integration tests during tf acceptance tests")`
`27`	`28`	`}`
`@@ -145,6 +146,8 @@ func TestIntegration(t *testing.T) {`
`145`	`146`	`},`
`146`	`147`	`} {`
`147`	`148`	`t.Run(tt.name,func(t*testing.T) {`
	`149`	`+t.Parallel()`
	`150`	`+`
`148`	`151`	`client:=StartCoder(ctx,t,tt.name,true)`
`149`	`152`	`wd,err:=os.Getwd()`
`150`	`153`	`require.NoError(t,err)`

`‎internal/provider/group_data_source_test.go‎`

Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -16,6 +16,7 @@ import (`
`16`	`16`	`)`
`17`	`17`
`18`	`18`	`funcTestAccGroupDataSource(t*testing.T) {`
	`19`	`+t.Parallel()`
`19`	`20`	`ifos.Getenv("TF_ACC")=="" {`
`20`	`21`	`t.Skip("Acceptance tests are disabled.")`
`21`	`22`	`}`

`‎internal/provider/group_resource_test.go‎`

Lines changed: 2 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -16,6 +16,7 @@ import (`
`16`	`16`	`)`
`17`	`17`
`18`	`18`	`funcTestAccGroupResource(t*testing.T) {`
	`19`	`+t.Parallel()`
`19`	`20`	`ifos.Getenv("TF_ACC")=="" {`
`20`	`21`	`t.Skip("Acceptance tests are disabled.")`
`21`	`22`	`}`
`@@ -133,6 +134,7 @@ func TestAccGroupResource(t *testing.T) {`
`133`	`134`	`}`
`134`	`135`
`135`	`136`	`funcTestAccGroupResourceAGPL(t*testing.T) {`
	`137`	`+t.Parallel()`
`136`	`138`	`ifos.Getenv("TF_ACC")=="" {`
`137`	`139`	`t.Skip("Acceptance tests are disabled.")`
`138`	`140`	`}`

`‎internal/provider/license_resource_test.go‎`

Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -13,6 +13,7 @@ import (`
`13`	`13`	`)`
`14`	`14`
`15`	`15`	`funcTestAccLicenseResource(t*testing.T) {`
	`16`	`+t.Parallel()`
`16`	`17`	`ifos.Getenv("TF_ACC")=="" {`
`17`	`18`	`t.Skip("Acceptance tests are disabled.")`
`18`	`19`	`}`

`‎internal/provider/organization_data_source_test.go‎`

Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -16,6 +16,7 @@ import (`
`16`	`16`	`)`
`17`	`17`
`18`	`18`	`funcTestAccOrganizationDataSource(t*testing.T) {`
	`19`	`+t.Parallel()`
`19`	`20`	`ifos.Getenv("TF_ACC")=="" {`
`20`	`21`	`t.Skip("Acceptance tests are disabled.")`
`21`	`22`	`}`

`‎internal/provider/organization_resource_test.go‎`

Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -19,6 +19,7 @@ import (`
`19`	`19`	`)`
`20`	`20`
`21`	`21`	`funcTestAccOrganizationResource(t*testing.T) {`
	`22`	`+t.Parallel()`
`22`	`23`	`ifos.Getenv("TF_ACC")=="" {`
`23`	`24`	`t.Skip("Acceptance tests are disabled.")`
`24`	`25`	`}`

`‎internal/provider/organization_sync_settings_resource_test.go‎`

Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -18,6 +18,7 @@ import (`
`18`	`18`	`)`
`19`	`19`
`20`	`20`	`funcTestAccOrganizationSyncSettingsResource(t*testing.T) {`
	`21`	`+t.Parallel()`
`21`	`22`	`ifos.Getenv("TF_ACC")=="" {`
`22`	`23`	`t.Skip("Acceptance tests are disabled.")`
`23`	`24`	`}`

`‎internal/provider/provider_test.go‎`

Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -16,6 +16,7 @@ var testAccProtoV6ProviderFactories = map[string]func() (tfprotov6.ProviderServe`
`16`	`16`	`}`
`17`	`17`
`18`	`18`	`functestAccPreCheck(t*testing.T) {`
	`19`	`+t.Helper()`
`19`	`20`	`// You can add code here to run prior to any test case execution, for example assertions`
`20`	`21`	`// about the appropriate environment variables being set are common to see in a pre-check`
`21`	`22`	`// function.`

0 commit comments

Comments

(0)

[8]ページ先頭

©2009-2025 Movatter.jp