reduced load for race tests

Author: gammazero

go.mod

@@ -4,6 +4,7 @@ go 1.23
 
 require (
 	github.com/google/uuid v1.6.0
+	github.com/ipfs/go-detect-race v0.0.1
 	github.com/ipfs/go-ipfs-delay v0.0.1
 	go.opentelemetry.io/otel v1.16.0
 	go.opentelemetry.io/otel/trace v1.16.0

go.sum

@@ -9,6 +9,8 @@ github.com/google/go-cmp v0.5.9 h1:O2Tfq5qg4qc4AmwVlvv0oLiVAGB7enBSJ2x2DqQFi38=
 github.com/google/go-cmp v0.5.9/go.mod h1:17dUlkBOakJ0+DkrSSNjCkIjxS6bF9zb3elmeNGIjoY=
 github.com/google/uuid v1.6.0 h1:NIvaJDMOsjHA8n1jAhLSgzrAzy1Hgr+hNrb57e+94F0=
 github.com/google/uuid v1.6.0/go.mod h1:TIyPZe4MgqvfeYDBFedMoGGpEw/LqOeaOT+nhxU+yHo=
+github.com/ipfs/go-detect-race v0.0.1 h1:qX/xay2W3E4Q1U7d9lNs1sU9nvguX0a7319XbyQ6cOk=
+github.com/ipfs/go-detect-race v0.0.1/go.mod h1:8BNT7shDZPo99Q74BpGMK+4D8Mn4j46UU0LZ723meps=
 github.com/ipfs/go-ipfs-delay v0.0.1 h1:r/UXYyRcddO6thwOnhiznIAiSvxMECGgtv35Xs1IeRQ=
 github.com/ipfs/go-ipfs-delay v0.0.1/go.mod h1:8SP1YXK1M1kXuc4KJZINY3TQQ03J2rwBG9QfXmbRPrw=
 github.com/kr/pretty v0.2.0 h1:s5hAObm+yFO5uHYt5dYjxi2rXrsnmRpJx4OYvIWUaQs=

test/basic_tests.go

@@ -11,13 +11,22 @@ import (
 
 	dstore "github.com/ipfs/go-datastore"
 	dsq "github.com/ipfs/go-datastore/query"
+	detectrace "github.com/ipfs/go-detect-race"
 )
 
 // ElemCount sets with how many elements the datastore suit
 // tests are usually run with. Best to set to round numbers like
 // 20, 30, 40... and at least to 20.
 var ElemCount = 100
 
+func init() {
+	// Reduce the default element count when the race detector is enabled so these tests don't
+	// take forever.
+	if detectrace.WithRace() {
+		ElemCount = 20
+	}
+}
+
 func TestElemCount(t *testing.T) {
 	if ElemCount < 20 {
 		t.Fatal("ElemCount should be set to 20 at least")

test/suite.go

@@ -8,6 +8,7 @@ import (
 
 	dstore "github.com/ipfs/go-datastore"
 	query "github.com/ipfs/go-datastore/query"
+	detectrace "github.com/ipfs/go-detect-race"
 )
 
 // BasicSubtests is a list of all basic tests.
@@ -21,7 +22,13 @@ var BasicSubtests = []func(t *testing.T, ds dstore.Datastore){
 	SubtestManyKeysAndQuery,
 	SubtestReturnSizes,
 	SubtestBasicSync,
-	SubtestCombinations,
+}
+
+// Only enable the expensive "combinations" test when not running the race detector.
+func init() {
+	if !detectrace.WithRace() {
+		BasicSubtests = append(BasicSubtests, SubtestCombinations)
+	}
 }
 
 // BatchSubtests is a list of all basic batching datastore tests.