Merge pull request #94 from sarmbruster/params_npe

fix NPE if 'null' as parameter value is used

Author: nigelsmall

driver/src/main/java/org/neo4j/driver/v1/internal/summary/ResultBuilder.java

@@ -52,7 +52,7 @@ public class ResultBuilder implements StreamCollector
     public ResultBuilder( String statement, Map<String, Value> parameters )
     {
         Map<String, Value> unmodifiableParameters =
-            parameters.isEmpty() ? NO_PARAMETERS : unmodifiableMap( parameters );
+            (parameters == null) || (parameters.isEmpty()) ? NO_PARAMETERS : unmodifiableMap( parameters );
         this.summaryBuilder = new SummaryBuilder( new Statement( statement, unmodifiableParameters ) );
     }
 

driver/src/test/java/org/neo4j/driver/v1/integration/TransactionIT.java

@@ -133,4 +133,15 @@ public void shouldBeOpenBeforeCommit() throws Throwable
         assertTrue( tx.isOpen() );
     }
 
+    @Test
+    public void shouldHandleNullParametersGracefully()
+    {
+        // When
+        session.run("match (n) return count(n)", null);
+
+        // Then
+        // pass - no exception thrown
+
+    }
+
 }