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

Add @BeanParam support for Jersey2. #536

Closed
wants to merge 1 commit into from
Closed

Add @BeanParam support for Jersey2. #536

wants to merge 1 commit into from

Conversation

shuz
Copy link
Contributor

@shuz shuz commented Apr 20, 2014

Related Issue: #446
Add @BeanParam support.
Add unit test for @BeanParam support.
Fix PathParamTargetTest. The list detailsOps has two items, POST and GET. The order is not guaranteed. Add filter to make sure POST is used for the unit test.

Related Issue: #446
Add @BeanParam support.
Add unit test for @BeanParam support.
Fix PathParamTargetTest. The list detailsOps has two items, POST and GET. The order is not guaranteed. Add filter to make sure POST is used for the unit test.
@shuz
Copy link
Contributor Author

shuz commented Apr 20, 2014

The error is: org.scalatest.exceptions.TestFailedException: "...ties":{"theCount":{"[$ref":"Int]"}}}]" was not equal to "...ties":{"theCount":{"[type":"integer","format":"int32]"}}}]"

Seems Int is not recognized as a primitive. Tried to run the test case locally and it could pass.

@shuz shuz closed this Apr 20, 2014
@shuz
Copy link
Contributor Author

shuz commented Apr 20, 2014

After investigation, the issue is related to a unit test bug. I can reproduce the behavior by adding:

ModelConverters.addConverter(new SnakeCaseConverter, true)

To PropertyAnnotationTest.

@shuz
Copy link
Contributor Author

shuz commented Apr 20, 2014

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.

1 participant