> + public void testDelete() throws Exception {
> + URI uri = api().delete(resourcegroup);
> + assertNotNull(uri);
> + assertTrue(uri.toString().contains("api-version"));
> + assertTrue(uri.toString().contains("operationresults"));
> +
> + String[] params = uri.getQuery().split("&");
> + String apiversion = "";
> + for (String param : params)
> + {
> + String name = param.split("=")[0];
> + if (name.equals("api-version")){
> + apiversion = param.split("=")[1];
> + break;
> + }
> + }
Looking at this code, it looks like the returned uri in the Location header
already has the api-version query parameter. With my suggestion of annotating
the parameter with `@EndpointParam`, the entire URI will be used as the
endpoint. Would that eliminate the need to parse the query parameters too? And
in that case, could the "jobStatus" method take only the URI parameter, as the
api version one would no longer be used?
---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/250/files/8d7d68bf33361d7f5df68c7e7232b03116ed6977#r58624217
