@util.js/node-files 0.41.5 → 0.41.6

package/lib/Files.js

@@ -92,8 +92,22 @@ module.exports = class Files {
     return path().dirname(...arguments);
   }
 
-  exists() {
-    return promises().applyCallback(fs(), fs().exists, arguments);
+  exists(path, callback) {
+    if (callback) {
+      return fs().exists(path, callback);
+    }
+    return new Promise((resolve, reject) => {
+      // From https://nodejs.org/api/fs.html#fsexistspath-callback:
+      // The parameters for this callback are not consistent with other Node.js callbacks.
+      // Normally, the first parameter to a Node.js callback is an err parameter, optionally 
+      // followed by other parameters. The fs.exists() callback has only one boolean 
+      // parameter. 
+      try {
+        fs().exists(path, resolve);
+      } catch (err) {
+        reject(err);
+      }
+    });
   }
 
   existsSync() {

package/package.json

@@ -1,9 +1,9 @@
 {
   "author": "Chris Topher <c@creemama.com>",
   "dependencies": {
-    "@util.js/objects": "0.41.5",
-    "@util.js/promises": "0.41.5",
-    "@util.js/strings": "0.41.5",
+    "@util.js/objects": "0.41.6",
+    "@util.js/promises": "0.41.6",
+    "@util.js/strings": "0.41.6",
     "graceful-fs": "4.2.11",
     "mkdirp": "3.0.1",
     "ncp": "2.0.0",
@@ -38,5 +38,5 @@
     "type": "git",
     "url": "https://github.com/creemama/utiljs/tree/master/packages/utiljs-node-files"
   },
-  "version": "0.41.5"
+  "version": "0.41.6"
 }

package/test/Files_exists_Test.js

@@ -0,0 +1,31 @@
+"use strict";
+
+const { expect } = require("chai");
+const files = require("..");
+
+describe("Files#exists", () => {
+  const targetDir = files.join(__dirname, "..", "target");
+  files.mkdirpSync(targetDir);
+
+  it("should return true for an existing file via a callback", (done) => {
+    files.exists(targetDir, (exists) => {
+      expect(exists).to.be.true;
+      done();
+    });
+  });
+
+  it("should return true for an existing file via a Promise", async () => {
+    expect(await files.exists(targetDir)).to.be.true;
+  });
+
+  it("should return false for a nonexistent file via a callback", (done) => {
+    files.exists(targetDir + "-nonexistent", (exists) => {
+      expect(exists).to.be.false;
+      done();
+    });
+  });
+
+  it("should return false for an nonexistent file via a Promise", async () => {
+    expect(await files.exists(targetDir + "-nonexistent")).to.be.false;
+  });
+});