Merge pull request #56 from lge88/master

test('-L', badlink) should return true
shelljs · Apr 6, 2013 · 3b57b24 · 3b57b24
2 parents f7d2e3c + 38f1d6d
commit 3b57b24
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 3 deletions.
diff --git a/shell.js b/shell.js
@@ -584,15 +584,20 @@ function _test(options, path) {
   if (!canInterpret)
     error('could not interpret expression');
 
+  if (options.link) {
+    try {
+      return fs.lstatSync(path).isSymbolicLink();
+    } catch(e) {
+      return false;
+    }
+  }
+
   if (!fs.existsSync(path))
     return false;
 
   if (options.exists)
     return true;
 
-  if (options.link)
-    return fs.lstatSync(path).isSymbolicLink();
-
   var stats = fs.statSync(path);
 
   if (options.block)

diff --git a/test/resources/badlink b/test/resources/badlink
@@ -0,0 +1 @@
+not_existed_file
diff --git a/test/test.js b/test/test.js
@@ -80,4 +80,12 @@ var result = shell.test('-L', 'resources/link');
 assert.equal(shell.error(), null);
 assert.equal(result, true);//true
 
+var result = shell.test('-L', 'resources/badlink');
+assert.equal(shell.error(), null);
+assert.equal(result, true);//true
+
+var result = shell.test('-L', 'resources/404');
+assert.equal(shell.error(), null);
+assert.equal(result, false);//false
+
 shell.exit(123);