Merge pull request #115 from JHUAPL/Issue113

Issue113

changelog

@@ -1,3 +1,6 @@
+[2.2]
+ -- 113 Applied instance name when mock instance is used
+ -- 114 Made map reduce elements serialiable
 [2.1]
 Change Log started
  --109,103 Merged Metadata tables

src/main/java/edu/jhuapl/tinkerpop/AccumuloGraphConfiguration.java

@@ -256,6 +256,9 @@ implements Serializable {
       setPassword("");
       setCreate(true);
     }
+    if(InstanceType.Mock.equals(type)){
+      setInstanceName("mock-instance");
+    }
 
     return this;
   }

src/main/java/edu/jhuapl/tinkerpop/mapreduce/MapReduceEdge.java

@@ -14,6 +14,8 @@
  */
 package edu.jhuapl.tinkerpop.mapreduce;
 
+import java.io.Serializable;
+
 import com.tinkerpop.blueprints.Direction;
 import com.tinkerpop.blueprints.Edge;
 import com.tinkerpop.blueprints.Vertex;
@@ -21,7 +23,7 @@ import com.tinkerpop.blueprints.util.ExceptionFactory;
 
 import edu.jhuapl.tinkerpop.AccumuloGraph;
 
-public class MapReduceEdge extends MapReduceElement implements Edge {
+public class MapReduceEdge extends MapReduceElement implements Edge, Serializable {
 
   String sourceId;
   String label;
@@ -30,6 +32,8 @@ public class MapReduceEdge extends MapReduceElement implements Edge {
   MapReduceEdge(AccumuloGraph parent) {
     super(parent);
   }
+  
+  MapReduceEdge() { super(); }
 
   void setSourceId(String id) {
     sourceId = id;

src/main/java/edu/jhuapl/tinkerpop/mapreduce/MapReduceElement.java

@@ -42,7 +42,11 @@ public abstract class MapReduceElement implements Element, WritableComparable<Ma
   AccumuloGraph parent;
 
   MapReduceElement(AccumuloGraph parent) {
+    this();
     this.parent = parent;
+  }
+  
+  MapReduceElement(){
     properties = new HashMap<String,Object>();
     newProperties = new HashMap<String,Object>();
   }

src/main/java/edu/jhuapl/tinkerpop/mapreduce/MapReduceVertex.java

@@ -17,6 +17,7 @@ package edu.jhuapl.tinkerpop.mapreduce;
 import java.io.DataInput;
 import java.io.DataOutput;
 import java.io.IOException;
+import java.io.Serializable;
 import java.util.Arrays;
 import java.util.Collection;
 import java.util.HashSet;
@@ -33,7 +34,7 @@ import com.tinkerpop.blueprints.util.DefaultVertexQuery;
 
 import edu.jhuapl.tinkerpop.AccumuloGraph;
 
-public class MapReduceVertex extends MapReduceElement implements Vertex {
+public class MapReduceVertex extends MapReduceElement implements Vertex, Serializable {
 
   List<Edge> inEdges;
   List<Edge> outEdges;
@@ -43,6 +44,12 @@ public class MapReduceVertex extends MapReduceElement implements Vertex {
     inEdges = new LinkedList<Edge>();
     outEdges = new LinkedList<Edge>();
   }
+  
+  MapReduceVertex(){
+    super();
+    inEdges = new LinkedList<Edge>();
+    outEdges = new LinkedList<Edge>();
+  }
 
   void prepareEdge(String id, String src, String label, String dest) {
     MapReduceEdge mre = new MapReduceEdge(parent, id, src, label, dest);

src/test/java/edu/jhuapl/tinkerpop/AccumuloGraphConfigurationTest.java

@@ -1,296 +1,309 @@
-/* Copyright 2014 The Johns Hopkins University Applied Physics Laboratory
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-package edu.jhuapl.tinkerpop;
-
-import static org.junit.Assert.*;
-
-import java.util.ArrayList;
-import java.util.Collection;
-import java.util.List;
-
-import javax.xml.namespace.QName;
-
-import org.apache.commons.configuration.Configuration;
-import org.apache.hadoop.io.Text;
-import org.junit.Test;
-
-import com.tinkerpop.blueprints.GraphFactory;
-import com.tinkerpop.blueprints.Vertex;
-
-import edu.jhuapl.tinkerpop.AccumuloGraphConfiguration.InstanceType;
-
-public class AccumuloGraphConfigurationTest {
-
-  @Test
-  public void testConfigurationInterface() throws Exception {
-    Configuration conf = AccumuloGraphTestUtils.generateGraphConfig("setPropsValid");
-    for (String key : AccumuloGraphConfiguration.getValidInternalKeys()) {
-      // This is bad... but we should allow them if they are valid keys.
-      conf.setProperty(key, "value");
-    }
-
-    conf = AccumuloGraphTestUtils.generateGraphConfig("setPropsInvalid");
-    try {
-      conf.setProperty("invalidKey", "value");
-      fail();
-    } catch (Exception e) { }
-  }
-
-  @Test
-  public void testSplits() throws Exception {
-    AccumuloGraphConfiguration cfg;
-
-    // Tests for splits string.
-    cfg = AccumuloGraphTestUtils.generateGraphConfig("nullSplits").setSplits((String) null);
-    AccumuloGraph graph = (AccumuloGraph) GraphFactory.open(cfg.getConfiguration());
-    for (String table : cfg.getTableNames()) {
-      assertEquals(0, cfg.getConnector().tableOperations().listSplits(table).size());
-    }
-    graph.shutdown();
-
-    cfg = AccumuloGraphTestUtils.generateGraphConfig("emptySplits").setSplits("");
-    graph = (AccumuloGraph) GraphFactory.open(cfg.getConfiguration());
-    for (String table : cfg.getTableNames()) {
-      assertEquals(0, cfg.getConnector().tableOperations().listSplits(table).size());
-    }
-    graph.shutdown();
-
-    cfg = AccumuloGraphTestUtils.generateGraphConfig("threeSplits").setSplits(" a b c ");
-    graph = (AccumuloGraph) GraphFactory.open(cfg.getConfiguration());
-    for (String table : cfg.getTableNames()) {
-      Collection<Text> splits = cfg.getConnector().tableOperations().listSplits(table);
-      assertEquals(3, splits.size());
-      List<Text> arr = new ArrayList<Text>(splits);
-      assertEquals("a", arr.get(0).toString());
-      assertEquals("b", arr.get(1).toString());
-      assertEquals("c", arr.get(2).toString());
-    }
-    graph.shutdown();
-
-    // Tests for splits array.
-    cfg = AccumuloGraphTestUtils.generateGraphConfig("nullSplitsArray").setSplits((String[]) null);
-    graph = (AccumuloGraph) GraphFactory.open(cfg.getConfiguration());
-    for (String table : cfg.getTableNames()) {
-      assertEquals(0, cfg.getConnector().tableOperations().listSplits(table).size());
-    }
-    graph.shutdown();
-
-    cfg = AccumuloGraphTestUtils.generateGraphConfig("emptySplitsArray").setSplits(new String[] {});
-    graph = (AccumuloGraph) GraphFactory.open(cfg.getConfiguration());
-    for (String table : cfg.getTableNames()) {
-      assertEquals(0, cfg.getConnector().tableOperations().listSplits(table).size());
-    }
-    graph.shutdown();
-
-    cfg = AccumuloGraphTestUtils.generateGraphConfig("threeSplitsArray").setSplits(new String[] {"d", "e", "f"});
-    graph = (AccumuloGraph) GraphFactory.open(cfg.getConfiguration());
-    for (String table : cfg.getTableNames()) {
-      Collection<Text> splits = cfg.getConnector().tableOperations().listSplits(table);
-      assertEquals(3, splits.size());
-      List<Text> arr = new ArrayList<Text>(splits);
-      assertEquals("d", arr.get(0).toString());
-      assertEquals("e", arr.get(1).toString());
-      assertEquals("f", arr.get(2).toString());
-    }
-    graph.shutdown();
-  }
-
-  @Test
-  public void testPropertyValues() throws Exception {
-    AccumuloGraph graph = new AccumuloGraph(AccumuloGraphTestUtils.generateGraphConfig("propertyValues"));
-    // Tests for serialization/deserialization of properties.
-    QName qname = new QName("ns", "prop");
-    Vertex v = graph.addVertex(null);
-    v.setProperty("qname", qname);
-    assertTrue(v.getProperty("qname") instanceof QName);
-    assertTrue(qname.equals(v.getProperty("qname")));
-  }
-
-  @Test
-  public void testIsEmpty() throws Exception {
-    AccumuloGraphConfiguration cfg = AccumuloGraphTestUtils.generateGraphConfig("isEmpty");
-    AccumuloGraph graph = new AccumuloGraph(cfg);
-    assertTrue(graph.isEmpty());
-
-    graph.addVertex("A");
-    assertFalse(graph.isEmpty());
-
-    graph.clear();
-    assertTrue(graph.isEmpty());
-  }
-
-  @Test
-  public void testCreateAndClear() throws Exception {
-    AccumuloGraphConfiguration cfg = AccumuloGraphTestUtils.generateGraphConfig("noCreate").setCreate(false);
-    try {
-      new AccumuloGraph(cfg);
-      fail("Create is disabled and graph does not exist");
-    } catch (Exception e) {
-      assertTrue(true);
-    }
-
-    cfg = AccumuloGraphTestUtils.generateGraphConfig("yesCreate").setCreate(true);
-    for (String t : cfg.getTableNames()) {
-      assertFalse(cfg.getConnector().tableOperations().exists(t));
-    }
-    AccumuloGraph graph = new AccumuloGraph(cfg);
-    for (String t : cfg.getTableNames()) {
-      assertTrue(cfg.getConnector().tableOperations().exists(t));
-    }
-    graph.shutdown();
-
-    graph = new AccumuloGraph(cfg.clone().setCreate(false));
-    assertTrue(graph.isEmpty());
-    graph.addVertex("A");
-    graph.addVertex("B");
-    assertFalse(graph.isEmpty());
-    graph.shutdown();
-
-    graph = new AccumuloGraph(cfg.clone().setClear(true));
-    assertTrue(graph.isEmpty());
-    graph.shutdown();
-  }
-
-  @Test
-  public void testPrint() throws Exception {
-    AccumuloGraphConfiguration cfg =
-        AccumuloGraphTestUtils.generateGraphConfig("printTest");
-    cfg.print();
-  }
-
-  @Test
-  public void testInvalidCacheParams() throws Exception {
-    int size = 100;
-    int timeout = 30000;
-
-    AccumuloGraphConfiguration cfg = AccumuloGraphTestUtils
-        .generateGraphConfig("cacheParams");
-    cfg.validate();
-
-
-    // Vertex cache.
-
-    assertFalse(cfg.getVertexCacheEnabled());
-
-    try {
-      cfg.setVertexCacheParams(-1, timeout);
-      fail();
-    } catch (Exception e) { }
-
-    try {
-      cfg.setVertexCacheParams(size, -1);
-      fail();
-    } catch (Exception e) { }
-
-    assertFalse(cfg.getVertexCacheEnabled());
-
-    cfg.setVertexCacheParams(size, timeout);
-    cfg.validate();
-    assertTrue(cfg.getVertexCacheEnabled());
-    assertEquals(size, cfg.getVertexCacheSize());
-    assertEquals(timeout, cfg.getVertexCacheTimeout());
-
-    cfg.setVertexCacheParams(-1, -1);
-    cfg.validate();
-    assertFalse(cfg.getVertexCacheEnabled());
-
-
-    // Edge cache.
-
-    assertFalse(cfg.getEdgeCacheEnabled());
-
-    try {
-      cfg.setEdgeCacheParams(-1, timeout);
-      fail();
-    } catch (Exception e) { }
-
-    try {
-      cfg.setEdgeCacheParams(size, -1);
-      fail();
-    } catch (Exception e) { }
-
-    assertFalse(cfg.getEdgeCacheEnabled());
-
-    cfg.setEdgeCacheParams(size, timeout);
-    cfg.validate();
-    assertTrue(cfg.getEdgeCacheEnabled());
-    assertEquals(size, cfg.getEdgeCacheSize());
-    assertEquals(timeout, cfg.getEdgeCacheTimeout());
-
-    cfg.setEdgeCacheParams(-1, -1);
-    cfg.validate();
-    assertFalse(cfg.getEdgeCacheEnabled());
-  }
-
-  /**
-   * Test different kinds of graph names (hyphens, punctuation, etc).
-   * @throws Exception
-   */
-  @Test
-  public void testGraphNames() throws Exception {
-    AccumuloGraphConfiguration conf = new AccumuloGraphConfiguration();
-
-    String[] valid = new String[] {
-        "alpha", "12345", "alnum12345",
-        "12345alnum", "under_score1", "_under_score_2"};
-    String[] invalid = new String[] {"hyph-en",
-        "dot..s", "quo\"tes"};
-
-    for (String name : valid) {
-      conf.setGraphName(name);
-    }
-
-    for (String name : invalid) {
-      try {
-        conf.setGraphName(name);
-        fail();
-      } catch (Exception e) { }
-    }
-  }
-
-  @Test
-  public void testPreloadedProperties() {
-    // Don't allow "all" and "some" preloaded properties.
-    AccumuloGraphConfiguration conf = new AccumuloGraphConfiguration();
-    conf.setPreloadAllProperties(true);
-    conf.setPreloadedProperties(new String[]{"one", "two", "three"});
-    try {
-      conf.validate();
-      fail();
-    } catch (Exception e) { }
-  }
-
-  @Test
-  public void testImmutableConnector() throws Exception {
-    AccumuloGraphConfiguration cfg = new AccumuloGraphConfiguration().setInstanceType(
-        InstanceType.Mock).setGraphName("immutableConnector")
-        .setCreate(true).setAutoFlush(false);
-
-    cfg.getConnector();
-
-    try {
-      cfg.setCreate(false);
-      fail();
-    } catch (Exception e) { }
-
-    try {
-      cfg.setAutoFlush(true);
-      fail();
-    } catch (Exception e) { }
-
-    assertTrue(cfg.getCreate());
-    assertFalse(cfg.getAutoFlush());
-  }
-}
+/* Copyright 2014 The Johns Hopkins University Applied Physics Laboratory
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package edu.jhuapl.tinkerpop;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertTrue;
+import static org.junit.Assert.fail;
+
+import java.util.ArrayList;
+import java.util.Collection;
+import java.util.List;
+
+import javax.xml.namespace.QName;
+
+import org.apache.accumulo.core.client.Instance;
+import org.apache.accumulo.core.client.mock.MockInstance;
+import org.apache.commons.configuration.Configuration;
+import org.apache.hadoop.io.Text;
+import org.junit.Test;
+
+import com.tinkerpop.blueprints.GraphFactory;
+import com.tinkerpop.blueprints.Vertex;
+
+import edu.jhuapl.tinkerpop.AccumuloGraphConfiguration.InstanceType;
+
+public class AccumuloGraphConfigurationTest {
+
+  @Test
+  public void testConfigurationInterface() throws Exception {
+    Configuration conf = AccumuloGraphTestUtils.generateGraphConfig("setPropsValid");
+    for (String key : AccumuloGraphConfiguration.getValidInternalKeys()) {
+      // This is bad... but we should allow them if they are valid keys.
+      conf.setProperty(key, "value");
+    }
+
+    conf = AccumuloGraphTestUtils.generateGraphConfig("setPropsInvalid");
+    try {
+      conf.setProperty("invalidKey", "value");
+      fail();
+    } catch (Exception e) { }
+  }
+
+  @Test
+  public void testSplits() throws Exception {
+    AccumuloGraphConfiguration cfg;
+
+    // Tests for splits string.
+    cfg = AccumuloGraphTestUtils.generateGraphConfig("nullSplits").setSplits((String) null);
+    AccumuloGraph graph = (AccumuloGraph) GraphFactory.open(cfg.getConfiguration());
+    for (String table : cfg.getTableNames()) {
+      assertEquals(0, cfg.getConnector().tableOperations().listSplits(table).size());
+    }
+    graph.shutdown();
+
+    cfg = AccumuloGraphTestUtils.generateGraphConfig("emptySplits").setSplits("");
+    graph = (AccumuloGraph) GraphFactory.open(cfg.getConfiguration());
+    for (String table : cfg.getTableNames()) {
+      assertEquals(0, cfg.getConnector().tableOperations().listSplits(table).size());
+    }
+    graph.shutdown();
+
+    cfg = AccumuloGraphTestUtils.generateGraphConfig("threeSplits").setSplits(" a b c ");
+    graph = (AccumuloGraph) GraphFactory.open(cfg.getConfiguration());
+    for (String table : cfg.getTableNames()) {
+      Collection<Text> splits = cfg.getConnector().tableOperations().listSplits(table);
+      assertEquals(3, splits.size());
+      List<Text> arr = new ArrayList<Text>(splits);
+      assertEquals("a", arr.get(0).toString());
+      assertEquals("b", arr.get(1).toString());
+      assertEquals("c", arr.get(2).toString());
+    }
+    graph.shutdown();
+
+    // Tests for splits array.
+    cfg = AccumuloGraphTestUtils.generateGraphConfig("nullSplitsArray").setSplits((String[]) null);
+    graph = (AccumuloGraph) GraphFactory.open(cfg.getConfiguration());
+    for (String table : cfg.getTableNames()) {
+      assertEquals(0, cfg.getConnector().tableOperations().listSplits(table).size());
+    }
+    graph.shutdown();
+
+    cfg = AccumuloGraphTestUtils.generateGraphConfig("emptySplitsArray").setSplits(new String[] {});
+    graph = (AccumuloGraph) GraphFactory.open(cfg.getConfiguration());
+    for (String table : cfg.getTableNames()) {
+      assertEquals(0, cfg.getConnector().tableOperations().listSplits(table).size());
+    }
+    graph.shutdown();
+
+    cfg = AccumuloGraphTestUtils.generateGraphConfig("threeSplitsArray").setSplits(new String[] {"d", "e", "f"});
+    graph = (AccumuloGraph) GraphFactory.open(cfg.getConfiguration());
+    for (String table : cfg.getTableNames()) {
+      Collection<Text> splits = cfg.getConnector().tableOperations().listSplits(table);
+      assertEquals(3, splits.size());
+      List<Text> arr = new ArrayList<Text>(splits);
+      assertEquals("d", arr.get(0).toString());
+      assertEquals("e", arr.get(1).toString());
+      assertEquals("f", arr.get(2).toString());
+    }
+    graph.shutdown();
+  }
+
+  @Test
+  public void testPropertyValues() throws Exception {
+    AccumuloGraph graph = new AccumuloGraph(AccumuloGraphTestUtils.generateGraphConfig("propertyValues"));
+    // Tests for serialization/deserialization of properties.
+    QName qname = new QName("ns", "prop");
+    Vertex v = graph.addVertex(null);
+    v.setProperty("qname", qname);
+    assertTrue(v.getProperty("qname") instanceof QName);
+    assertTrue(qname.equals(v.getProperty("qname")));
+  }
+
+  @Test
+  public void testIsEmpty() throws Exception {
+    AccumuloGraphConfiguration cfg = AccumuloGraphTestUtils.generateGraphConfig("isEmpty");
+    AccumuloGraph graph = new AccumuloGraph(cfg);
+    assertTrue(graph.isEmpty());
+
+    graph.addVertex("A");
+    assertFalse(graph.isEmpty());
+
+    graph.clear();
+    assertTrue(graph.isEmpty());
+  }
+
+  @Test
+  public void testCreateAndClear() throws Exception {
+    AccumuloGraphConfiguration cfg = AccumuloGraphTestUtils.generateGraphConfig("noCreate").setCreate(false);
+    try {
+      new AccumuloGraph(cfg);
+      fail("Create is disabled and graph does not exist");
+    } catch (Exception e) {
+      assertTrue(true);
+    }
+
+    cfg = AccumuloGraphTestUtils.generateGraphConfig("yesCreate").setCreate(true);
+    for (String t : cfg.getTableNames()) {
+      assertFalse(cfg.getConnector().tableOperations().exists(t));
+    }
+    AccumuloGraph graph = new AccumuloGraph(cfg);
+    for (String t : cfg.getTableNames()) {
+      assertTrue(cfg.getConnector().tableOperations().exists(t));
+    }
+    graph.shutdown();
+
+    graph = new AccumuloGraph(cfg.clone().setCreate(false));
+    assertTrue(graph.isEmpty());
+    graph.addVertex("A");
+    graph.addVertex("B");
+    assertFalse(graph.isEmpty());
+    graph.shutdown();
+
+    graph = new AccumuloGraph(cfg.clone().setClear(true));
+    assertTrue(graph.isEmpty());
+    graph.shutdown();
+  }
+
+  @Test
+  public void testPrint() throws Exception {
+    AccumuloGraphConfiguration cfg =
+        AccumuloGraphTestUtils.generateGraphConfig("printTest");
+    cfg.print();
+  }
+
+  @Test
+  public void testInvalidCacheParams() throws Exception {
+    int size = 100;
+    int timeout = 30000;
+
+    AccumuloGraphConfiguration cfg = AccumuloGraphTestUtils
+        .generateGraphConfig("cacheParams");
+    cfg.validate();
+
+
+    // Vertex cache.
+
+    assertFalse(cfg.getVertexCacheEnabled());
+
+    try {
+      cfg.setVertexCacheParams(-1, timeout);
+      fail();
+    } catch (Exception e) { }
+
+    try {
+      cfg.setVertexCacheParams(size, -1);
+      fail();
+    } catch (Exception e) { }
+
+    assertFalse(cfg.getVertexCacheEnabled());
+
+    cfg.setVertexCacheParams(size, timeout);
+    cfg.validate();
+    assertTrue(cfg.getVertexCacheEnabled());
+    assertEquals(size, cfg.getVertexCacheSize());
+    assertEquals(timeout, cfg.getVertexCacheTimeout());
+
+    cfg.setVertexCacheParams(-1, -1);
+    cfg.validate();
+    assertFalse(cfg.getVertexCacheEnabled());
+
+
+    // Edge cache.
+
+    assertFalse(cfg.getEdgeCacheEnabled());
+
+    try {
+      cfg.setEdgeCacheParams(-1, timeout);
+      fail();
+    } catch (Exception e) { }
+
+    try {
+      cfg.setEdgeCacheParams(size, -1);
+      fail();
+    } catch (Exception e) { }
+
+    assertFalse(cfg.getEdgeCacheEnabled());
+
+    cfg.setEdgeCacheParams(size, timeout);
+    cfg.validate();
+    assertTrue(cfg.getEdgeCacheEnabled());
+    assertEquals(size, cfg.getEdgeCacheSize());
+    assertEquals(timeout, cfg.getEdgeCacheTimeout());
+
+    cfg.setEdgeCacheParams(-1, -1);
+    cfg.validate();
+    assertFalse(cfg.getEdgeCacheEnabled());
+  }
+
+  /**
+   * Test different kinds of graph names (hyphens, punctuation, etc).
+   * @throws Exception
+   */
+  @Test
+  public void testGraphNames() throws Exception {
+    AccumuloGraphConfiguration conf = new AccumuloGraphConfiguration();
+
+    String[] valid = new String[] {
+        "alpha", "12345", "alnum12345",
+        "12345alnum", "under_score1", "_under_score_2"};
+    String[] invalid = new String[] {"hyph-en",
+        "dot..s", "quo\"tes"};
+
+    for (String name : valid) {
+      conf.setGraphName(name);
+    }
+
+    for (String name : invalid) {
+      try {
+        conf.setGraphName(name);
+        fail();
+      } catch (Exception e) { }
+    }
+  }
+
+  @Test
+  public void testPreloadedProperties() {
+    // Don't allow "all" and "some" preloaded properties.
+    AccumuloGraphConfiguration conf = new AccumuloGraphConfiguration();
+    conf.setPreloadAllProperties(true);
+    conf.setPreloadedProperties(new String[]{"one", "two", "three"});
+    try {
+      conf.validate();
+      fail();
+    } catch (Exception e) { }
+  }
+
+  @Test
+  public void testImmutableConnector() throws Exception {
+    AccumuloGraphConfiguration cfg = new AccumuloGraphConfiguration().setInstanceType(
+        InstanceType.Mock).setGraphName("immutableConnector")
+        .setCreate(true).setAutoFlush(false);
+
+    cfg.getConnector();
+
+    try {
+      cfg.setCreate(false);
+      fail();
+    } catch (Exception e) { }
+
+    try {
+      cfg.setAutoFlush(true);
+      fail();
+    } catch (Exception e) { }
+
+    assertTrue(cfg.getCreate());
+    assertFalse(cfg.getAutoFlush());
+  }
+  
+  @Test
+  public void testMockInstanceValue(){
+    AccumuloGraphConfiguration conf = new AccumuloGraphConfiguration().setInstanceType(InstanceType.Mock);
+    assertNotNull(conf.getInstanceName());
+    assertEquals("mock-instance", conf.getInstanceName());
+  }
+}