search

openservicebrokerapi / osb-checker

An automatic checker to verify an Open Service Broker API implementation against the specification
https://github.com/openservicebrokerapi/servicebroker/
Apache License 2.0
48 stars 40 forks source link

Incorrect query string in async deprovision test #57

Closed stefan-sto closed 5 years ago

stefan-sto commented 5 years ago

It looks like the query string in async deprovsion test is incorrect (two question marks): /v2/service_instances/8f020c46-c74e-54d5-f3a6-c8c25f422d6c?plan_id=0f4008b5-XXXX-XXXX-XXXX-dace631cd648&service_id=acb56d7c-XXXX-XXXX-XXXX-feb140a59a66?accepts_incomplete=false

I think that the line should be changed to use '&' instead of '?': .delete(handler + '&accepts_incomplete=false')

Also it seems like the code that invokes .send() is not needed as the service_id and the plan_id are part of the query string.

leonwanghui commented 5 years ago

Hi @stefan-sto , good catch. I think the main issue is in testAsyncParameter method, and because the handler is /v2/service_instances/' + instanceId + '?plan_id=' + queryStrings.plan_id + '&service_id=' + queryStrings.service_id, so line 627 should be changed. Is that what you think?

leonwanghui commented 5 years ago

BTW, would you like to pull a PR to solve it, or I can do that for you : )

stefan-sto commented 5 years ago

Hi @leonwanghui . I think changing the '?' to '&' on line 627 would be the easiest solution (it is also possible to use the .query method but this would require some refactoring of the code). Could you make this PR, please? I saw that it is required to sign a CLA so we may discuss in our organization to have a corporate agreement, but it will take some time. Thanks, Stefan

leonwanghui commented 5 years ago

@stefan-sto Sure, my glad : )