Adds foundations for Path

Author: itamarhaber

src/main/java/io/rejson/Client.java

@@ -1,5 +1,7 @@
 package io.rejson;
 
+import io.rejson.Path;
+
 import redis.clients.jedis.Jedis;
 import redis.clients.jedis.JedisPool;
 import redis.clients.jedis.JedisPoolConfig;
@@ -83,12 +85,12 @@ public Client(final String host, final int port) {
      * @param path a path in the object
      * @return the number of paths deleted (0 or 1)
      */
-    public Long del(final String key, final String path) throws Exception {
+    public Long del(final String key, final Path path) throws Exception {
         Jedis conn = _conn();
         ArrayList<byte[]> args = new ArrayList(3);
 
         args.add(SafeEncoder.encode(key));
-        args.add(SafeEncoder.encode(path));
+        args.add(SafeEncoder.encode(path.toString()));
 
         Long rep = conn.getClient()
                 .sendCommand(Command.DEL, args.toArray(new byte[args.size()][]))
@@ -104,13 +106,13 @@ public Long del(final String key, final String path) throws Exception {
      * @param path a path in the object
      * @return the requested object
      */
-    public Object get(final String key, final String path) throws Exception {
+    public Object get(final String key, final Path path) throws Exception {
         // TODO: need the variadic paths variant?
         Jedis conn = _conn();
         ArrayList<byte[]> args = new ArrayList(2);
 
         args.add(SafeEncoder.encode(key));
-        args.add(SafeEncoder.encode(path));
+        args.add(SafeEncoder.encode(path.toString()));
 
         String rep = conn.getClient()
                 .sendCommand(Command.GET, args.toArray(new byte[args.size()][]))
@@ -129,13 +131,13 @@ public Object get(final String key, final String path) throws Exception {
      * @param path a path in the object
      * @param object the Java object to store
      */
-    public void set(final String key, final String path, final Object object) throws Exception {
+    public void set(final String key, final Path path, final Object object) throws Exception {
         // TODO: support NX|XX flags
         Jedis conn = _conn();
         ArrayList<byte[]> args = new ArrayList(3);
 
         args.add(SafeEncoder.encode(key));
-        args.add(SafeEncoder.encode(path));
+        args.add(SafeEncoder.encode(path.toString()));
         args.add(SafeEncoder.encode(gson.toJson(object)));
 
         String status = conn.getClient()
@@ -152,12 +154,12 @@ public void set(final String key, final String path, final Object object) throws
      * @param path a path in the object
      * @return the Java class of the requested object
      */
-    public Class<? extends Object> type(final String key, final String path) throws Exception {
+    public Class<? extends Object> type(final String key, final Path path) throws Exception {
         Jedis conn = _conn();
         ArrayList<byte[]> args = new ArrayList(2);
 
         args.add(SafeEncoder.encode(key));
-        args.add(SafeEncoder.encode(path));
+        args.add(SafeEncoder.encode(path.toString()));
 
         String rep = conn.getClient()
                 .sendCommand(Command.TYPE, args.toArray(new byte[args.size()][]))

src/main/java/io/rejson/Path.java

@@ -0,0 +1,18 @@
+package io.rejson;
+
+public class Path {
+    private final String strPath;
+
+    public Path(final String strPath) {
+        this.strPath = strPath;
+    }
+
+    public static Path RootPath() {
+        return new Path(".");
+    }
+
+    @Override
+    public String toString() {
+        return strPath;
+    }
+}

src/test/java/io/rejson/ClientTest.java

@@ -26,59 +26,59 @@ public void initialize() {
     public void set() throws Exception {
         c._conn().flushDB();
 
-        c.set("null", ".", null);
-        c.set("foobar", ".", new FooBarObject());
+        c.set("null", Path.RootPath(), null);
+        c.set("foobar", Path.RootPath(), new FooBarObject());
     }
 
     @Test(expected = Exception.class)
     public void setException() throws Exception {
         c._conn().flushDB();
 
         // should error on non root path for new key
-        c.set("test", ".foo", "bar");
+        c.set("test", new Path(".foo"), "bar");
     }
 
     @Test
     public void get() throws Exception {
         c._conn().flushDB();
-        c.set("test", ".", "foo");
-        assertEquals("foo", c.get("test", "."));
+        c.set("test", Path.RootPath(), "foo");
+        assertEquals("foo", c.get("test", Path.RootPath()));
     }
 
     @Test(expected = Exception.class)
     public void getException() throws Exception {
         c._conn().flushDB();
-        c.set("test", ".", "foo");
-        c.get("test", ".bar");
+        c.set("test", Path.RootPath(), "foo");
+        c.get("test", new Path(".bar"));
     }
 
     @Test
     public void del() throws Exception {
         c._conn().flushDB();
-        c.set("foobar", ".", new FooBarObject());
-        c.del("foobar", ".foo");
+        c.set("foobar", Path.RootPath(), new FooBarObject());
+        c.del("foobar", new Path(".foo"));
     }
 
     @Test(expected = Exception.class)
     public void delException() throws Exception {
         c._conn().flushDB();
-        c.set("foobar", ".", new FooBarObject());
-        c.del("foobar", ".foo[1]");
+        c.set("foobar", Path.RootPath(), new FooBarObject());
+        c.del("foobar", new Path(".foo[1]"));
     }
 
     @Test
     public void type() throws Exception {
         c._conn().flushDB();
-        c.set("foobar", ".", new FooBarObject());
-        assertSame(Object.class, c.type("foobar", "."));
-        assertSame(String.class, c.type("foobar", ".foo"));
+        c.set("foobar", Path.RootPath(), new FooBarObject());
+        assertSame(Object.class, c.type("foobar", Path.RootPath()));
+        assertSame(String.class, c.type("foobar", new Path(".foo")));
     }
 
     @Test(expected = Exception.class)
     public void typeException() throws Exception {
         c._conn().flushDB();
-        c.set("foobar", ".", new FooBarObject());
-        c.type("foobar", ".foo[1]");
+        c.set("foobar", Path.RootPath(), new FooBarObject());
+        c.type("foobar", new Path(".foo[1]"));
     }
 
 }