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

[Slim4] Data mocker for scalar types #4751

Merged
merged 11 commits into from
Dec 16, 2019

Conversation

ybelenko
Copy link
Contributor

@ybelenko ybelenko commented Dec 10, 2019

PR checklist

  • Read the contribution guidelines.
  • If contributing template-only or documentation-only changes which will change sample output, build the project before.
  • Run the shell script(s) under ./bin/ (or Windows batch scripts under.\bin\windows) to update Petstore samples related to your fix. This is important, as CI jobs will verify all generator outputs of your HEAD commit, and these must match the expectations made by your contribution. You only need to run ./bin/{LANG}-petstore.sh, ./bin/openapi3/{LANG}-petstore.sh if updating the code or mustache templates for a language ({LANG}) (e.g. php, ruby, python, etc).
  • File the PR against the correct branch: master, 4.3.x, 5.0.x. Default: master.
  • Copy the technical committee to review the pull request if your PR is targeting a particular programming language.

I'm working on mock server for Slim generator. Maybe my OpenApiDataMockerInterface can be useful in other PHP generators. Right now only scalar data types implemented. array and object are coming next. Then I'll add enum option to mockString and method mockSchema to generate data from spec schema object.(will be implemented in next PRs)

Related to #3545

cc @jebentier, @dkarlovi, @mandrean, @jfastnacht, @ackintosh, @renepardon

@ybelenko ybelenko marked this pull request as ready for review December 11, 2019 00:29
@ybelenko ybelenko changed the title [Slim4] Data mocker [Slim4] Data mocker for scalar types Dec 11, 2019
@ybelenko
Copy link
Contributor Author

Right now only scalar data types implemented. array and object are coming next. Then I'll add enum option to mockString and method mockSchema to generate data from spec schema object.

I've reconsidered. Small PR will be easier to review. Let's keep this PR with scalar types only, I think it's a good starting point. I'll work on other types in local branch and submit when this will be merged.

@ybelenko ybelenko closed this Dec 11, 2019
@ybelenko ybelenko reopened this Dec 11, 2019
@ybelenko
Copy link
Contributor Author

How to test

  1. Checkout into feature branch
$ git checkout -b ybelenko-slim_data_mocker master
$ git pull https://github.com/ybelenko/openapi-generator.git slim_data_mocker
  1. Generate new Slim4 samples and go to just created build
$ mvn clean package
$ bin/php-slim4-server-petstore.sh
$ cd samples/server/petstore/php-slim4
  1. Start Composer installation and run unit tests
$ composer install
$ composer test
$ composer phpcs

I've rejected the idea to keep Composer dependencies in Java/codegen
variables at some point of time. It seems that after Git rebase I forgot
to delete them.
Copy link
Member

@wing328 wing328 left a comment

Choose a reason for hiding this comment

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

Looks good to me but I've not tested it locally.

@wing328 wing328 merged commit 7bd378c into OpenAPITools:master Dec 16, 2019
@ybelenko ybelenko deleted the slim_data_mocker branch December 16, 2019 09:39
@wing328 wing328 added this to the 4.2.3 milestone Jan 31, 2020
@ybelenko ybelenko restored the slim_data_mocker branch February 5, 2020 09:20
@ybelenko ybelenko deleted the slim_data_mocker branch May 9, 2020 07:26
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