Enable and update execution guard tests for driver.

neo4j · Sep 9, 2016 · 80f090a · 80f090a
1 parent 71c9933
commit 80f090a
Show file tree

Hide file tree

Showing 2 changed files with 59 additions and 24 deletions.
diff --git a/community/bolt/src/main/java/org/neo4j/bolt/v1/messaging/BoltMessageRouter.java b/community/bolt/src/main/java/org/neo4j/bolt/v1/messaging/BoltMessageRouter.java
@@ -20,17 +20,17 @@
 
 package org.neo4j.bolt.v1.messaging;
 
+import java.io.IOException;
+import java.util.HashMap;
+import java.util.Map;
+
 import org.neo4j.bolt.v1.runtime.BoltResponseHandler;
 import org.neo4j.bolt.v1.runtime.BoltWorker;
 import org.neo4j.bolt.v1.runtime.Neo4jError;
-import org.neo4j.bolt.v1.runtime.spi.Record;
 import org.neo4j.bolt.v1.runtime.spi.BoltResult;
+import org.neo4j.bolt.v1.runtime.spi.Record;
 import org.neo4j.logging.Log;
 
-import java.io.IOException;
-import java.util.HashMap;
-import java.util.Map;
-
 /**
  * This class is responsible for routing incoming request messages to a worker
  * as well as handling outgoing response messages via appropriate handlers.
@@ -141,9 +141,6 @@ public void onStart()
         @Override
         public void onRecords( BoltResult result, boolean pull ) throws Exception
         {
-            // Overridden if records are returned, therefore
-            // should fail if called but not overridden.
-            assert false;
         }
 
         @Override

diff --git a/integrationtests/src/test/java/org/neo4j/TransactionGuardIntegrationTest.java b/integrationtests/src/test/java/org/neo4j/TransactionGuardIntegrationTest.java
@@ -42,6 +42,7 @@
 import org.neo4j.graphdb.config.Setting;
 import org.neo4j.graphdb.factory.GraphDatabaseBuilder;
 import org.neo4j.graphdb.factory.GraphDatabaseSettings;
+import org.neo4j.harness.internal.Ports;
 import org.neo4j.helpers.collection.MapUtil;
 import org.neo4j.io.fs.FileUtils;
 import org.neo4j.kernel.GraphDatabaseDependencies;
@@ -103,6 +104,8 @@ public class TransactionGuardIntegrationTest
     private static GraphDatabaseAPI databaseWithTimeoutAndGuard;
     private static GraphDatabaseAPI databaseWithoutTimeout;
     private static CommunityNeoServer neoServer;
+    private static int boltPortCustomGuard;
+    private static int boltPortDatabaseWithTimeout;
 
     @Test
     public void terminateLongRunningTransaction()
@@ -304,15 +307,15 @@ public void terminateLongRunningRestTransactionalEndpointWithCustomTimeoutQuery(
         assertDatabaseDoesNotHaveNodes( database );
     }
 
-//    @Test
+    @Test
     public void terminateLongRunningDriverQuery() throws Exception
     {
         GraphDatabaseAPI database = startDatabaseWithTimeout();
         CommunityNeoServer neoServer = startNeoServer( (GraphDatabaseFacade) database );
 
         org.neo4j.driver.v1.Config driverConfig = getDriverConfig();
 
-        try ( Driver driver = GraphDatabase.driver( "bolt://localhost", driverConfig );
+        try ( Driver driver = GraphDatabase.driver( "bolt://localhost:" + boltPortDatabaseWithTimeout, driverConfig );
               Session session = driver.session() )
         {
             org.neo4j.driver.v1.Transaction transaction = session.beginTransaction();
@@ -332,12 +335,35 @@ public void terminateLongRunningDriverQuery() throws Exception
         assertDatabaseDoesNotHaveNodes( database );
     }
 
+    @Test
+    public void terminateLongRunningDriverPeriodicCommitQuery() throws Exception
+    {
+        GraphDatabaseAPI database = startDatabaseWithTimeoutCustomGuard();
+        CommunityNeoServer neoServer = startNeoServer( (GraphDatabaseFacade) database );
+
+        org.neo4j.driver.v1.Config driverConfig = getDriverConfig();
+
+        try ( Driver driver = GraphDatabase.driver( "bolt://localhost:" + boltPortCustomGuard, driverConfig );
+              Session session = driver.session() )
+        {
+            URL url = prepareTestImportFile( 8 );
+            session.run( "USING PERIODIC COMMIT 5 LOAD CSV FROM '" + url + "' AS line CREATE ();" ).consume();
+            fail("Transaction should be already terminated by execution guard.");
+        }
+        catch ( Exception expected )
+        {
+            //
+        }
+        assertDatabaseDoesNotHaveNodes( database );
+    }
+
     private GraphDatabaseAPI startDatabaseWithTimeoutCustomGuard()
     {
         if ( databaseWithTimeoutAndGuard == null )
         {
-            Map<Setting<?>,String> configMap = getSettingsWithTransactionTimeout();
-            databaseWithTimeoutAndGuard = startCustomGuardedDatabase( testDirectory.directory( "dbWithoutTimeoutAndguard" ), configMap );
+            boltPortCustomGuard = findFreePort();
+            Map<Setting<?>,String> configMap = getSettingsWithTimeoutAndBolt( boltPortCustomGuard );
+            databaseWithTimeoutAndGuard = startCustomGuardedDatabase( testDirectory.directory( "dbWithoutTimeoutAndGuard" ), configMap );
         }
         return databaseWithTimeoutAndGuard;
     }
@@ -346,9 +372,9 @@ private GraphDatabaseAPI startDatabaseWithTimeout()
     {
         if ( databaseWithTimeout == null )
         {
-            Map<Setting<?>,String> configMap = getSettingsWithTimeoutAndBolt();
-            databaseWithTimeout = startCustomDatabase( testDirectory.directory( "dbWithTimeout" ),
-                    configMap );
+            boltPortDatabaseWithTimeout = findFreePort();
+            Map<Setting<?>,String> configMap = getSettingsWithTimeoutAndBolt( boltPortDatabaseWithTimeout );
+            databaseWithTimeout = startCustomDatabase( testDirectory.directory( "dbWithTimeout" ), configMap );
         }
         return databaseWithTimeout;
     }
@@ -389,21 +415,16 @@ private CommunityNeoServer startNeoServer( GraphDatabaseFacade database ) throws
         return neoServer;
     }
 
-    private Map<Setting<?>,String> getSettingsWithTimeoutAndBolt()
+    private Map<Setting<?>,String> getSettingsWithTimeoutAndBolt( int boltPort )
     {
-        Map<Setting<?>,String> configMap = getSettingsWithTransactionTimeout();
         GraphDatabaseSettings.BoltConnector boltConnector = boltConnector( "0" );
-        MapUtil.genericMap(configMap,
+        return MapUtil.genericMap(
+                GraphDatabaseSettings.transaction_timeout, "2s",
+                boltConnector.address, "localhost:" + boltPort,
                 boltConnector.type, "BOLT",
                 boltConnector.enabled, "true",
                 boltConnector.encryption_level, GraphDatabaseSettings.BoltConnector.EncryptionLevel.DISABLED.name(),
                 GraphDatabaseSettings.auth_enabled, "false" );
-        return configMap;
-    }
-
-    private Map<Setting<?>,String> getSettingsWithTransactionTimeout()
-    {
-        return MapUtil.genericMap( GraphDatabaseSettings.transaction_timeout, "2s" );
     }
 
     private Map<Setting<?>,String> getSettingsWithoutTransactionTimeout()
@@ -429,6 +450,23 @@ private URL prepareTestImportFile( int lines ) throws IOException
         return tempFile.toURI().toURL();
     }
 
+    private int findFreePort()
+    {
+        return freePort( 8000, 8100 );
+    }
+
+    private int freePort(int startRange, int endRange)
+    {
+        try
+        {
+            return Ports.findFreePort( Ports.INADDR_LOCALHOST, new int[]{startRange, endRange} ).getPort();
+        }
+        catch ( IOException e )
+        {
+            throw new RuntimeException( "Unable to find an available port: " + e.getMessage(), e );
+        }
+    }
+
     private Response execute( GraphDatabaseShellServer shellServer,
             CollectingOutput output, Serializable clientId, String command ) throws ShellException
     {