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

Propose osb-checker go-based test framework #77

Merged
merged 30 commits into from
Apr 29, 2019
Merged

Propose osb-checker go-based test framework #77

merged 30 commits into from
Apr 29, 2019

Conversation

leonwanghui
Copy link
Collaborator

@leonwanghui leonwanghui commented Mar 25, 2019

Hi all, here is the initial version of test framework, though it's not finished yet, but users can just try to test it in their development pipelines.

Design

For design details, please refer to here

Test

  • Firstly, u need to deploy your own service broker to be tested.
  • Modify the config_mock.yaml under test/configs folder.
  • Just run go test -v ./test/ to start the test job.

Generate model

All the model will be generated from swagger.yaml automatically. Here are some steps for developers to generate the model:

  • First developers should download go-swagger tool from here
  • Then run these command below:
cd ./autogenerated && wget https://raw.githubusercontent.com/openservicebrokerapi/servicebroker/master/swagger.yaml
go-swagger generate model ./swagger.yaml

@leonwanghui leonwanghui self-assigned this Mar 25, 2019
@cfdreddbot
Copy link

✅ Hey leonwanghui! The commit authors and yourself have already signed the CLA.

Signed-off-by: leonwanghui <wanghui71leon@gmail.com>
Signed-off-by: leonwanghui <wanghui71leon@gmail.com>
@leonwanghui
Copy link
Collaborator Author

@fmui @mattmcneeney Any comments or concerns on this PR?

@mattmcneeney
Copy link
Contributor

I haven't actually tried it and can't review the code, but happy for you to make the call if others are happy!

@leonwanghui leonwanghui merged commit 26c6e93 into openservicebrokerapi:dev Apr 29, 2019
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.

None yet

3 participants