Skip to content

Commit 5489803

Browse files
authored
fix: dbpurge: disable parallel tests (#13848)
1 parent c1440ac commit 5489803

File tree

1 file changed

+5
-10
lines changed

1 file changed

+5
-10
lines changed

coderd/database/dbpurge/dbpurge_test.go

Lines changed: 5 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -43,9 +43,8 @@ func TestPurge(t *testing.T) {
4343
require.NoError(t, err)
4444
}
4545

46+
//nolint:paralleltest // It uses LockIDDBPurge.
4647
func TestDeleteOldWorkspaceAgentStats(t *testing.T) {
47-
t.Parallel()
48-
4948
db, _ := dbtestutil.NewDB(t)
5049
logger := slogtest.Make(t, &slogtest.Options{IgnoreErrors: true}).Leveled(slog.LevelDebug)
5150

@@ -162,9 +161,8 @@ func containsWorkspaceAgentStat(stats []database.GetWorkspaceAgentStatsRow, need
162161
})
163162
}
164163

164+
//nolint:paralleltest // It uses LockIDDBPurge.
165165
func TestDeleteOldWorkspaceAgentLogs(t *testing.T) {
166-
t.Parallel()
167-
168166
db, _ := dbtestutil.NewDB(t)
169167
org := dbgen.Organization(t, db, database.Organization{})
170168
user := dbgen.User(t, db, database.User{})
@@ -175,9 +173,8 @@ func TestDeleteOldWorkspaceAgentLogs(t *testing.T) {
175173
logger := slogtest.Make(t, &slogtest.Options{IgnoreErrors: true})
176174
now := dbtime.Now()
177175

176+
//nolint:paralleltest // It uses LockIDDBPurge.
178177
t.Run("AgentHasNotConnectedSinceWeek_LogsExpired", func(t *testing.T) {
179-
t.Parallel()
180-
181178
ctx, cancel := context.WithTimeout(context.Background(), testutil.WaitShort)
182179
defer cancel()
183180

@@ -209,9 +206,8 @@ func TestDeleteOldWorkspaceAgentLogs(t *testing.T) {
209206
require.NotContains(t, agentLogs, t.Name())
210207
})
211208

209+
//nolint:paralleltest It uses LockIDDBPurge.
212210
t.Run("AgentConnectedSixDaysAgo_LogsValid", func(t *testing.T) {
213-
t.Parallel()
214-
215211
ctx, cancel := context.WithTimeout(context.Background(), testutil.WaitShort)
216212
defer cancel()
217213

@@ -283,9 +279,8 @@ func containsAgentLog(daemons []database.WorkspaceAgentLog, output string) bool
283279
})
284280
}
285281

282+
//nolint:paralleltest // It uses LockIDDBPurge.
286283
func TestDeleteOldProvisionerDaemons(t *testing.T) {
287-
t.Parallel()
288-
289284
db, _ := dbtestutil.NewDB(t, dbtestutil.WithDumpOnFailure())
290285
defaultOrg := dbgen.Organization(t, db, database.Organization{})
291286
logger := slogtest.Make(t, &slogtest.Options{IgnoreErrors: true})

0 commit comments

Comments
 (0)