Add Schema Validation Error Message about json-everything

[whatevergeek]

thanks a lot for the clarification @gregsdennis

I've set the OutputFormat as advised and it works now.
Here's how I've implemented:

    public class Validator
    {
        public string Schema { get; set; }
        public bool IsValid(string data, out string result)
        {
            if (!string.IsNullOrWhiteSpace(Schema))
            {
                var doc = JsonDocument.Parse(data);
                JsonSchema schema = JsonSchema.FromText(Schema);

                var options = new ValidationOptions { OutputFormat = OutputFormat.Detailed };
                var validationResult = schema.Validate(doc.RootElement, options);

                result = GetValidationResultMessage(validationResult);
                return validationResult.IsValid;
            }
            else
            {
                throw new Exception("Schema not set");
            }
        }

        private string GetValidationResultMessage(ValidationResults validationResult)
        {
            var resultMessage = string.Empty;
            if (validationResult.IsValid)
            {
                resultMessage = "Validation Succeeded.";
            }
            else if (validationResult.NestedResults.Count > 0)
            {
                resultMessage = "Validation Errors: ";
                foreach (var result in validationResult.NestedResults)
                {
                    resultMessage += $"{result.Message} found at {result.SchemaLocation.Source}| ";
                }
            }
            else
            {
                resultMessage = $"Validation Errors: {validationResult.Message} found at {validationResult.SchemaLocation.Source}| ";
            }

            return resultMessage;
        }

    }

I've added some test cases in the sample demo project to confirm the behaviour.
Just an observation, if validationResult.NestedResults.Count is more than 1, the main validationResult.Message returns null.
Hence, I had to check for it in my implementation above. Is this behavior intended?
Anyway, I'm ok to proceed with this. Feel free to let me know if there's a better usage.

[whatevergeek]

thanks a lot @gregsdennis
i'm so glad to be able to finally use your library
i'm closing this question now
all the best and thanks for your open source effort/contributions
highly appreciated! :-)

[gregsdennis]

How coincidental! I was looking at your repo yesterday.

It looks like you're getting just the basic validation result (and the default because it's faster), which is just a pass/fail. To get more information, including an error message, you need to set up for one of the other output formats.

var options = new ValidationOptions { OutputFormat = OutputFormat.Detailed };
var results = schema.Validate(instance, options);

See the docs for more information.

[gregsdennis]

This behavior is intended. Keywords that hold schemas are called "applicators." These keywords have no message (in this implementation due to my choice) because it's the messages from the contained keywords that hold the interesting data.

The spec says that the error messaging is something that the implementation can decide.