Commit0bc49ff

authored

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.

1 parent7f6e513 commit0bc49ffCopy full SHA for 0bc49ff

File tree

-2

lines changed

-2

lines changed

Lines changed: 1 addition & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -225,9 +225,8 @@ func TestRoleSyncTable(t *testing.T) {`
`225`	`225`	`// deployment. This tests all organizations being synced together.`
`226`	`226`	`// The reason we do them individually, is that it is much easier to`
`227`	`227`	`// debug a single test case.`
	`228`	`+//nolint:paralleltest, tparallel // This should run after all the individual tests`
`228`	`229`	`t.Run("AllTogether",func(t*testing.T) {`
`229`		`-t.Parallel()`
`230`		`-`
`231`	`230`	`db,_:=dbtestutil.NewDB(t)`
`232`	`231`	`manager:=runtimeconfig.NewManager()`
`233`	`232`	`s:=idpsync.NewAGPLSync(slogtest.Make(t,&slogtest.Options{`

Comments

(0)