Skip to content

Commit

Permalink
test: remove redundant common.mustCall
Browse files Browse the repository at this point in the history
All calls verify the cb is called, thus using `common.mustCall` is
not required inside of this function.

PR-URL: nodejs#26738
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
  • Loading branch information
BridgeAR committed Mar 23, 2019
1 parent 3fe1e80 commit e96e3f9
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/parallel/test-fs-readfile-error.js
Original file line number Diff line number Diff line change
Expand Up @@ -38,12 +38,12 @@ function test(env, cb) {
const filename = fixtures.path('test-fs-readfile-error.js');
const execPath = `"${process.execPath}" "${filename}"`;
const options = { env: Object.assign({}, process.env, env) };
exec(execPath, options, common.mustCall((err, stdout, stderr) => {
exec(execPath, options, (err, stdout, stderr) => {
assert(err);
assert.strictEqual(stdout, '');
assert.notStrictEqual(stderr, '');
cb(String(stderr));
}));
});
}

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

0 comments on commit e96e3f9

Please sign in to comment.