CLIENTS-1063: Added equals() and hashCode in StoreIndex.

Author: christopherfrieler

src/main/java/com/basho/riak/client/api/commands/search/StoreIndex.java

@@ -3,6 +3,7 @@
 import com.basho.riak.client.api.AsIsRiakCommand;
 import com.basho.riak.client.core.operations.YzPutIndexOperation;
 import com.basho.riak.client.core.query.search.YokozunaIndex;
+import java.util.Objects;
 
 /**
  * Command used to store a search index in Riak.
@@ -36,6 +37,24 @@ protected YzPutIndexOperation buildCoreOperation()
         return opBuilder.build();
     }
 
+    @Override
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (!(other instanceof StoreIndex)) {
+            return false;
+        }
+        Builder otherStoreIndex = ((StoreIndex) other).cmdBuilder;
+        return Objects.equals(cmdBuilder.index, otherStoreIndex.index)
+                && Objects.equals(cmdBuilder.timeout, otherStoreIndex.timeout);
+    }
+
+    @Override
+    public int hashCode() {
+        return Objects.hash(cmdBuilder.index, cmdBuilder.timeout);
+    }
+
     /**
      * Builder for a StoreIndex command.
      */

src/test/java/com/basho/riak/client/api/commands/search/StoreIndexTest.java

@@ -0,0 +1,34 @@
+package com.basho.riak.client.api.commands.search;
+
+import com.basho.riak.client.core.query.search.YokozunaIndex;
+import org.junit.Test;
+import static org.hamcrest.CoreMatchers.equalTo;
+import static org.hamcrest.CoreMatchers.is;
+import static org.hamcrest.CoreMatchers.not;
+import static org.hamcrest.MatcherAssert.assertThat;
+
+public class StoreIndexTest {
+	@Test
+	public void equalsReturnsTrueForEqualIndexAndTimeout() {
+		YokozunaIndex index1 = new YokozunaIndex("index");
+		YokozunaIndex index2 = new YokozunaIndex("index");
+
+		StoreIndex storeIndex1 = new StoreIndex.Builder(index1).withTimeout(5).build();
+		StoreIndex storeIndex2 = new StoreIndex.Builder(index2).withTimeout(5).build();
+
+		assertThat(storeIndex1, is(equalTo(storeIndex2)));
+		assertThat(storeIndex2, is(equalTo(storeIndex1)));
+	}
+
+	@Test
+	public void equalsReturnsFalseForDifferentIndexAndTimeout() {
+		YokozunaIndex index1 = new YokozunaIndex("index1");
+		YokozunaIndex index2 = new YokozunaIndex("index2");
+
+		StoreIndex storeIndex1 = new StoreIndex.Builder(index1).withTimeout(5).build();
+		StoreIndex storeIndex2 = new StoreIndex.Builder(index2).withTimeout(8).build();
+
+		assertThat(storeIndex1, is(not(equalTo(storeIndex2))));
+		assertThat(storeIndex2, is(not(equalTo(storeIndex1))));
+	}
+}