From 7e27c8be2c2632f461287db067dc11fdd8f8d834 Mon Sep 17 00:00:00 2001 From: Jeffrey Espiritu Date: Tue, 9 Oct 2018 13:04:19 -0400 Subject: [PATCH] Fix Issue #400 - add test for passing file to fs.promise.readdir() (#409) * Fix Issue #400 - add test for passing file to fs.promise.readdir() This adds a test to ensure that `fs.promise.readdir()` throws an error when passing in a file. * Made changes to `fsPromises.readdir` to be more promise-friendly --- tests/spec/fs.readdir.spec.js | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+) diff --git a/tests/spec/fs.readdir.spec.js b/tests/spec/fs.readdir.spec.js index 0e9a1ec..da8d175 100644 --- a/tests/spec/fs.readdir.spec.js +++ b/tests/spec/fs.readdir.spec.js @@ -55,3 +55,24 @@ describe('fs.readdir', function() { }); }); }); + +/** + * fsPromises tests + */ + +describe('fsPromises.readdir', function() { + beforeEach(util.setup); + afterEach(util.cleanup); + + it('should return an error if the path is a file', function() { + var fsPromises = util.fs().promises; + + return fsPromises.open('/myfile', 'w') + .then(fd => fsPromises.close(fd)) + .then(() => fsPromises.readdir('/myfile')) + .catch(error => { + expect(error).to.exist; + expect(error.code).to.equal('ENOTDIR'); + }); + }); +}); \ No newline at end of file