Merge pull request #270 from pontusmelke/4.0-duration-format

Nicer formatting for Duration

Author: pontusmelke

cypher-shell/src/integration-test/java/org/neo4j/shell/commands/CypherShellVerboseIntegrationTest.java

@@ -74,6 +74,16 @@ public void tearDown() throws Exception
         shell.execute( "MATCH (n) DETACH DELETE (n)" );
     }
 
+    @Test
+    public void parseDuration() throws CommandException
+    {
+        //when
+        shell.execute( "RETURN duration({months:0.75})" );
+
+        //then
+        assertThat( linePrinter.output(), containsString( "P22DT19H51M49.5S" ) );
+    }
+
     @Test
     public void cypherWithNoReturnStatements() throws CommandException
     {

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

@@ -39,6 +39,7 @@
 import org.neo4j.driver.types.Point;
 import org.neo4j.driver.types.Relationship;
 import org.neo4j.shell.state.BoltResult;
+import org.neo4j.values.storable.DurationValue;
 
 import static java.util.Arrays.asList;
 import static java.util.concurrent.TimeUnit.MILLISECONDS;
@@ -205,6 +206,8 @@ default String formatValue( final Value value )
             return pathAsString( value.asPath() );
         case POINT:
             return pointAsString( value.asPoint() );
+        case DURATION:
+            return DurationValue.parse( value.toString() ).prettyPrint();
         case ANY:
         case BOOLEAN:
         case BYTES:
@@ -217,7 +220,6 @@ default String formatValue( final Value value )
         case DATE_TIME:
         case LOCAL_TIME:
         case LOCAL_DATE_TIME:
-        case DURATION:
         case NULL:
         default:
             return value.toString();