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 <starkwang@126.com>
Reviewed-By: Yorkie Liu <yorkiefixer@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ouyang Yadong <oyydoibh@gmail.com>
  • Loading branch information
luoyu authored and targos committed Apr 29, 2019
1 parent e8d5b62 commit 9fa5ba8
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 9fa5ba8

Please sign in to comment.