Skip to content

Commit 10413d6

Browse files
committed
fixup! fs: add throwIfNoEntry option to fs.lstat/fs.promises.lstat
honey, I broke the linter
1 parent 46eebb6 commit 10413d6

1 file changed

Lines changed: 6 additions & 9 deletions

File tree

test/parallel/test-fs-stat.js

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -227,20 +227,17 @@ fs.lstat(__filename, undefined, common.mustCall());
227227
const path = 'does_not_exist';
228228
const opts = { throwIfNoEntry: false };
229229

230-
assert.strictEqual(fs.statSync(path, opts), undefined);
231-
assert.strictEqual(fs.lstatSync(path, opts), undefined);
232-
233-
/* eslint-disable node-core/must-call-assert */
234-
235230
const assertResult = (stats) => {
231+
// eslint-disable-next-line node-core/must-call-assert
236232
assert.strictEqual(stats, undefined);
237233
};
238234

235+
assertResult(fs.statSync(path, opts));
236+
assertResult(fs.lstatSync(path, opts));
237+
239238
fs.stat(path, opts, common.mustSucceed(assertResult));
240239
fs.lstat(path, opts, common.mustSucceed(assertResult));
241240

242-
fs.promises.stat(path, opts).then(common.mustCall(assertResult));
243-
fs.promises.lstat(path, opts).then(common.mustCall(assertResult));
244-
245-
/* eslint-enable node-core/must-call-assert */
241+
fs.promises.stat(path, opts).then(assertResult).then(common.mustCall());
242+
fs.promises.lstat(path, opts).then(assertResult).then(common.mustCall());
246243
}

0 commit comments

Comments
 (0)