#13 Fix, add a try-catch on getting file by path with LocalFileSystem

As it appears that it throws an exception on invalid paths like one having "//.." in it

Author: klesun-itn

src/org/klesun/deep_js_completion/entry/PathStrGoToDecl.scala

@@ -20,7 +20,13 @@ object PathStrGoToDecl {
       .flatMap(f => Option(f.getContainingDirectory))
       .flatMap(f => Option(f.getVirtualFile))
       .map(f => f.getPath + "/" + relPath + (if (relPath.matches(".*\\.[a-zA-Z0-9]+$")) "" else ".js"))
-      .flatMap(fullPath => Option(LocalFileSystem.getInstance.findFileByPath(fullPath)))
+      .flatMap(fullPath => {
+        try {
+          Option(LocalFileSystem.getInstance.findFileByPath(fullPath))
+        } catch {
+          case exc: Throwable => None
+        }
+      })
       .flatMap(f => Option(PsiManager.getInstance(caretFile.getProject).findFile(f)))
   }
 

tests/index.js

@@ -1,6 +1,10 @@
 
+let SomeInvalidPathModule = require('.//../asd.js');
 let SomeCjsModule = require('./SomeCjsModule.js');
 
+// tried to reproduce #13, did not succeed, but will add the try-catch nevertheless
+SomeInvalidPathModule.asd;
+
 SomeCjsModule({}).runInputCmd('*R').then(resp => {
     resp.o;
 });