test: fix flake in TestRoleSyncTable with test cases sharing resources

Emyrk · Emyrk · commit 1a2e47f492b3 · 2025-04-16T18:08:33.000-05:00
The test case definition shares maps that can have concurrent access
if run in parallel. This fixes that
diff --git a/coderd/idpsync/role_test.go b/coderd/idpsync/role_test.go
@@ -225,9 +225,8 @@ func TestRoleSyncTable(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()
 		s := idpsync.NewAGPLSync(slogtest.Make(t, &slogtest.Options{
