test: fix http2 connection abort test

Refs: https://github.com/nodejs/node/issues/21836
Refs: https://github.com/nodejs/node/pull/21561

PR-URL: https://github.com/nodejs/node/pull/21861
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Minwoo Jung <minwoo@nodesource.com>

1 files changed, 3 insertions, 8 deletions

diff --git a/test/parallel/test-http2-respond-with-file-connection-abort.js b/test/parallel/test-http2-respond-with-file-connection-abort.js
index 25926b2c98..6c8337ba8b 100644
--- a/test/parallel/test-http2-respond-with-file-connection-abort.js
+++ b/test/parallel/test-http2-respond-with-file-connection-abort.js
@@ -3,6 +3,7 @@
 const common = require('../common');
 if (!common.hasCrypto)
   common.skip('missing crypto');
+const assert = require('assert');
 const http2 = require('http2');
 const net = require('net');
 
@@ -12,6 +13,7 @@ const {
 
 const server = http2.createServer();
 server.on('stream', common.mustCall((stream) => {
+  stream.on('error', (err) => assert.strictEqual(err.code, 'ECONNRESET'));
   stream.respondWithFile(process.execPath, {
     [HTTP2_HEADER_CONTENT_TYPE]: 'application/octet-stream'
   });
@@ -22,16 +24,9 @@ server.listen(0, common.mustCall(() => {
   const req = client.request();
 
   req.on('response', common.mustCall(() => {}));
-  req.on('data', common.mustCall(() => {
+  req.on('data', common.mustCallAtLeast(() => {
     net.Socket.prototype.destroy.call(client.socket);
     server.close();
   }));
   req.end();
 }));
-
-// TODO(addaleax): This is a *hack*. HTTP/2 needs to have a proper way of
-// dealing with this kind of issue.
-process.once('uncaughtException', (err) => {
-  if (err.code === 'ECONNRESET') return;
-  throw err;
-});