jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ignasi Barrera <notificati...@github.com>
Subject Re: [jclouds] JCLOUDS-1008: Add @Encoded annotation. (#861)
Date Thu, 01 Oct 2015 21:31:23 GMT
> @@ -1543,6 +1549,14 @@ public void testPathParamExtractor() throws Exception {
>     }
>  
>     @Test
> +   public void testPathParamEncoding() throws Exception {
> +      Invokable<?, ?> method = method(TestPath.class, "onePathEncodedParam", String.class);
> +      GeneratedHttpRequest request = processor.apply(Invocation.create(method,
> +            ImmutableList.<Object> of("foo/bar")));
> +      assertRequestLineEquals(request, "GET http://localhost:9999/foo%2Fbar HTTP/1.1");

Use a more complex path in the test to also verify the fact that the entire path is not encoded.
We need to verify that in the tests too.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/861/files#r40968986
Mime
View raw message