READ_ENUMS_USING_TO_STRING doesn't support null values

[andersonbd1]

The test added in EnumDeserializationTest.java fails w/out the fix in EnumResolver.java.

https://github.com/FasterXML/jackson-databind/pull/2307/files

[andersonbd1]

pull request for the fix is here:
#2307

[andersonbd1]

issue posted to facebook-java-business-sdk as well:
facebook/facebook-java-business-sdk#235

[cowtowncoder]

But is it even legal to return null from toString()?

I guess I could instead throw an exception to indicate bad return value. I am not sure I'm comfortable coercing null into "", although that is a reasonable suggestion in general.

[andersonbd1]

@cowtowncoder - any word on this? I'm not saying my solution is the best/most correct, but can you introduce a way to make the failing test (that I demonstrate) work?

[cowtowncoder]

@andersonbd1 I haven't had time to look but I'll add this to my todo list for another glance. Thank you for reminder.

[andersonbd1]

Thanks @cowtowncoder . I just got a reply from facebook here - facebook/facebook-java-business-sdk#235 (comment)

Sounds like they're more concerned about supporting gson.

[cowtowncoder]

@andersonbd1 that makes sense wrt Facebook, they don't really need to worry about json handling performance. :)

But regardless I am interested in Jackson side of things.