Skip to content

[Jaxrs-*] Java-based generator + unittest for spec/cxf/resteasy #5028

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

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

jfiala
Copy link
Contributor

@jfiala jfiala commented Mar 11, 2017

PR checklist

  • Read the contribution guildelines.
  • Ran the shell/batch script under ./bin/ to update Petstore sample so that CIs can verify the change. (For instance, only need to run ./bin/{LANG}-petstore.sh and ./bin/security/{LANG}-petstore.sh if updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates)
  • Filed the PR against the correct branch: master for non-breaking changes and 2.3.0 branch for breaking (non-backward compatible) changes.

Description of the PR

see #4087, generator + unittest für cxf, also updated samples
pls merge PR #4995 first, this PR was branched from jfiala:generator_jaxrs-spec_4087

Currently covered languages/shellscripts:

  • jaxrs-cxf
    • jaxrs-cxf-petstore-server.sh
  • jaxrs-cxf-client
    • currently no shellscript
  • jaxrs-resteasy
    • jaxrs-resteasy-petstore-server.sh
    • jaxrs-resteasy-joda-petstore-server.sh
  • jaxrs-spec
    • jaxrs-spec-petstore-server.sh

@jfiala
Copy link
Contributor Author

jfiala commented Mar 11, 2017

@wing328 I added the commandline invocation as well, this is equivalent to the shellscript (see JaxrsSpec_Generator_Petstore.java and JavaCXFServer_Generator_Petstore.java).

Pls review and let me know if I should do some refactoring before copying the java classes for all current shellscripts or if I should leave it as is for simplicity?

@jfiala
Copy link
Contributor Author

jfiala commented Mar 12, 2017

@wing328 I added resteasy default + joda and cxf-client and did some refactorings.
Pls review and merge if it is OK, then the corresponding shellscripts could be removed in the next step.

@jfiala jfiala changed the title [Jaxrs-cxf] Java-based generator + unittest for cxf-server [Jaxrs-*] Java-based generator + unittest for cxf/resteasy/spec Mar 12, 2017
@jfiala jfiala changed the title [Jaxrs-*] Java-based generator + unittest for cxf/resteasy/spec [Jaxrs-*] Java-based generator + unittest for spec/cxf/resteasy Mar 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants