Hash value of null breaks parsing

[toadle]

Hey everybody, I don't now if this would belong into #203

but I had a GraphQL-JSON-collection that I tried to parse with representable.
Because of an error the collection came out as:

{
...,
collection: [
 {
...
},
null,
{
...
}
]
...}

This completely broke the parsing flow and error-out with NoMethodError: undefined method has_key? for nil:NilClass on

representable/lib/representable/hash/binding.rb

Line 12 in df685c8

def read(hash, as)

Expected behaviour would have been to just return nothing. I could even imagine a scenario, where null-entries would result in nil.

I resorted to filter the collection with a skip_parse-first, but an exception in this case is kinda annoying.

[yogeshjain999]

@toadle thanks for creating the PR!

Changes looks good to me! Could you also add test for this ?

[toadle]

@yogeshjain999 here you go!

[yogeshjain999]

@toadle we don't have to consider rescuing NoMethodError now as we know it shouldn't occur. If it does, then it must be due to some other bug and should be handled differently.

So in this test, we can just assert on return value of read, like in above test.

it "..." do
  assert_equal Representable::Binding::FragmentNotFound, @property.read(nil, "song")
end

[toadle]

@yogeshjain999 here you go

[toadle]

@yogeshjain999 Wanna merge?

[yogeshjain999]

Sorry, almost forgot :)