Merge pull request #107 from Trivadis/bugfix/issue-106-param-with-spaces

Bugfix/issue #106 Cannot process RootPath with spaces

sqlcl/format.js

@@ -452,11 +452,11 @@ var run = function(args) {
 }
 
 var getArgs = function(cmdLine) {
-    var p = javaPattern.compile('("[^"]*")|([^ ]+)');
+    var p = javaPattern.compile('("([^"]*)")|([^ ]+)');
     var m = p.matcher(cmdLine.trim());
     var args = [];
     while (m.find()) {
-        args.push(m.group());
+        args.push(m.group(3) != null ? m.group(3) : m.group(2));
     }
     return args;
 }

tests/src/test/java/com/trivadis/plsql/formatter/sqlcl/tests/TvdFormatPathWithSpaceTest.java

@@ -0,0 +1,29 @@
+package com.trivadis.plsql.formatter.sqlcl.tests;
+
+import org.junit.Assert;
+import org.junit.Before;
+import org.junit.Test;
+
+import java.io.IOException;
+import java.nio.file.Files;
+import java.nio.file.Path;
+import java.nio.file.Paths;
+
+public class TvdFormatPathWithSpaceTest extends AbstractSqlclTest {
+
+    @Before
+    public void registerCommandBeforeTest() {
+        runScript("--register");
+        byteArrayOutputStream.reset();
+    }
+
+    @Test
+    public void path_with_spaces() throws IOException {
+        final Path myFilesDir = Files.createTempDirectory("my files");
+        final String actual = runCommand("tvdformat \"" + myFilesDir.toAbsolutePath() + "\"");
+        // directory is empty, no files processed
+        Assert.assertTrue(actual.trim().isEmpty());
+    }
+
+}
+