Skip to content

Commit e96e3f9

Browse files
committed
test: remove redundant common.mustCall
All calls verify the cb is called, thus using `common.mustCall` is not required inside of this function. PR-URL: #26738 Reviewed-By: Gus Caplan <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Joyee Cheung <[email protected]>
1 parent 3fe1e80 commit e96e3f9

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

test/parallel/test-fs-readfile-error.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -38,12 +38,12 @@ function test(env, cb) {
3838
const filename = fixtures.path('test-fs-readfile-error.js');
3939
const execPath = `"${process.execPath}" "${filename}"`;
4040
const options = { env: Object.assign({}, process.env, env) };
41-
exec(execPath, options, common.mustCall((err, stdout, stderr) => {
41+
exec(execPath, options, (err, stdout, stderr) => {
4242
assert(err);
4343
assert.strictEqual(stdout, '');
4444
assert.notStrictEqual(stderr, '');
4545
cb(String(stderr));
46-
}));
46+
});
4747
}
4848

4949
test({ NODE_DEBUG: '' }, common.mustCall((data) => {

0 commit comments

Comments
 (0)