Add client test case for list<?> query parameters

palantir / conjure-verification

A testing tool that ensures all Conjure clients and servers adhere to a common wire spec

Apache License 2.0

9 stars 6 forks source link

Add client test case for list<?> query parameters #170

Open uschi2000 opened 5 years ago

uschi2000 commented 5 years ago

SingleQueryParamService tests for primitives, enums, and optional<?>, but not for list<?>. According to https://palantir.github.io/conjure/#/docs/spec/conjure_definitions?id=argumentdefinition , this should be supported, and thus tested. Note that jaxrs clients today serialize this as expected: /path?paramName=value1&paramName=value2&paramName=value3

uschi2000 commented 5 years ago

Cf https://github.com/palantir/conjure-java-runtime/pull/1081