uber / zanzibar / 223

69%

push

travis-ci-com

Argo/query params for all methods (#625)

Allow query params on ALL http methods

Nothing in the standards definitions preclude query params for methods other than GET
We have use-cases where an endpoint wanted to use DELETE with query params and that was used by the downstream service.

With this change, a few things:
 - Allowed other methods to process query params
 - Fixed the test-cases -- all query params need to have the http.ref = "query.xxx" annotation
   In the test cases, we were grandfathering non-annotated fields in the struct as query params since these were only for GET and it worked well
 - One test case added for delete with query params

51 of 51 new or added lines in 2 files covered. (100.0%)

7664 of 11330 relevant lines covered (67.64%)

1.35 hits per line