>        Set<String> nodes = api.listNodesInEnvironment("dev");
>        assertTrue(nodes.isEmpty(), String.format("Expected nodes to be empty 
> but was: %s", nodes));
>     }
>  
> +   public void testListCookbooksInEnvironmentReturnsEmptySetOn404() {
> +      ChefApi api = requestSendsResponse(
> +            signed(getHttpRequestBuilder("GET", 
> "/environments/dev/cookbooks").build()),
> +            HttpResponse.builder().statusCode(404).build());
> +      Set<CookbookDefinition> cookbooks = 
> api.listCookbooksInEnvironment("dev");
> +      assertTrue(cookbooks.isEmpty(), String.format("Expected cookbooks to 
> be empty but was: %s", cookbooks));
> +   }
> +
> +   public void 
> testListCookbooksInEnvironmentWithNumVersionReturnsEmptySetOn404() {
> +      ChefApi api = requestSendsResponse(
> +            signed(getHttpRequestBuilder("GET", 
> "/environments/dev/cookbooks?num_versions=2").build()),

Minor cosmetic thing: use better the `addQueryParam` method.

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

Reply via email to