remove some parallel tests

Emyrk · Emyrk · commit cd90dff574ff · 2025-04-28T11:41:14.000-05:00
diff --git a/coderd/idpsync/group_test.go b/coderd/idpsync/group_test.go
@@ -248,8 +248,11 @@ func TestGroupSyncTable(t *testing.T) {
 
 	for _, tc := range testCases {
 		tc := tc
+		// The final test, "AllTogether", cannot run in parallel.
+		// These tests are nearly instant using the memory db, so
+		// this is still fast without being in parallel.
+		//nolint:paralleltest, tparallel
 		t.Run(tc.Name, func(t *testing.T) {
-			t.Parallel()
 
 			db, _ := dbtestutil.NewDB(t)
 			manager := runtimeconfig.NewManager()
@@ -289,8 +292,8 @@ func TestGroupSyncTable(t *testing.T) {
 	// deployment. This tests all organizations being synced together.
 	// The reason we do them individually, is that it is much easier to
 	// debug a single test case.
+	//nolint:paralleltest, tparallel // This should run after all the individual tests
 	t.Run("AllTogether", func(t *testing.T) {
-		t.Parallel()
 
 		db, _ := dbtestutil.NewDB(t)
 		manager := runtimeconfig.NewManager()
