refactor(coderd/database): minor refactoring in tests

evgeniy-scherbina · evgeniy-scherbina · commit 574cdf645bf9 · 2025-02-27T10:47:34.000-05:00
diff --git a/coderd/database/querier_test.go b/coderd/database/querier_test.go
@@ -2166,10 +2166,10 @@ func TestExpectOne(t *testing.T) {
 }
 
 func TestGetProvisionerJobsByIDsWithQueuePosition(t *testing.T) {
-	t.Parallel()
 	if !dbtestutil.WillUsePostgres() {
 		t.SkipNow()
 	}
+	t.Parallel()
 
 	now := dbtime.Now()
 	ctx := testutil.Context(t, testutil.WaitShort)
@@ -2470,10 +2470,10 @@ func TestGetProvisionerJobsByIDsWithQueuePosition(t *testing.T) {
 }
 
 func TestGetProvisionerJobsByIDsWithQueuePosition_MixedStatuses(t *testing.T) {
-	t.Parallel()
 	if !dbtestutil.WillUsePostgres() {
 		t.SkipNow()
 	}
+	t.Parallel()
 
 	db, _ := dbtestutil.NewDB(t)
 	now := dbtime.Now()
@@ -2611,10 +2611,10 @@ func TestGetProvisionerJobsByIDsWithQueuePosition_MixedStatuses(t *testing.T) {
 }
 
 func TestGetProvisionerJobsByIDsWithQueuePosition_OrderValidation(t *testing.T) {
-	t.Parallel()
 	if !dbtestutil.WillUsePostgres() {
 		t.SkipNow()
 	}
+	t.Parallel()
 
 	db, _ := dbtestutil.NewDB(t)
 	now := dbtime.Now()

Original file line number	Diff line number	Diff line change
`@@ -2166,10 +2166,10 @@ func TestExpectOne(t *testing.T) {`
`2166`	`2166`	`}`
`2167`	`2167`
`2168`	`2168`	`func TestGetProvisionerJobsByIDsWithQueuePosition(t *testing.T) {`
`2169`		`- t.Parallel()`
`2170`	`2169`	`if !dbtestutil.WillUsePostgres() {`
`2171`	`2170`	`t.SkipNow()`
`2172`	`2171`	`}`
	`2172`	`+ t.Parallel()`
`2173`	`2173`
`2174`	`2174`	`now := dbtime.Now()`
`2175`	`2175`	`ctx := testutil.Context(t, testutil.WaitShort)`
`@@ -2470,10 +2470,10 @@ func TestGetProvisionerJobsByIDsWithQueuePosition(t *testing.T) {`
`2470`	`2470`	`}`
`2471`	`2471`
`2472`	`2472`	`func TestGetProvisionerJobsByIDsWithQueuePosition_MixedStatuses(t *testing.T) {`
`2473`		`- t.Parallel()`
`2474`	`2473`	`if !dbtestutil.WillUsePostgres() {`
`2475`	`2474`	`t.SkipNow()`
`2476`	`2475`	`}`
	`2476`	`+ t.Parallel()`
`2477`	`2477`
`2478`	`2478`	`db, _ := dbtestutil.NewDB(t)`
`2479`	`2479`	`now := dbtime.Now()`
`@@ -2611,10 +2611,10 @@ func TestGetProvisionerJobsByIDsWithQueuePosition_MixedStatuses(t *testing.T) {`
`2611`	`2611`	`}`
`2612`	`2612`
`2613`	`2613`	`func TestGetProvisionerJobsByIDsWithQueuePosition_OrderValidation(t *testing.T) {`
`2614`		`- t.Parallel()`
`2615`	`2614`	`if !dbtestutil.WillUsePostgres() {`
`2616`	`2615`	`t.SkipNow()`
`2617`	`2616`	`}`
	`2617`	`+ t.Parallel()`
`2618`	`2618`
`2619`	`2619`	`db, _ := dbtestutil.NewDB(t)`
`2620`	`2620`	`now := dbtime.Now()`