Handle MultiJson ParseErrors

[ethanabrooks]

Previously, the first rspec test was failing because the matcher.rb was only rescuing JSON::ParseErrors but not MultiJson::ParseErrors. For whatever reason, the empty string raised the latter, not the former, error.

With this minor fix, all tests are green.

[seanpdoyle]

Thanks for the patch!

What do you think about combining the errors on a single line:

rescue JSON::ParserError, MultiJson::ParseError
  raise InvalidSchemaError
end

[ethanabrooks]

Sold! Let's do it.

[seanpdoyle]

On further inspection, what is the source of the MultiJson::ParseError?

What happens when the multi_json isn't included as a dependency?

[ethanabrooks]

For some reason, it tries to parse the empty string JSON schema in the first rspec test with MultiJson, not the regular JSON validator.

I looked up the docs for MultiJson, and it seems to be kind of a catch-all JSON coder. Not sure how Ruby decides between multijson and the regular JSON validator.

On Oct 6, 2015, at 9:10 PM, Sean Doyle notifications@github.com wrote:

On further inspection, what is the source of the MultiJson::ParseError?

What happens when the multi_json https://github.com/intridea/multi_json isn't included?

—
Reply to this email directly or view it on GitHub #20 (comment).

[ethanabrooks]

A small follow up:
I placed a break point just above the call to JSON::Validator and confirmed that, for some reason, when we call that method, ruby invokes the MultiJson library. The documentation on JSON is pretty sparse, but I will dig a little further to figure out why.

On Oct 6, 2015, at 9:13 PM, Ethan Brooks ethanabrooks@gmail.com wrote:

For some reason, it tries to parse the empty string JSON schema in the first rspec test with MultiJson, not the regular JSON validator.

I looked up the docs for MultiJson, and it seems to be kind of a catch-all JSON coder. Not sure how Ruby decides between multijson and the regular JSON validator.

On Oct 6, 2015, at 9:10 PM, Sean Doyle <notifications@github.com mailto:notifications@github.com> wrote:

On further inspection, what is the source of the MultiJson::ParseError?

What happens when the multi_json https://github.com/intridea/multi_json isn't included?

—
Reply to this email directly or view it on GitHub #20 (comment).