Commitd3ed6fe

authored

chore(coderd/autobuild): use dbtestutil.WillUsePostgres instead of os.Getenv in test (#18145)

Standardizing on `WillUsePostgres` will make it easier to remove thecheck entirely once dbmem is removed.Related to#15109.

1 parent782d01b commitd3ed6feCopy full SHA for d3ed6fe

File tree

-2

lines changed

-2

lines changed

Lines changed: 1 addition & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -2,7 +2,6 @@ package autobuild_test`
`2`	`2`
`3`	`3`	`import (`
`4`	`4`	`"context"`
`5`		`-"os"`
`6`	`5`	`"testing"`
`7`	`6`	`"time"`
`8`	`7`
`@@ -741,7 +740,7 @@ func TestExecutorWorkspaceAutostopNoWaitChangedMyMind(t *testing.T) {`
`741`	`740`	`}`
`742`	`741`
`743`	`742`	`funcTestExecutorAutostartMultipleOK(t*testing.T) {`
`744`		`-ifos.Getenv("DB")=="" {`
	`743`	`+if!dbtestutil.WillUsePostgres() {`
`745`	`744`	t.Skip(`This test only really works when using a "real" database, similar to a HA setup`)
`746`	`745`	`}`
`747`	`746`

Comments

(0)