Skip to content
New issue

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

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

Already on GitHub? # to your account

Add auto-generated tests and travis config to Dart clients #2909

Merged
merged 4 commits into from
May 25, 2019
Merged

Conversation

wing328
Copy link
Member

@wing328 wing328 commented May 16, 2019

PR checklist

  • Read the contribution guidelines.
  • Ran the shell script under ./bin/ to update Petstore sample so that CIs can verify the change. (For instance, only need to run ./bin/{LANG}-petstore.sh, ./bin/openapi3/{LANG}-petstore.sh if updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates). Windows batch files can be found in .\bin\windows\. If contributing template-only or documentation-only changes which will change sample output, be sure to build the project first.
  • Filed the PR against the correct branch: master, 4.1.x, 5.0.x. Default: master.
  • Copied the technical committee to review the pull request if your PR is targeting a particular programming language.

Description of the PR

Add auto-generated tests and travis config to Dart clients

cc @ircecho (2017/07) @swipesight (2018/09) @jaumard (2018/09)

@wing328 wing328 added this to the 4.0.1 milestone May 16, 2019
@wing328
Copy link
Member Author

wing328 commented May 16, 2019

➜  openapi git:(dart-travis) pub run test
00:06 +47: All tests passed!

Tests passed locally

@jaumard
Copy link
Contributor

jaumard commented May 25, 2019

LGTM just not sure how it will affect jaguar generator as it extends DartClientCodegen.java

@wing328
Copy link
Member Author

wing328 commented May 25, 2019

LGTM just not sure how it will affect jaguar generator as it extends DartClientCodegen.java

I'll need to add it separately. Let's actually do it in this PR as well. Stay tuned.

Copy link
Contributor

@jaumard jaumard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wing328
Copy link
Member Author

wing328 commented May 25, 2019

Actually let's merge this one first to see if the Shippable CI issue disappears

@wing328
Copy link
Member Author

wing328 commented May 25, 2019

Still broken after merging into master. Reverted via #3001

Will file a new PR including the auto-generated test for Dart Jaguar client as well.

@wing328 wing328 removed the WIP Work in Progress label May 25, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants