fix(loadFilesFromDir): validating options.cwd

checking if options.cwd is a valid string.

lib/utils.js

@@ -2,16 +2,23 @@
 
 const Glob = require('glob');
 const Hoek = require('hoek');
+const Joi = require('joi');
 const _ = require('lodash');
 
 const internals = {
-    options: {
-        cwd: process.cwd()
-    }
+    options: {}
 };
 
 exports.loadFilesFromDir = (pattern, options) => {
 
+    const validation = Joi.validate(options, Joi.object().keys({
+        cwd: Joi.string()
+    }));
+
+    if (validation.error) {
+        throw new Error(validation.error.annotate());
+    }
+
     const files = Glob.sync(pattern, Hoek.applyToDefaults(internals.options, options));
     return _.map(files, (file) => (`${options.cwd}/${file}`));
 };

package.json

@@ -42,5 +42,8 @@
     "commitizen": {
       "path": "node_modules/cz-conventional-changelog"
     }
+  },
+  "dependencies": {
+    "joi": "10.2.1"
   }
 }

test/utils.js

@@ -26,7 +26,20 @@ Lab.describe(`${Lab.pkg.name}:utils`, () => {
         return done();
     });
 
-    Lab.it('should use fail if not cwd were passed.', (done) => {
+    Lab.it('should return an empty array if not files were found.', (done) => {
+
+        const files = Utils.loadFilesFromDir('*.js');
+
+        Lab.expect(files).to.exist();
+        Lab.expect(files).to.be.an.array();
+        Lab.expect(files).to.have.length(0);
+
+        return done();
+    });
+
+    Lab.it('should fail if an invalid cwd option were passed.', (done) => {
+
+        Lab.expect(() => Utils.loadFilesFromDir('*.js', { cwd: 123 })).to.throw(Error);
 
         return done();
     });