fix: only use recommended settings in root user tests

testcontainers · Oct 30, 2024 · 88ab497 · 88ab497
1 parent f7dc17f
commit 88ab497
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/modules/cockroachdb/cockroachdb_test.go b/modules/cockroachdb/cockroachdb_test.go
@@ -28,6 +28,7 @@ func TestCockroach_NotRoot(t *testing.T) {
 		url: "postgres://test@localhost:xxxxx/defaultdb?sslmode=disable",
 		opts: []testcontainers.ContainerCustomizer{
 			cockroachdb.WithUser("test"),
+      cockroachdb.WithStatements(),
 		},
 	})
 }
@@ -38,6 +39,7 @@ func TestCockroach_Password(t *testing.T) {
 		opts: []testcontainers.ContainerCustomizer{
 			cockroachdb.WithUser("foo"),
 			cockroachdb.WithPassword("bar"),
+      cockroachdb.WithStatements(),
 		},
 	})
 }

diff --git a/modules/cockroachdb/options.go b/modules/cockroachdb/options.go
@@ -71,6 +71,7 @@ func WithTLS(cfg *TLSConfig) Option {
 }
 
 // DefaultStatements are the settings recommended by Cockroach Labs for testing clusters.
+// Note that to use these defaults the user needs to have MODIFYCLUSTERSETTING privilege.
 // See https://www.cockroachlabs.com/docs/stable/local-testing for more information.
 var DefaultStatements = []string{
 	"SET CLUSTER SETTING kv.range_merge.queue_interval = '50ms'",