test: fix flake in TestRoleSyncTable with test cases sharing resources (#17441)

The test case definition shares maps that can have concurrent access if run in parallel.
This commit is contained in:
Steven Masley
2025-04-17 00:14:11 +00:00
committed by GitHub
parent 7f6e5139eb
commit 0bc49ff5ae
+1 -2
View File
@@ -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{