Skip to content

Commit 7c53faa

Browse files
califlowerdaveshanley
authored andcommitted
Fix nil pointer dereference in SchemaRenderer test
- Initialize rand field in createSchemaRenderer() test helper - Add missing math/rand import to schema_renderer_test.go This fixes the panic that occurred when tests created empty SchemaRenderer instances without initializing the new rand field.
1 parent 4eb2680 commit 7c53faa

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

renderer/schema_renderer_test.go

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ import (
1010
"errors"
1111
"fmt"
1212
"io"
13+
"math/rand"
1314
"net/url"
1415
"os"
1516
"regexp"
@@ -50,7 +51,9 @@ func createSchemaRenderer() *SchemaRenderer {
5051
}
5152

5253
// return empty renderer, will generate random strings
53-
return &SchemaRenderer{}
54+
return &SchemaRenderer{
55+
rand: rand.New(rand.NewSource(time.Now().UnixNano())),
56+
}
5457
}
5558

5659
func getSchema(schema []byte) *highbase.Schema {

0 commit comments

Comments
 (0)