You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jclouds.apache.org by na...@apache.org on 2014/10/10 15:13:42 UTC

[16/50] [abbrv] git commit: Rename test method names to reflect what is being tested.

Rename test method names to reflect what is being tested.


Project: http://git-wip-us.apache.org/repos/asf/jclouds/repo
Commit: http://git-wip-us.apache.org/repos/asf/jclouds/commit/d9a79366
Tree: http://git-wip-us.apache.org/repos/asf/jclouds/tree/d9a79366
Diff: http://git-wip-us.apache.org/repos/asf/jclouds/diff/d9a79366

Branch: refs/heads/master
Commit: d9a79366faa84192f162fcb0a20ec65949bf05ba
Parents: 61258a6
Author: Noorul Islam K M <no...@noorul.com>
Authored: Fri Sep 20 06:52:48 2013 +0530
Committer: Ignasi Barrera <na...@apache.org>
Committed: Fri Sep 20 09:31:20 2013 +0200

----------------------------------------------------------------------
 .../java/org/jclouds/chef/ChefApiExpectTest.java    | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/jclouds/blob/d9a79366/apis/chef/src/test/java/org/jclouds/chef/ChefApiExpectTest.java
----------------------------------------------------------------------
diff --git a/apis/chef/src/test/java/org/jclouds/chef/ChefApiExpectTest.java b/apis/chef/src/test/java/org/jclouds/chef/ChefApiExpectTest.java
index 3a7bae4..c3a9372 100644
--- a/apis/chef/src/test/java/org/jclouds/chef/ChefApiExpectTest.java
+++ b/apis/chef/src/test/java/org/jclouds/chef/ChefApiExpectTest.java
@@ -53,7 +53,7 @@ public class ChefApiExpectTest extends BaseChefApiExpectTest<ChefApi> {
                   .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 @@ public class ChefApiExpectTest extends BaseChefApiExpectTest<ChefApi> {
       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 @@ public class ChefApiExpectTest extends BaseChefApiExpectTest<ChefApi> {
       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 @@ public class ChefApiExpectTest extends BaseChefApiExpectTest<ChefApi> {
       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 @@ public class ChefApiExpectTest extends BaseChefApiExpectTest<ChefApi> {
       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 @@ public class ChefApiExpectTest extends BaseChefApiExpectTest<ChefApi> {
       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 @@ public class ChefApiExpectTest extends BaseChefApiExpectTest<ChefApi> {
       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 @@ public class ChefApiExpectTest extends BaseChefApiExpectTest<ChefApi> {
       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());