-
-
Notifications
You must be signed in to change notification settings - Fork 28
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
Add tests for local elements when elementFormDefault is set #46
base: master
Are you sure you want to change the base?
Conversation
806f479
to
c78b78d
Compare
The test is fixed with both goetas-webservices/xsd2php#108 and goetas-webservices/xsd-reader#52. I have used https://github.com/gossi/composer-localdev-plugin to symlink to local development packages, works like a charm. Please wait before merging those pull request, as I wish to add more test cases based on this ID3 Webservice, and try integration in my real world app. |
c78b78d
to
f277e28
Compare
Hi @goetas I pretty sure you are really busy on many other subjects, but you could review and merge this pull requests and others related to this one ?
I'm currently running in an app that's now live in production the following patches from my pull requests.
Everything is working properly with those patches, i'm pretty sure you could merge those pull requests and perform new release with confidence :) Please let me know, best regards. |
Hi, sorry if this is taking so long, but recently i've been really busy. I'm really interested in goetas-webservices/xsd-reader#52 (and in the scenario that fixes) but I would like to have time to test it a bit better (I tend to follow the SOAP interpretation done by SoapUI... so would like to have some time to test it with it). I think that the quality of both pull requests is very good, is just that i would like to find time to test them properly. |
same note for the tests as in goetas-webservices/xsd2php#108 |
7ca2a9d
to
e31a4d4
Compare
Test have been simplified. They fail on CI because they need to run with goetas-webservices/xsd2php#108 and goetas-webservices/xsd-reader#52 |
Shouldnt we first merge goetas-webservices/xsd2php#108? To make tests pass you can add |
e31a4d4
to
aadcbde
Compare
aadcbde
to
8fa3f8c
Compare
Thanks ! Thanks. I think we also need to release wsdl2php with this change in composer.json : goetas-webservices/wsdl2php#12 |
Related to goetas-webservices/xsd-reader#51
This currently adds a test in soap-client matching the issue. (and test is actually failing to demonstrate the issue)