Skip to content

Commit

Permalink
test: add mustCall to test-fs-readfile-pipe-large
Browse files Browse the repository at this point in the history
PR-URL: #27460
Reviewed-By: Weijia Wang <[email protected]>
Reviewed-By: Yorkie Liu <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Ouyang Yadong <[email protected]>
  • Loading branch information
luoyu authored and oyyd committed Apr 28, 2019
1 parent 7cbbd51 commit 54c973c
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/parallel/test-fs-readfile-pipe-large.js
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ const exec = require('child_process').exec;
const f = JSON.stringify(__filename);
const node = JSON.stringify(process.execPath);
const cmd = `cat ${filename} | ${node} ${f} child`;
exec(cmd, { maxBuffer: 1000000 }, function(err, stdout, stderr) {
exec(cmd, { maxBuffer: 1000000 }, common.mustCall((err, stdout, stderr) => {
assert.ifError(err);
assert.strictEqual(
stdout,
Expand All @@ -42,7 +42,7 @@ exec(cmd, { maxBuffer: 1000000 }, function(err, stdout, stderr) {
`expect that it does not write to stderr, but got : ${stderr}`
);
console.log('ok');
});
}));

process.on('exit', function() {
fs.unlinkSync(filename);
Expand Down

0 comments on commit 54c973c

Please sign in to comment.