Commit 99104e1
test: improve fs.exists coverage
By adding a test case using a path with illegal protocol
PR-URL: #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>1 parent e54f75b commit 99104e1
1 file changed
Lines changed: 5 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -23,6 +23,7 @@ | |||
| 23 | 23 | const common = require('../common'); | |
| 24 | 24 | const assert = require('assert'); | |
| 25 | 25 | const fs = require('fs'); | |
| 26 | + const { URL } = require('url'); | ||
| 26 | 27 | const f = __filename; | |
| 27 | 28 | ||
| 28 | 29 | fs.exists(f, common.mustCall(function(y) { | |
@@ -33,5 +34,9 @@ fs.exists(`${f}-NO`, common.mustCall(function(y) { | |||
| 33 | 34 | assert.strictEqual(y, false); | |
| 34 | 35 | })); | |
| 35 | 36 | ||
| 37 | + fs.exists(new URL('https://foo'), common.mustCall(function(y) { | ||
| 38 | + assert.strictEqual(y, false); | ||
| 39 | + })); | ||
| 40 | + | ||
| 36 | 41 | assert(fs.existsSync(f)); | |
| 37 | 42 | assert(!fs.existsSync(`${f}-NO`)); | |
0 commit comments