1
68%
master: 69%

Ran 21 Aug 2019 05:43PM UTC

Files 155

Run time 11s

Badge

Embed ▾

Committed 21 Aug 2019 06:57AM UTC coverage: 67.643%. First build

Job # 225.1

Build Type

push

travis-ci-com

Committed by

web-flow

Commit Message

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

Run Details

7664 of 11330 relevant lines covered (67.64%)

0.68 hits per line

uber / zanzibar / 225 / 1
68%
master: 69%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 225.1

uber / zanzibar / 225 / 1 68% master: 69%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 225.1

uber / zanzibar / 225 / 1
68%
master: 69%

README BADGES
x