commit d73e3452d209852aa4db95eac38022cb1df52e85
author Noorul Islam K M <noorul@noorul.com> Fri Sep 20 06:52:48 2013 +0530
committer Ignasi Barrera <nacx@apache.org> Fri Sep 20 09:31:20 2013 +0200
tree 5791c9c37ac75395a82c70b6275aca181853efed
parent f2207bbedf15638717413702a702519c2515574f

Rename test method names to reflect what is being tested.

core/src/test/java/org/jclouds/chef/ChefApiExpectTest.java

diff --git a/core/src/test/java/org/jclouds/chef/ChefApiExpectTest.java b/core/src/test/java/org/jclouds/chef/ChefApiExpectTest.java
index 3a7bae4..c3a9372 100644
--- a/core/src/test/java/org/jclouds/chef/ChefApiExpectTest.java
+++ b/core/src/test/java/org/jclouds/chef/ChefApiExpectTest.java
@@ -53,7 +53,7 @@
                   .addHeader("Accept", MediaType.APPLICATION_JSON);
    }
 
-   public void testListClientsReturns2xx() {
+   public void testListClientsReturnsValidSet() {
       ChefApi api = requestSendsResponse(
             signed(getHttpRequestBuilder("GET", "/clients").build()),
             HttpResponse.builder().statusCode(200) //
@@ -64,7 +64,7 @@
       assertTrue(nodes.contains("adam"), String.format("Expected nodes to contain 'adam' but was: %s", nodes));
    }
 
-   public void testListClientsReturns404() {
+   public void testListClientsReturnsEmptySetOn404() {
       ChefApi api = requestSendsResponse(
             signed(getHttpRequestBuilder("GET", "/clients").build()),
             HttpResponse.builder().statusCode(404)
@@ -73,7 +73,7 @@
       assertTrue(clients.isEmpty(), String.format("Expected clients to be empty but was: %s", clients));
    }
 
-   public void testListNodesReturns2xx() {
+   public void testListNodesReturnsValidSet() {
       ChefApi api = requestSendsResponse(
             signed(getHttpRequestBuilder("GET", "/nodes").build()),
             HttpResponse.builder().statusCode(200)
@@ -84,7 +84,7 @@
       assertTrue(nodes.contains("blah"), String.format("Expected nodes to contain 'blah' but was: %s", nodes));
    }
 
-   public void testListNodesReturns404() {
+   public void testListNodesReturnsEmptySetOn404() {
       ChefApi api = requestSendsResponse(
             signed(getHttpRequestBuilder("GET", "/nodes").build()),
             HttpResponse.builder().statusCode(404).build());
@@ -92,7 +92,7 @@
       assertTrue(nodes.isEmpty(), String.format("Expected nodes to be empty but was: %s", nodes));
    }
    
-   public void testListRecipesInEnvironmentReturns2xx() {
+   public void testListRecipesInEnvironmentReturnsValidSet() {
       ChefApi api = requestSendsResponse(
             signed(getHttpRequestBuilder("GET", "/environments/dev/recipes").build()),
             HttpResponse.builder().statusCode(200)
@@ -103,7 +103,7 @@
       assertTrue(recipes.contains("apache2"), String.format("Expected recipes to contain 'apache2' but was: %s", recipes));
    }
 
-   public void testListRecipesInEnvironmentReturns404() {
+   public void testListRecipesInEnvironmentReturnsEmptySetOn404() {
       ChefApi api = requestSendsResponse(
             signed(getHttpRequestBuilder("GET", "/environments/dev/recipes").build()),
             HttpResponse.builder().statusCode(404).build());
@@ -111,7 +111,7 @@
       assertTrue(recipes.isEmpty(), String.format("Expected recipes to be empty but was: %s", recipes));
    }
 
-   public void testListNodesInEnvironmentReturns2xx() {
+   public void testListNodesInEnvironmentReturnsValidSet() {
       ChefApi api = requestSendsResponse(
             signed(getHttpRequestBuilder("GET", "/environments/dev/nodes").build()),
             HttpResponse.builder().statusCode(200)
@@ -122,7 +122,7 @@
       assertTrue(nodes.contains("blah"), String.format("Expected nodes to contain 'blah' but was: %s", nodes));
    }
 
-   public void testListNodesInEnvironmentReturns404() {
+   public void testListNodesInEnvironmentReturnsEmptySetOn404() {
       ChefApi api = requestSendsResponse(
             signed(getHttpRequestBuilder("GET", "/environments/dev/nodes").build()),
             HttpResponse.builder().statusCode(404).build());