Fix: listTree must return an empty array if the directory does not ex…

…ists ... just like q-io/fs
nknapp · Dec 20, 2016 · 71da3f0 · 71da3f0
1 parent 64ce630
commit 71da3f0
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/fs.js b/fs.js
@@ -128,6 +128,10 @@ function walk (directoryPath, filter, collector) {
   var defer = Q.defer()
   fs.stat(directoryPath, function (err, stat) {
     if (err) {
+      if (err.code === 'ENOENT') {
+        // Like q-io/fs, return an empty array, if the directory does not exist
+        return defer.resolve([])
+      }
       return defer.reject(err)
     }
     // Call filter to get result, "true" if no filter is set

diff --git a/test/fs-spec.js b/test/fs-spec.js
@@ -51,6 +51,10 @@ describe('m-io/fs', function () {
   })
 
   describe('the list-tree function', function () {
+    it('should return an empty array if the directory does not exist', function () {
+      return mfs.listTree('test/fixtures/no-directory').should.eventually.deep.equal([])
+    })
+
     it('should return a file listing as array', function () {
       return mfs.listTree('test/fixtures/tree').then(sort).should.eventually.deep.equal([
         'test/fixtures/tree',