search

crystal-community / msgpack-crystal

MessagePack implementation in Crystal msgpack.org[Crystal]
136 stars 18 forks source link

not correct handle union type in mapping #32

Closed kostya closed 7 years ago

kostya commented 7 years ago 
class A
  MessagePack.mapping({
    bla: {type: String, nilable: true}
  })
end

p A.from_msgpack({"bla" => nil}.to_msgpack) # => #<A:0x100e2af20 @bla=nil>

class B
  MessagePack.mapping({
    bla: String?
  })
end

p B.from_msgpack({"bla" => nil}.to_msgpack) # => Nil assertion failed (Exception)