process: swallow stdout/stderr error events

This is a step in making `console` safe, swallows EPIPE and other
errors by registering an empty 'error' event listener.

Also fixes writes to stdout/stderr not working after `end()` is called,
even though we already override `destroy()` so the stream is never
actually closed.

Refs: nodejs#831
Fixes: nodejs#947
Fixes: nodejs#9403

lib/internal/process/stdio.js

@@ -8,26 +8,64 @@ function setupStdio() {
   function getStdout() {
     if (stdout) return stdout;
     stdout = createWritableStdioStream(1);
+
+    Object.defineProperty(stdout._writableState, 'ended', {
+      set() {},
+      get() {
+        return false;
+      }
+    });
+    // XXX(Fishrock123): This should work I think, but it causes
+    // problems with some child-process related tests.
+    //
+    // Object.defineProperty(stdout, 'destroyed', {
+    //   set() {},
+    //   get() {
+    //     return true;
+    //   }
+    // });
+
     stdout.destroy = stdout.destroySoon = function(er) {
+      stdout.destroyed = true;
       er = er || new Error('process.stdout cannot be closed.');
       stdout.emit('error', er);
     };
     if (stdout.isTTY) {
       process.on('SIGWINCH', () => stdout._refreshSize());
     }
+    stdout.on('error', () => {});
     return stdout;
   }
 
   function getStderr() {
     if (stderr) return stderr;
     stderr = createWritableStdioStream(2);
+
+    Object.defineProperty(stderr._writableState, 'ended', {
+      set() {},
+      get() {
+        return false;
+      }
+    });
+    // XXX(Fishrock123): This should work I think, but it causes
+    // problems with some child-process related tests.
+    //
+    // Object.defineProperty(stderr, 'destroyed', {
+    //   set() {},
+    //   get() {
+    //     return true;
+    //   }
+    // });
+
     stderr.destroy = stderr.destroySoon = function(er) {
+      stderr.destroyed = true;
       er = er || new Error('process.stderr cannot be closed.');
       stderr.emit('error', er);
     };
     if (stderr.isTTY) {
       process.on('SIGWINCH', () => stderr._refreshSize());
     }
+    stderr.on('error', () => {});
     return stderr;
   }
 

test/parallel/test-stdout-cannot-be-closed-child-process-pipe.js

@@ -24,9 +24,9 @@ function parent() {
   });
 
   child.on('close', function(code, signal) {
-    assert(code);
+    assert.strictEqual(code, 0, 'exit code should be zero');
     assert.equal(out, 'foo');
-    assert(/process\.stdout cannot be closed/.test(err));
+    assert.strictEqual(err, '', 'stderr should be empty');
     console.log('ok');
   });
 }

test/pseudo-tty/test-tty-stdout-end.js

@@ -1,14 +1,9 @@
 'use strict';
-require('../common');
+const common = require('../common');
 const assert = require('assert');
 
-const shouldThrow = function() {
-  process.stdout.end();
-};
-
-const validateError = function(e) {
-  return e instanceof Error &&
-    e.message === 'process.stdout cannot be closed.';
-};
-
-assert.throws(shouldThrow, validateError);
+process.stdout.on('error', common.mustCall(function(e) {
+  assert(e instanceof Error, 'e is an Error');
+  assert.strictEqual(e.message, 'process.stdout cannot be closed.');
+}));
+process.stdout.end();