diff options
author | Matteo Rossi <teorossi82@yahoo.it> | 2019-10-14 17:39:39 +0200 |
---|---|---|
committer | Rich Trott <rtrott@gmail.com> | 2019-10-16 09:52:07 -0700 |
commit | e22efba812b2a6c2ee6d35f4e11af5b08afd881d (patch) | |
tree | 84df82ac08104676e12f9cbab081cca758261320 | |
parent | 46648eb55d141ff050cbce8595402e5626f30b99 (diff) | |
download | android-node-v8-e22efba812b2a6c2ee6d35f4e11af5b08afd881d.tar.gz android-node-v8-e22efba812b2a6c2ee6d35f4e11af5b08afd881d.tar.bz2 android-node-v8-e22efba812b2a6c2ee6d35f4e11af5b08afd881d.zip |
test: add cb error test for fs.close()
Provides some missing test coverage.
PR-URL: https://github.com/nodejs/node/pull/29970
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
-rw-r--r-- | test/parallel/test-fs-close-errors.js | 16 |
1 files changed, 16 insertions, 0 deletions
diff --git a/test/parallel/test-fs-close-errors.js b/test/parallel/test-fs-close-errors.js index 42d990410f..6168d5c20a 100644 --- a/test/parallel/test-fs-close-errors.js +++ b/test/parallel/test-fs-close-errors.js @@ -17,3 +17,19 @@ const fs = require('fs'); assert.throws(() => fs.close(input), errObj); assert.throws(() => fs.closeSync(input), errObj); }); + +{ + // Test error when cb is not a function + const fd = fs.openSync(__filename, 'r'); + + const errObj = { + code: 'ERR_INVALID_CALLBACK', + name: 'TypeError' + }; + + ['', false, null, {}, []].forEach((input) => { + assert.throws(() => fs.close(fd, input), errObj); + }); + + fs.closeSync(fd); +} |