Skip to content

Conversation

@kscaldef
Copy link
Contributor

In older versions of the module, this test failed, although it currently passes. This is to ensure we don't regress in the future.

Old failure mode:

[info] - should honor the property naming strategy *** FAILED ***
[info] com.fasterxml.jackson.databind.JsonMappingException: Could not find creator property with name 'camel_case'
[info] at [Source: java.io.StringReader@30504d67; line: 1, column: 1]
[info] at com.fasterxml.jackson.databind.JsonMappingException.from(JsonMappingException.java:164)
[info] at com.fasterxml.jackson.databind.DeserializationContext.mappingException(DeserializationContext.java:608)
[info] at com.fasterxml.jackson.databind.deser.BeanDeserializerFactory.addBeanProps(BeanDeserializerFactory.java:551)
[info] at com.fasterxml.jackson.databind.deser.BeanDeserializerFactory.buildBeanDeserializer(BeanDeserializerFactory.java:267)
[info] at com.fasterxml.jackson.databind.deser.BeanDeserializerFactory.createBeanDeserializer(BeanDeserializerFactory.java:171)
[info] at com.fasterxml.jackson.databind.deser.DeserializerCache._createDeserializer(DeserializerCache.java:388)
[info] at com.fasterxml.jackson.databind.deser.DeserializerCache._createAndCache2(DeserializerCache.java:264)
[info] at com.fasterxml.jackson.databind.deser.DeserializerCache._createAndCacheValueDeserializer(DeserializerCache.java:244)
[info] at com.fasterxml.jackson.databind.deser.DeserializerCache.findValueDeserializer(DeserializerCache.java:143)
[info] at com.fasterxml.jackson.databind.DeserializationContext.findRootValueDeserializer(DeserializationContext.java:342)
[info] ...

…ng case classes

In older versions of the module, this failed, although it currently passes.  Adding this test
to ensure we don't regress in the future.
christophercurrie added a commit that referenced this pull request Feb 13, 2013
Test deserialization with PropertyNamingStrategies
@christophercurrie christophercurrie merged commit cdaf5ec into FasterXML:master Feb 13, 2013
@christophercurrie
Copy link
Member

Awesome, thank for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants