Skip to content

Conversation

viclovsky
Copy link
Contributor

@viclovsky viclovsky commented Apr 20, 2018

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 and ./bin/security/{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\.
  • Filed the PR against the correct branch: 3.0.0 branch for changes related to OpenAPI spec 3.0. Default: master.
  • Copied the technical committee to review the pull request if your PR is targeting a particular programming language.

Description of the PR

Hello, guys
As you know new rest-assured just has been released. New feature "Type instead of Class for mapping" was added in version 3.1.0. Here is the link: rest-assured/rest-assured#980. My aim is to use the feature in Rest-assured client.
So, the PR: updated rest-assured from 3.0.6 up to 3.1.0. Additional mapper was deleted for ApiClient& Now you are able to customise mapper via Rest-assured config easier than it was before.

Just want to notice that your test server (Petstore) is now returning 500 for some requests. For instance: curl -X GET "http://petstore.swagger.io/v2/store/inventory" -H "accept: application/json"

@wing328 wing328 added this to the v2.4.0 milestone Apr 21, 2018
@wing328 wing328 merged commit 7de4952 into swagger-api:master Apr 21, 2018
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