refactor(jsonschema): change error returned from json.Unmarshal to wrapped error (#83) (#85)

aakhan1 · web-flow · commit 08d62a2939dc · 2020-10-28T15:26:41.000+01:00
this allows the caller get to the underlying error
diff --git a/schema.go b/schema.go
@@ -325,7 +325,7 @@ func (s *Schema) validateSchemakeywords(ctx context.Context, currentState *Valid
 func (s *Schema) ValidateBytes(ctx context.Context, data []byte) ([]KeyError, error) {
 	var doc interface{}
 	if err := json.Unmarshal(data, &doc); err != nil {
-		return nil, fmt.Errorf("error parsing JSON bytes: %s", err.Error())
+		return nil, fmt.Errorf("error parsing JSON bytes: %w", err)
 	}
 	vs := s.Validate(ctx, doc)
 	return *vs.Errs, nil

Original file line number	Diff line number	Diff line change
`@@ -325,7 +325,7 @@ func (s Schema) validateSchemakeywords(ctx context.Context, currentState Valid`
`325`	`325`	`func (s *Schema) ValidateBytes(ctx context.Context, data []byte) ([]KeyError, error) {`
`326`	`326`	`var doc interface{}`
`327`	`327`	`if err := json.Unmarshal(data, &doc); err != nil {`
`328`		`- return nil, fmt.Errorf("error parsing JSON bytes: %s", err.Error())`
	`328`	`+ return nil, fmt.Errorf("error parsing JSON bytes: %w", err)`
`329`	`329`	`}`
`330`	`330`	`vs := s.Validate(ctx, doc)`
`331`	`331`	`return *vs.Errs, nil`