Skip to content

Commit

Permalink
Remove parallel from CRDB tests to address flakiness
Browse files Browse the repository at this point in the history
  • Loading branch information
josephschorr committed Feb 3, 2025
1 parent be575b1 commit a86e024
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions internal/datastore/crdb/crdb_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,6 @@ func crdbTestVersion() string {
}

func TestCRDBDatastoreWithoutIntegrity(t *testing.T) {
t.Parallel()
b := testdatastore.RunCRDBForTesting(t, "", crdbTestVersion())
test.All(t, test.DatastoreTesterFunc(func(revisionQuantization, gcInterval, gcWindow time.Duration, watchBufferLength uint16) (datastore.Datastore, error) {
ctx := context.Background()
Expand Down Expand Up @@ -172,7 +171,6 @@ var defaultKeyForTesting = proxy.KeyConfig{
}

func TestCRDBDatastoreWithIntegrity(t *testing.T) {
t.Parallel()
b := testdatastore.RunCRDBForTesting(t, "", crdbTestVersion())

test.All(t, test.DatastoreTesterFunc(func(revisionQuantization, gcInterval, gcWindow time.Duration, watchBufferLength uint16) (datastore.Datastore, error) {
Expand Down Expand Up @@ -224,7 +222,6 @@ func TestCRDBDatastoreWithIntegrity(t *testing.T) {
}

func TestWatchFeatureDetection(t *testing.T) {
t.Parallel()
pool, err := dockertest.NewPool("")
require.NoError(t, err)
cases := []struct {
Expand Down

0 comments on commit a86e024

Please sign in to comment.