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
[RESTEASY-2244] @Produces not picked up well #2123
Conversation
Thanks, @spyrkob. Would you be interested in writing a test in which the return media type comes from the resource class? |
@ronsigal sure, can do that, but without some hackery it won't be possible to reproduce the issue as the correct Producer would be returned by the method.getProduces() already. |
@spyrkob, yeah, you're right. Good point. I suspect that the best practice would be to have a test in place just to prevent any future regressions. With a note explaining that there is currently no way for the test to fail. If you don't mind. ;-) |
9a4c462
to
e8f14de
Compare
@ronsigal added the test, let me know if it looks OK |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@spyrkob , the new test is not compiling, because of HttpResponseCodes
which is in package org.jboss.resteasy.spi
in master.
e8f14de
to
ed6a551
Compare
@asoldano sorry my IDE must have messed up something... should be fixed now |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks
JIRA: https://issues.jboss.org/browse/RESTEASY-2244