-
-
Notifications
You must be signed in to change notification settings - Fork 78
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
Operations extends is broken in 1.1.1 #145
Comments
Can you post your faling test? |
Sorry can't post my code as is, and technically my unit tests are more integration tests as I connect to a live API (test server). Basically I have the above operations and use the following model: $models['result' => [
'type' => 'object',
'additionalProperties' => [
'location' => 'json'
]
]
]; I use a And when I do If I have some time I can try and provide some simple test code to reproduce this. The issue could still be linked to my external API but since the behavior clearly changes between the two versions I kind of doubt it. |
Ok I had time to write a quick test. I forked the project and added a test which works with tag 1.1.0 but no longer in master. See commit here: https://github.com/delamart/guzzle-services/commit/2d50fa4fb9f6ac4f65a6271f7d7079be91b396aa |
@delamart Thanks for your test. This helped to find the culprit. |
Great, thanks I upgraded to 1.1.2 and my lib works again. |
I just upgraded to version 1.1.1 and my unit tests stopped working because a parameter is no longer being set in my commands. Version 1.1.0 works fine.
I used the extends option like this:
The parameter
format=json
was being set correctly on all commands but now I have to set it manually in the command if I want it to work.The text was updated successfully, but these errors were encountered: