> @@ -75,4 +78,20 @@ public void testKeywordStatic() {
>        ListUsageRecordsOptions options = keyword("bob");
>        assertEquals(ImmutableSet.of("bob"), 
> options.buildQueryParameters().get("keyword"));
>     }
> +   
> +       public void testTypeStatic() {
> +        ListUsageRecordsOptions options = type("3");
> +        assertEquals(ImmutableSet.of("3"), 
> options.buildQueryParameters().get("type"));
> +    }
> +
> +    public void testPageStatic() {
> +        ListUsageRecordsOptions options = page("1");
> +        assertEquals(ImmutableSet.of("1"), 
> options.buildQueryParameters().get("page"));
> +    }
> +
> +    public void testPageSizeStatic() {
> +        ListUsageRecordsOptions options = pageSize("500");
> +        assertEquals(ImmutableSet.of("500"), 
> options.buildQueryParameters().get("pagesize"));
> +    }

Sorry for the late response.

Yes, these options / parameters are used when providing options in the request 
message. Here's a link to the API: 
http://cloudstack.apache.org/docs/api/apidocs-4.0.0/root_admin/listUsageRecords.html.

These test were added similar to the existing options - account Id, domain Id, 
keyword, etc. I believe the purpose of these tests is to make sure that the 
request options / parameters are being set correctly. 

I looked and I couldn't find an ExpectTest that this might fit under. Perhaps 
we can add those options to one of the existing tests in 
GlobalUsageApiTest.java?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/124/files#r6368831

Reply via email to