make json schema validation multi-error sorted (#96)

Author: jf-tech

schema_test.go

@@ -39,7 +39,7 @@ func TestNewSchema(t *testing.T) {
 			name:   "json schema validation for header failed",
 			schema: `{"parser_settings": {"versionx": "9999", "file_format_type": "exe" }}`,
 			exts:   nil,
-			err:    "schema 'test-schema' validation failed:\nparser_settings: version is required\nparser_settings: Additional property versionx is not allowed",
+			err:    "schema 'test-schema' validation failed:\nparser_settings: Additional property versionx is not allowed\nparser_settings: version is required",
 		},
 		{
 			name:   "no supported schema handler",

validation/jsonvalidate.go

@@ -4,6 +4,7 @@ package validation
 
 import (
 	"fmt"
+	"sort"
 	"strings"
 
 	"github.com/xeipuuv/gojsonschema"
@@ -25,6 +26,7 @@ func SchemaValidate(schemaName string, schemaContent []byte, jsonSchema string)
 	for _, err := range result.Errors() {
 		errs = append(errs, err.String())
 	}
+	sort.Strings(errs)
 	if len(errs) == 1 {
 		return fmt.Errorf("schema '%s' validation failed: %s", schemaName, errs[0])
 	}

validation/jsonvalidate_test.go

@@ -57,7 +57,7 @@ func TestSchemaValidate(t *testing.T) {
 						"unknown": "blah"
 					}
 				}`,
-			expectedErr: "schema 'test-schema' validation failed:\nparser_settings: file_format_type is required\nparser_settings: Additional property unknown is not allowed",
+			expectedErr: "schema 'test-schema' validation failed:\nparser_settings: Additional property unknown is not allowed\nparser_settings: file_format_type is required",
 		},
 	} {
 		t.Run(test.name, func(t *testing.T) {