Test: RelocationTests.testMoveShardsWhileRelocation should wait on nodes to start before asking for a client

The client call may create another node if none of the previous nodes are published. That node throws the test off.

Author: bleskes

src/test/java/org/elasticsearch/recovery/RelocationTests.java

@@ -337,14 +337,14 @@ public void testMoveShardsWhileRelocation() throws Exception {
         final String indexName = "test";
 
         ListenableFuture<String> blueFuture = internalCluster().startNodeAsync(ImmutableSettings.builder().put("node.color", "blue").build());
-        internalCluster().startNodeAsync(ImmutableSettings.builder().put("node.color", "green").build());
         ListenableFuture<String> redFuture = internalCluster().startNodeAsync(ImmutableSettings.builder().put("node.color", "red").build());
+        internalCluster().startNode(ImmutableSettings.builder().put("node.color", "green").build());
+        final String blueNodeName = blueFuture.get();
+        final String redNodeName = redFuture.get();
 
         ClusterHealthResponse response = client().admin().cluster().prepareHealth().setWaitForNodes(">=3").get();
         assertThat(response.isTimedOut(), is(false));
 
-        String blueNodeName = blueFuture.get();
-        final String redNodeName = redFuture.get();
 
         client().admin().indices().prepareCreate(indexName)
                 .setSettings(