From 1486476d26c5dc8fb0eadacb3d656ba567846516 Mon Sep 17 00:00:00 2001
From: Jordan Halterman <jordan.halterman@gmail.com>
Date: Wed, 21 Jan 2015 00:46:39 -0800
Subject: [PATCH] Add passive members to collections test cluster.

---
 .../copycat/collections/AsyncMapTest.java     |  4 +-
 .../copycat/collections/TestCluster.java      | 52 ++++++++++++++-----
 2 files changed, 40 insertions(+), 16 deletions(-)

diff --git a/collections/src/test/java/net/kuujo/copycat/collections/AsyncMapTest.java b/collections/src/test/java/net/kuujo/copycat/collections/AsyncMapTest.java
index 899272b350..3feaeb7358 100644
--- a/collections/src/test/java/net/kuujo/copycat/collections/AsyncMapTest.java
+++ b/collections/src/test/java/net/kuujo/copycat/collections/AsyncMapTest.java
@@ -34,7 +34,7 @@ public void testAsyncMapPutGet() throws Throwable {
     TestCluster<AsyncMap<String, String>> cluster = TestCluster.of((uri, config) -> AsyncMap.create("test", uri, config, new AsyncMapConfig().withLog(new BufferedLog())));
     cluster.open().thenRun(this::resume);
     await(5000);
-    AsyncMap<String, String> map = cluster.resources().get(0);
+    AsyncMap<String, String> map = cluster.activeResources().iterator().next();
     map.put("foo", "Hello world!").thenRun(() -> {
       map.get("foo").thenAccept(result -> {
         threadAssertEquals(result, "Hello world!");
@@ -51,7 +51,7 @@ public void testAsyncMapPutRemove() throws Throwable {
     TestCluster<AsyncMap<String, String>> cluster = TestCluster.of((uri, config) -> AsyncMap.create("test", uri, config, new AsyncMapConfig().withLog(new BufferedLog())));
     cluster.open().thenRun(this::resume);
     await(5000);
-    AsyncMap<String, String> map = cluster.resources().get(0);
+    AsyncMap<String, String> map = cluster.activeResources().iterator().next();
     map.put("foo", "Hello world!").thenRun(() -> {
       map.get("foo").thenAccept(r1 -> {
         threadAssertEquals(r1, "Hello world!");
diff --git a/collections/src/test/java/net/kuujo/copycat/collections/TestCluster.java b/collections/src/test/java/net/kuujo/copycat/collections/TestCluster.java
index 8e94174a3e..22ec87c2be 100644
--- a/collections/src/test/java/net/kuujo/copycat/collections/TestCluster.java
+++ b/collections/src/test/java/net/kuujo/copycat/collections/TestCluster.java
@@ -19,6 +19,7 @@
 import net.kuujo.copycat.cluster.ClusterConfig;
 import net.kuujo.copycat.protocol.LocalProtocol;
 
+import java.util.Arrays;
 import java.util.List;
 import java.util.concurrent.CompletableFuture;
 import java.util.function.BiFunction;
@@ -30,29 +31,45 @@
  * @author <a href="http://github.com/kuujo">Jordan Halterman</a>
  */
 public class TestCluster<T extends Resource<T>> {
-  private final List<T> resources;
+  private final List<T> activeResources;
+  private final List<T> passiveResources;
 
-  private TestCluster(List<T> resources) {
-    this.resources = resources;
+  private TestCluster(List<T> activeResources, List<T> passiveResources) {
+    this.activeResources = activeResources;
+    this.passiveResources = passiveResources;
   }
 
   /**
    * Creates a test cluster for the given resource factory.
    */
+  @SuppressWarnings("all")
   public static <T extends Resource<T>> TestCluster<T> of(BiFunction<String, ClusterConfig, T> factory) {
     ClusterConfig cluster = new ClusterConfig()
       .withProtocol(new LocalProtocol());
     for (int i = 1; i <= 3; i++) {
       cluster.addMember(String.format("local://test%d", i));
     }
-    return new TestCluster<T>(cluster.getMembers().stream().collect(Collectors.mapping(uri -> factory.apply(uri, cluster), Collectors.toList())));
+    List<T> activeResources = cluster.getMembers().stream().collect(Collectors.mapping(uri -> factory.apply(uri, cluster), Collectors.toList()));
+    List<T> passiveResources = Arrays.asList("local://test4", "local://test5").stream().collect(Collectors.mapping(uri -> factory.apply(uri, cluster), Collectors.toList()));
+    return new TestCluster<T>(activeResources, passiveResources);
   }
 
   /**
-   * Returns a list of cluster resources.
+   * Returns a list of active cluster resources.
+   *
+   * @return A list of active cluster resources.
    */
-  public List<T> resources() {
-    return resources;
+  public List<T> activeResources() {
+    return activeResources;
+  }
+
+  /**
+   * Returns a list of passive cluster resources.
+   *
+   * @return A list of passive cluster resources.
+   */
+  public List<T> passiveResources() {
+    return passiveResources;
   }
 
   /**
@@ -60,10 +77,13 @@ public List<T> resources() {
    */
   @SuppressWarnings("unchecked")
   public CompletableFuture<Void> open() {
-    CompletableFuture<Void>[] futures = new CompletableFuture[resources.size()];
-    for (int i = 0; i < resources.size(); i++) {
-      T resource = resources.get(i);
-      futures[i] = resources.get(i).open().thenRun(() -> System.out.println(resource.cluster().member().uri() + " started successfully!")).thenApply(v -> null);
+    CompletableFuture<Void>[] futures = new CompletableFuture[activeResources.size() + passiveResources.size()];
+    int i = 0;
+    for (T resource : activeResources) {
+      futures[i++] = resource.open().thenRun(() -> System.out.println(resource.cluster().member().uri() + " started successfully!")).thenApply(v -> null);
+    }
+    for (T resource : passiveResources) {
+      futures[i++] = resource.open().thenRun(() -> System.out.println(resource.cluster().member().uri() + " started successfully!")).thenApply(v -> null);
     }
     return CompletableFuture.allOf(futures);
   }
@@ -73,9 +93,13 @@ public CompletableFuture<Void> open() {
    */
   @SuppressWarnings("unchecked")
   public CompletableFuture<Void> close() {
-    CompletableFuture<Void>[] futures = new CompletableFuture[resources.size()];
-    for (int i = 0; i < resources.size(); i++) {
-      futures[i] = resources.get(i).close();
+    CompletableFuture<Void>[] futures = new CompletableFuture[activeResources.size() + passiveResources.size()];
+    int i = 0;
+    for (T resource : passiveResources) {
+      futures[i++] = resource.close();
+    }
+    for (T resource : activeResources) {
+      futures[i++] = resource.close();
     }
     return CompletableFuture.allOf(futures);
   }