Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test case for @JsonCreator on enum #119

Closed
wants to merge 351 commits into from

Conversation

kukido
Copy link

@kukido kukido commented Jul 19, 2017

No description provided.

Remove @ignored from a Jackson test since the feature is now available in the version.
Jackson now sort alphabetically the creator properties if asked to.
…enerator find a java.lang.Object property
It is now possible to add custom serializer and deserializer.
It just requires the user to create a class that extends AbstractConfiguration and add it into a gwt property.
The generator will parse the user configuration and use the defined serializer/deserializer when it encounters the type.
This will allows to easily customize the identity through a parameter
This will allow to gives parameters that are defined on properties annotations like @jsonformat
Nicolas Morel and others added 25 commits August 7, 2016 18:42
GWT 2.8.0-rc1 no longer include the annotations
Updated jackson version to 2.8.2
@kukido
Copy link
Author

kukido commented Jul 19, 2017

Here's the log message from the test run.

[INFO] testDeserializeValue(com.github.nmorel.gwtjackson.client.deser.EnumWithAnnotationDeserializerTest)  Time elapsed: 0.024 sec  <<< ERROR!
[INFO] java.lang.IllegalArgumentException: No enum constant com.github.nmorel.gwtjackson.client.deser.EnumWithAnnotationDeserializerTest.Direction.1
[INFO]  at java.lang.Enum.valueOf(Enum.java:238)

@kukido
Copy link
Author

kukido commented Jul 19, 2017

For issue #118

@nmorel nmorel changed the title Added test case Test case for @JsonCreator on enum Jul 21, 2017
@nmorel nmorel closed this Feb 21, 2021
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.

None yet