Skip to content
Closed
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
test: ensure _handle property existence
`test-stdtout-close-unref.js` will fail if `process.stdin._handle` does
not exist. On UNIX-like operating systems, you can see this failure this
way:

    ./node test/parallel/test-stdout-close-unref.js < /dev/null

This issue has been experienced by @bengl and @drewfish in a Docker
container. I'm not sure why they are experiencing it in their
environment, but since it is possible that the `_handle` property does
not exist, let's use `child_process.spawn()` to make sure it exists.
  • Loading branch information
Trott committed Mar 31, 2016
commit fa8f31f6f4782b88711abd2b3fcf47d5118ae886
38 changes: 26 additions & 12 deletions test/parallel/test-stdout-close-unref.js
Original file line number Diff line number Diff line change
@@ -1,16 +1,30 @@
'use strict';
require('../common');
var assert = require('assert');
const common = require('../common');
const assert = require('assert');
const spawn = require('child_process').spawn;

var errs = 0;
if (process.argv[2] === 'child') {
var errs = 0;

process.stdin.resume();
process.stdin._handle.close();
process.stdin._handle.unref(); // Should not segfault.
process.stdin.on('error', function(err) {
errs++;
});
process.stdin.resume();
process.stdin._handle.close();
process.stdin._handle.unref(); // Should not segfault.
process.stdin.on('error', function(err) {
errs++;
});

process.on('exit', function() {
assert.strictEqual(errs, 1);
});
process.on('exit', function() {
assert.strictEqual(errs, 1);
});
return;
}

// Use spawn so that we can be sure that stdin has a _handle property.
// Refs: https://github.com/nodejs/node/pull/5916
const proc = spawn(process.execPath, [__filename, 'child'], { stdio: 'pipe' });

proc.stderr.pipe(process.stderr);
proc.on('exit', common.mustCall(function(exitCode) {
if (exitCode !== 0)
process.exitCode = exitCode;
}));