Merge pull request #2200 from josephschorr/pg-serialization-test-perf

Disable retries on the serialization test for PG
authzed · Jan 9, 2025 · a2ac1e6 · a2ac1e6
2 parents 7b9f388 + bb005a2
commit a2ac1e6
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/internal/datastore/postgres/postgres_shared_test.go b/internal/datastore/postgres/postgres_shared_test.go
@@ -29,6 +29,7 @@ import (
 	"github.com/authzed/spicedb/internal/testfixtures"
 	testdatastore "github.com/authzed/spicedb/internal/testserver/datastore"
 	"github.com/authzed/spicedb/pkg/datastore"
+	"github.com/authzed/spicedb/pkg/datastore/options"
 	"github.com/authzed/spicedb/pkg/datastore/test"
 	"github.com/authzed/spicedb/pkg/migrate"
 	"github.com/authzed/spicedb/pkg/namespace"
@@ -394,7 +395,7 @@ func SerializationErrorTest(t *testing.T, ds datastore.Datastore) {
 		}
 		rwt.(*pgReadWriteTXN).tx = txWithSerializationError{rwt.(*pgReadWriteTXN).tx}
 		return rwt.WriteRelationships(ctx, updates)
-	})
+	}, options.WithDisableRetries(true) /* ensures the error is returned immediately */)
 
 	require.Contains(err.Error(), "unable to write relationships due to a serialization error")
 }