adding point type

Author: praveenag

cypher-shell/src/main/java/org/neo4j/shell/prettyprint/OutputFormatter.java

@@ -5,6 +5,7 @@
 import org.neo4j.driver.v1.Values;
 import org.neo4j.driver.v1.summary.Plan;
 import org.neo4j.driver.v1.summary.ResultSummary;
+import org.neo4j.driver.v1.types.IsoDuration;
 import org.neo4j.driver.v1.types.Node;
 import org.neo4j.driver.v1.types.Path;
 import org.neo4j.driver.v1.types.Point;
@@ -49,6 +50,7 @@ public interface OutputFormatter {
             case POINT:
                 return pointAsString(value.asPoint());
             case DURATION:
+                return durationAsString(value.asIsoDuration());
             case ANY:
             case BOOLEAN:
             case BYTES:
@@ -81,6 +83,11 @@ default String pointAsString(Point point) {
         return stringBuilder.toString();
     }
 
+    @Nonnull
+    default String durationAsString(IsoDuration duration) {
+        return duration.toString();
+    }
+
     @Nonnull
     default String pathAsString(@Nonnull Path path) {
         List<String> list = new ArrayList<>(path.length());

cypher-shell/src/test/java/org/neo4j/shell/prettyprint/TableOutputFormatterTest.java

@@ -2,12 +2,14 @@
 
 import org.hamcrest.CoreMatchers;
 import org.junit.Test;
+import org.neo4j.driver.internal.InternalIsoDuration;
 import org.neo4j.driver.internal.InternalNode;
 import org.neo4j.driver.internal.InternalPath;
 import org.neo4j.driver.internal.InternalPoint2D;
 import org.neo4j.driver.internal.InternalPoint3D;
 import org.neo4j.driver.internal.InternalRecord;
 import org.neo4j.driver.internal.InternalRelationship;
+import org.neo4j.driver.internal.value.DurationValue;
 import org.neo4j.driver.internal.value.NodeValue;
 import org.neo4j.driver.internal.value.PathValue;
 import org.neo4j.driver.internal.value.PointValue;
@@ -94,6 +96,25 @@ public void prettyPrintPoint() throws Exception {
         assertThat(actual, containsString("| point({srid:4326, x:1.7, y:26.79, z:34.23}) |"));
     }
 
+    @Test
+    public void prettyPrintDuration() throws Exception {
+        // given
+        StatementResult statementResult = mock(StatementResult.class);
+        List<String> keys = asList("d");
+
+        when(statementResult.summary()).thenReturn(mock(ResultSummary.class));
+        when(statementResult.keys()).thenReturn(keys);
+
+        Value duration = new DurationValue(new InternalIsoDuration(1, 2, 3, 4));
+        Record record = new InternalRecord(keys, new Value[]{duration});
+
+        // when
+        String actual = verbosePrinter.format(new BoltResult(asList(record), statementResult));
+
+        // then
+        assertThat(actual, containsString("| Duration{months=1, days=2, seconds=3, nanoseconds=4} |"));
+    }
+
     @Test
     public void prettyPrintNode() throws Exception {
         // given