Fixed Stat file type tests

Author: CMCDragonkai

lib/Stat.js

@@ -62,49 +62,49 @@ class Stat {
    * Checks if file.
    */
   isFile (): bool {
-    return !!(this.mode & constants.S_IFREG);
+    return (this.mode & constants.S_IFMT) == constants.S_IFREG;
   }
 
   /**
    * Checks if directory.
    */
   isDirectory (): bool {
-    return !!(this.mode & constants.S_IFDIR);
+    return (this.mode & constants.S_IFMT) == constants.S_IFDIR;
   }
 
   /**
    * Checks if block device.
    */
   isBlockDevice (): bool {
-    return !!(this.mode & constants.S_IFBLK);
+    return (this.mode & constants.S_IFMT) == constants.S_IFBLK;
   }
 
   /**
    * Checks if character device.
    */
   isCharacterDevice (): bool {
-    return !!(this.mode & constants.S_IFCHR);
+    return (this.mode & constants.S_IFMT) == constants.S_IFCHR;
   }
 
   /**
    * Checks if symbolic link.
    */
   isSymbolicLink (): bool {
-    return !!(this.mode & constants.S_IFLNK);
+    return (this.mode & constants.S_IFMT) == constants.S_IFLNK;
   }
 
   /**
    * Checks if FIFO.
    */
   isFIFO (): bool {
-    return !!(this.mode & constants.S_IFIFO);
+    return (this.mode & constants.S_IFMT) == constants.S_IFIFO;
   }
 
   /**
    * Checks if socket.
    */
   isSocket (): bool {
-    return !!(this.mode & constants.S_IFSOCK);
+    return (this.mode & constants.S_IFMT) == constants.S_IFSOCK;
   }
 
 }

test/VirtualFS.js

@@ -114,6 +114,50 @@ test.cb('asynchronous errors are passed to callbacks - async', t => {
   });
 });
 
+///////////////
+// stat type //
+///////////////
+
+test('file stat makes sense - sync', t => {
+  const fs = new VirtualFS;
+  fs.writeFileSync('/test', 'test data');
+  const stat = fs.statSync('/test');
+  t.true(stat.isFile());
+  t.false(stat.isDirectory());
+  t.false(stat.isBlockDevice());
+  t.false(stat.isCharacterDevice());
+  t.false(stat.isSocket());
+  t.false(stat.isSymbolicLink());
+  t.false(stat.isFIFO());
+});
+
+test('dir stat makes sense - sync', t => {
+  const fs = new VirtualFS;
+  fs.mkdirSync('/dir');
+  const stat = fs.statSync('/dir');
+  t.false(stat.isFile());
+  t.true(stat.isDirectory());
+  t.false(stat.isBlockDevice());
+  t.false(stat.isCharacterDevice());
+  t.false(stat.isSocket());
+  t.false(stat.isSymbolicLink());
+  t.false(stat.isFIFO());
+});
+
+test('symlink stat makes sense - sync', t => {
+  const fs = new VirtualFS;
+  fs.writeFileSync('/a', 'data');
+  fs.symlinkSync('/a', '/link-to-a');
+  const stat = fs.lstatSync('/link-to-a');
+  t.false(stat.isFile());
+  t.false(stat.isDirectory());
+  t.false(stat.isBlockDevice());
+  t.false(stat.isCharacterDevice());
+  t.false(stat.isSocket());
+  t.true(stat.isSymbolicLink());
+  t.false(stat.isFIFO());
+});
+
 ///////////
 // files //
 ///////////