http: add test for incomingmessage destroy

Test uncaught exceptions when destroying IncomingMessage.

PR-URL: #33035
Refs: #30625
Reviewed-By: Robert Nagy <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Rich Trott <[email protected]>

Author: dnlup

lib/_http_incoming.js

@@ -361,14 +361,13 @@ IncomingMessage.prototype._dump = function _dump() {
   }
 };
 
-function onError(instance, error, cb) {
+function onError(self, error, cb) {
   // This is to keep backward compatible behavior.
-  // An error is emitted only if there are listeners attached to
-  // the event.
-  if (instance.listenerCount('error') > 0) {
-    cb(error);
-  } else {
+  // An error is emitted only if there are listeners attached to the event.
+  if (self.listenerCount('error') === 0) {
     cb();
+  } else {
+    cb(error);
   }
 }
 

test/parallel/test-http-client-incomingmessage-destroy.js

@@ -0,0 +1,25 @@
+'use strict';
+
+const common = require('../common');
+const { createServer, get } = require('http');
+const assert = require('assert');
+
+const server = createServer(common.mustCall((req, res) => {
+  res.writeHead(200);
+  res.write('Part of res.');
+}));
+
+function onUncaught(error) {
+  assert.strictEqual(error.message, 'Destroy test');
+  server.close();
+}
+
+process.on('uncaughtException', common.mustCall(onUncaught));
+
+server.listen(0, () => {
+  get({
+    port: server.address().port
+  }, common.mustCall((res) => {
+    res.destroy(new Error('Destroy test'));
+  }));
+});

test/parallel/test-http-server-incomingmessage-destroy.js

@@ -0,0 +1,25 @@
+'use strict';
+
+const common = require('../common');
+const { createServer, get } = require('http');
+const assert = require('assert');
+
+const server = createServer(common.mustCall((req, res) => {
+  req.destroy(new Error('Destroy test'));
+}));
+
+function onUncaught(error) {}
+
+process.on('uncaughtException', common.mustNotCall(onUncaught));
+
+server.listen(0, common.mustCall(() => {
+  get({
+    port: server.address().port
+  }, (res) => {
+    res.resume();
+  }).on('error', (error) => {
+    assert.strictEqual(error.message, 'socket hang up');
+    assert.strictEqual(error.code, 'ECONNRESET');
+    server.close();
+  });
+}));