soap-client icon indicating copy to clipboard operation
soap-client copied to clipboard

Add tests for local elements when elementFormDefault is set

Open Toilal opened this issue 5 years ago • 8 comments

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)

Toilal avatar Feb 12 '20 00:02 Toilal

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.

Toilal avatar Feb 12 '20 00:02 Toilal

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 ?

  • https://github.com/goetas-webservices/xsd2php/pull/108
  • https://github.com/goetas-webservices/xsd-reader/pull/52

I'm currently running in an app that's now live in production the following patches from my pull requests.

        "patches": {
            "goetas-webservices/soap-client": {
                "Add ID3 wsdl test for local element property": "https://github.com/goetas-webservices/soap-client/pull/46.patch",
                "Add symfony 4+ support": "https://github.com/goetas-webservices/soap-client/pull/47.patch"
            },
            "goetas-webservices/xsd2php": {
                "Use elements local property to generate proper JMS mappings": "https://github.com/goetas-webservices/xsd2php/pull/108.patch"
            },
            "goetas-webservices/xsd-reader": {
                "Add local property to elements": "https://github.com/goetas-webservices/xsd-reader/pull/52.patch"
            }
        }

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.

Toilal avatar Jun 19 '20 13:06 Toilal

Hi, sorry if this is taking so long, but recently i've been really busy.

I'm really interested in https://github.com/goetas-webservices/xsd-reader/pull/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.

goetas avatar Jun 24 '20 09:06 goetas

same note for the tests as in https://github.com/goetas-webservices/xsd2php/pull/108

goetas avatar Jun 24 '20 09:06 goetas

Test have been simplified. They fail on CI because they need to run with https://github.com/goetas-webservices/xsd2php/pull/108 and https://github.com/goetas-webservices/xsd-reader/pull/52

Toilal avatar Aug 22 '20 00:08 Toilal

Shouldnt we first merge https://github.com/goetas-webservices/xsd2php/pull/108?

To make tests pass you can add @dev in the composer deps, in this way you will get the latest version

goetas avatar Aug 22 '20 08:08 goetas

https://github.com/goetas-webservices/xsd2php/releases/tag/v0.3.12 has been tagged

goetas avatar Aug 24 '20 12:08 goetas

Thanks ! Thanks. I think we also need to release wsdl2php with this change in composer.json : https://github.com/goetas-webservices/wsdl2php/pull/12

Toilal avatar Aug 24 '20 13:08 Toilal