Change rand.Seed to rand.New

snowflakedb · Jun 25, 2024 · 24b6c54 · 24b6c54
1 parent 527d733
commit 24b6c54
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 13 deletions.
diff --git a/put_get_test.go b/put_get_test.go
@@ -15,21 +15,10 @@ import (
 	"strconv"
 	"strings"
 	"testing"
-	"time"
 )
 
 const createStageStmt = "CREATE OR REPLACE STAGE %v URL = '%v' CREDENTIALS = (%v)"
 
-func randomString(n int) string {
-	rand.Seed(time.Now().UnixNano())
-	alpha := []rune("abcdefghijklmnopqrstuvwxyz")
-	b := make([]rune, n)
-	for i := range b {
-		b[i] = alpha[rand.Intn(len(alpha))]
-	}
-	return string(b)
-}
-
 func TestPutError(t *testing.T) {
 	if isWindows {
 		t.Skip("permission model is different")

diff --git a/telemetry_test.go b/telemetry_test.go
@@ -21,8 +21,8 @@ func TestTelemetryAddLog(t *testing.T) {
 			enabled:   true,
 			flushSize: defaultFlushSize,
 		}
-		rand.Seed(time.Now().UnixNano())
-		randNum := rand.Int() % 10000
+		r := rand.New(rand.NewSource(time.Now().UnixNano()))
+		randNum := r.Int() % 10000
 		for i := 0; i < randNum; i++ {
 			if err := st.addLog(&telemetryData{
 				Message: map[string]string{

diff --git a/util_test.go b/util_test.go
@@ -390,3 +390,13 @@ func skipOnJenkins(t *testing.T, message string) {
 		t.Skip("Skipping test on Jenkins: " + message)
 	}
 }
+
+func randomString(n int) string {
+	r := rand.New(rand.NewSource(time.Now().UnixNano()))
+	alpha := []rune("abcdefghijklmnopqrstuvwxyz")
+	b := make([]rune, n)
+	for i := range b {
+		b[i] = alpha[r.Intn(len(alpha))]
+	}
+	return string(b)
+}