227
68%
master: 69%

Ran 21 Aug 2019 08:28PM UTC

Jobs 2

Files 155

Run time 23min

Badge

Embed ▾

pending completion

Build # 227

Build Type

push

travis-ci-com

Committed by

argouber

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

7662 of 11327 relevant lines covered (67.64%)

1.35 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	227.1	21 Aug 2019 08:28PM UTC	0	67.64	Travis Job 227.1
2	227.2	21 Aug 2019 08:52PM UTC	0	67.64	Travis Job 227.2

Source Files on build 227

Detailed source file information is not available for this build.

uber / zanzibar / 227
68%
master: 69%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 227

uber / zanzibar / 227 68% master: 69%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 227

uber / zanzibar / 227
68%
master: 69%

README BADGES
x