1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: improve fs.exists coverage

Browse Source 
By adding a test case using a path with illegal protocol

PR-URL: https://github.com/nodejs/node/pull/14301
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
This commit is contained in:
jkzing 2017-07-16 16:59:34 +08:00 committed by Tobias Nießen
parent 97c4033ebf
commit 201e756ef1
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
test/parallel/test-fs-exists.js
View File

@ -23,6 +23,7 @@
const common = require('../common'); const common = require('../common');
const assert = require('assert'); const assert = require('assert');
const fs = require('fs'); const fs = require('fs');
const { URL } = require('url');
const f = __filename; const f = __filename;
fs.exists(f, common.mustCall(function(y) { fs.exists(f, common.mustCall(function(y) {
@ -33,5 +34,9 @@ fs.exists(`${f}-NO`, common.mustCall(function(y) {
assert.strictEqual(y, false); assert.strictEqual(y, false);
})); }));
fs.exists(new URL('https://foo'), common.mustCall(function(y) {
assert.strictEqual(y, false);
}));
assert(fs.existsSync(f)); assert(fs.existsSync(f));
assert(!fs.existsSync(`${f}-NO`)); assert(!fs.existsSync(`${f}-NO`));