fixes #1262: apoc.cypher.runFile fails when there is a empty transact…

…ion (#1349)

src/main/java/apoc/cypher/Cypher.java

@@ -172,7 +172,9 @@ private Object executeStatement(BlockingQueue<RowResult> queue, String stmt, Map
     private String removeShellControlCommands(String stmt) {
         Matcher matcher = shellControl.matcher(stmt.trim());
         if (matcher.find()) {
-            stmt = matcher.replaceAll("");
+            // an empty file get transformed into ":begin\n:commit" and that statement is not matched by the pattern
+            // because ":begin\n:commit".replaceAll("") => "\n:commit" with the recursion we avoid the problem
+            return removeShellControlCommands(matcher.replaceAll(""));
         }
         return stmt;
     }

src/test/java/apoc/cypher/CypherTest.java

@@ -457,4 +457,10 @@ public void testCaseDoElseCondition() throws Exception {
                     assertEquals("C", ((Map) r.get("value")).get("cName"));
                 });
     }
+
+    @Test
+    public void testRunFileWithEmptyFile() throws Exception {
+        testResult(db, "CALL apoc.cypher.runFile('src/test/resources/empty.cypher')",
+                r -> assertFalse("should be empty", r.hasNext()));
+    }
 }

src/test/resources/empty.cypher

@@ -0,0 +1,2 @@
+:begin
+:commit