3adda4b2adThis adds an ESLint rule to enforce the use of `assert.ifError(err)` instead of `if (err) throw err;` in tests. PR-URL: https://github.com/nodejs/node/pull/10671 Ref: https://github.com/nodejs/node/pull/10543 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: Michal Zasso <targos@protonmail.com>
33 lines
899 B
JavaScript
33 lines
899 B
JavaScript
'use strict';
|
|
const common = require('../common');
|
|
const assert = require('assert');
|
|
|
|
// simulate `cat readfile.js | node readfile.js`
|
|
|
|
if (common.isWindows || common.isAix) {
|
|
common.skip(`No /dev/stdin on ${process.platform}.`);
|
|
return;
|
|
}
|
|
|
|
const fs = require('fs');
|
|
|
|
const dataExpected = fs.readFileSync(__filename, 'utf8');
|
|
|
|
if (process.argv[2] === 'child') {
|
|
fs.readFile('/dev/stdin', function(er, data) {
|
|
assert.ifError(er);
|
|
process.stdout.write(data);
|
|
});
|
|
return;
|
|
}
|
|
|
|
const exec = require('child_process').exec;
|
|
const f = JSON.stringify(__filename);
|
|
const node = JSON.stringify(process.execPath);
|
|
const cmd = `cat ${f} | ${node} ${f} child`;
|
|
exec(cmd, function(err, stdout, stderr) {
|
|
assert.ifError(err);
|
|
assert.strictEqual(stdout, dataExpected, 'it reads the file and outputs it');
|
|
assert.strictEqual(stderr, '', 'it does not write to stderr');
|
|
console.log('ok');
|
|
});
|