Comment 13 for bug 831553

Revision history for this message
jsgaarde (jakob-simon-gaarde) wrote :

Hi George.

I do testing like 80% of the time while developing. Unfortunately I don't have time to update the automated testing framework all the time, cause setting up specific tests take time too, I do save these tests on my local system and hope to add them to the unit testing framework at some point. This is an open source project done in my free time, so some testing is up to the those users who choose to use it.

You are right about the SOAP request handler missing the '_' to '-' conversion - it was there before I re-wrote the whole request parser - thanks. I missed that test, besides that it has been pretty thouroghly tested:
* Unbound arguments of primitives
* Arguments of Complex Types
* Arguments with nested Complex Types
* Arguments with nested array based Complex Types
* Arguments with primitive arrays
* and more

The missing variable you found is actually not missing - something must have gone wrong when you merged. Take a look at the trunk repository and you will see that the fix_path = urlparse(client_path) (line) you found missing is actually in the trunk version at the time you merged:
http://bazaar.launchpad.net/~ladon-dev-team/ladon/ladon/view/head:/frameworks/python/src/ladon/server/wsgi_application.py#L190

Have you found more bugs or what do you mean by "all over the place"?

Best regards
Jakob Simon-Gaarde