summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatteo Rossi <teorossi82@yahoo.it>2019-10-14 17:39:39 +0200
committerRich Trott <rtrott@gmail.com>2019-10-16 09:52:07 -0700
commite22efba812b2a6c2ee6d35f4e11af5b08afd881d (patch)
tree84df82ac08104676e12f9cbab081cca758261320
parent46648eb55d141ff050cbce8595402e5626f30b99 (diff)
downloadandroid-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.js16
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);
+}