Operation on transactions should work with routing

We weren't checking for the cluster faillures when running on transactions.
All operations should check for leader-switches and connection failures and
handle them appropriately.

Author: pontusmelke

driver/src/main/java/org/neo4j/driver/internal/RoutingNetworkSession.java

@@ -39,6 +39,9 @@
 import static java.lang.String.format;
 import static org.neo4j.driver.v1.Values.value;
 
+/**
+ * A session that safely handles routing errors.
+ */
 public class RoutingNetworkSession implements Session
 {
     protected final Session delegate;
@@ -107,13 +110,13 @@ public TypeSystem typeSystem()
     @Override
     public Transaction beginTransaction()
     {
-        return delegate.beginTransaction();
+        return new RoutingTransaction( delegate.beginTransaction(), mode, address, onError);
     }
 
     @Override
     public Transaction beginTransaction( String bookmark )
     {
-        return delegate.beginTransaction(bookmark);
+        return new RoutingTransaction( delegate.beginTransaction(bookmark), mode, address, onError);
     }
 
     @Override

driver/src/main/java/org/neo4j/driver/internal/RoutingTransaction.java

@@ -0,0 +1,143 @@
+/**
+ * Copyright (c) 2002-2016 "Neo Technology,"
+ * Network Engine for Objects in Lund AB [http://neotechnology.com]
+ *
+ * This file is part of Neo4j.
+ *
+ * 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 org.neo4j.driver.internal;
+
+
+import java.util.Map;
+
+import org.neo4j.driver.internal.net.BoltServerAddress;
+import org.neo4j.driver.v1.AccessMode;
+import org.neo4j.driver.v1.Record;
+import org.neo4j.driver.v1.Statement;
+import org.neo4j.driver.v1.StatementResult;
+import org.neo4j.driver.v1.Transaction;
+import org.neo4j.driver.v1.Value;
+import org.neo4j.driver.v1.Values;
+import org.neo4j.driver.v1.exceptions.ClientException;
+import org.neo4j.driver.v1.exceptions.ConnectionFailureException;
+import org.neo4j.driver.v1.types.TypeSystem;
+
+import static org.neo4j.driver.internal.RoutingNetworkSession.filterFailureToWrite;
+import static org.neo4j.driver.internal.RoutingNetworkSession.sessionExpired;
+import static org.neo4j.driver.v1.Values.value;
+
+/**
+ * A transaction that safely handles routing errors.
+ */
+public class RoutingTransaction implements Transaction
+{
+    protected final Transaction delegate;
+    private final AccessMode mode;
+    private final BoltServerAddress address;
+    private final RoutingErrorHandler onError;
+
+    RoutingTransaction( Transaction delegate, AccessMode mode, BoltServerAddress address,
+            RoutingErrorHandler onError )
+    {
+        this.delegate = delegate;
+        this.mode = mode;
+        this.address = address;
+        this.onError = onError;
+    }
+
+    @Override
+    public StatementResult run( String statementText )
+    {
+        return run( statementText, Values.EmptyMap );
+    }
+
+    @Override
+    public StatementResult run( String statementText, Map<String,Object> statementParameters )
+    {
+        Value params = statementParameters == null ? Values.EmptyMap : value( statementParameters );
+        return run( statementText, params );
+    }
+
+    @Override
+    public StatementResult run( String statementTemplate, Record statementParameters )
+    {
+        Value params = statementParameters == null ? Values.EmptyMap : value( statementParameters.asMap() );
+        return run( statementTemplate, params );
+    }
+
+    @Override
+    public StatementResult run( String statementText, Value statementParameters )
+    {
+        return run( new Statement( statementText, statementParameters ) );
+    }
+
+    @Override
+    public StatementResult run( Statement statement )
+    {
+        try
+        {
+            return new RoutingStatementResult( delegate.run( statement ), mode, address, onError );
+        }
+        catch ( ConnectionFailureException e )
+        {
+            throw sessionExpired( e, onError, address );
+        }
+        catch ( ClientException e )
+        {
+            throw filterFailureToWrite( e, mode, onError, address );
+        }
+    }
+
+    @Override
+    public TypeSystem typeSystem()
+    {
+        return delegate.typeSystem();
+    }
+
+
+    @Override
+    public void success()
+    {
+
+    }
+
+    @Override
+    public void failure()
+    {
+
+    }
+
+    @Override
+    public boolean isOpen()
+    {
+        return false;
+    }
+
+    @Override
+    public void close()
+    {
+        try
+        {
+            delegate.close();
+        }
+        catch ( ConnectionFailureException e )
+        {
+            throw sessionExpired(e, onError, address);
+        }
+        catch ( ClientException e )
+        {
+            throw filterFailureToWrite( e, mode, onError, address );
+        }
+    }
+}