test: add regression test for #11257
Refs: https://github.com/nodejs/node/issues/11257 PR-URL: https://github.com/nodejs/node/pull/20391 Reviewed-By: Anatoli Papirovski <apapirovski@mac.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
This commit is contained in:
parent
8d3131b0af
commit
d942573393
37
test/parallel/test-stdio-pipe-stderr.js
Normal file
37
test/parallel/test-stdio-pipe-stderr.js
Normal file
@ -0,0 +1,37 @@
|
||||
'use strict';
|
||||
require('../common');
|
||||
const tmpdir = require('../common/tmpdir');
|
||||
const assert = require('assert');
|
||||
const fs = require('fs');
|
||||
const join = require('path').join;
|
||||
const spawn = require('child_process').spawnSync;
|
||||
|
||||
// Test that invoking node with require, and piping stderr to file,
|
||||
// does not result in exception,
|
||||
// see: https://github.com/nodejs/node/issues/11257
|
||||
|
||||
tmpdir.refresh();
|
||||
const fakeModulePath = join(tmpdir.path, 'batman.js');
|
||||
const stderrOutputPath = join(tmpdir.path, 'stderr-output.txt');
|
||||
// we need to redirect stderr to a file to produce #11257
|
||||
const stream = fs.createWriteStream(stderrOutputPath);
|
||||
|
||||
// the error described in #11257 only happens when we require a
|
||||
// non-built-in module.
|
||||
fs.writeFileSync(fakeModulePath, '', 'utf8');
|
||||
|
||||
stream.on('open', () => {
|
||||
spawn(process.execPath, {
|
||||
input: `require("${fakeModulePath.replace(/\\/g, '/')}")`,
|
||||
stdio: ['pipe', 'pipe', stream]
|
||||
});
|
||||
const stderr = fs.readFileSync(stderrOutputPath, 'utf8').trim();
|
||||
assert.strictEqual(
|
||||
stderr,
|
||||
'',
|
||||
`piping stderr to file should not result in exception: ${stderr}`
|
||||
);
|
||||
stream.end();
|
||||
fs.unlinkSync(stderrOutputPath);
|
||||
fs.unlinkSync(fakeModulePath);
|
||||
});
|
Loading…
x
Reference in New Issue
Block a user