test: deflake test-http-many-ended-pipelines

lpinca · lpinca · commit cc4ee6cba87e · 2021-04-08T19:29:56.000+02:00
The socket might be destroyed by the other peer while data is still being written. Add the missing error handler. PR-URL: #38018 Fixes: #37291 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
diff --git a/test/parallel/test-http-many-ended-pipelines.js b/test/parallel/test-http-many-ended-pipelines.js
@@ -21,6 +21,7 @@
 
 'use strict';
 const common = require('../common');
+const assert = require('assert');
 const http = require('http');
 const net = require('net');
 
@@ -43,6 +44,14 @@ const server = http.createServer(function(req, res) {
 server.listen(0, function() {
   const client = net.connect({ port: this.address().port,
                                allowHalfOpen: true });
+
+  client.on('error', function(err) {
+    // The socket might be destroyed by the other peer while data is still
+    // being written. The `'EPIPE'` and `'ECONNABORTED'` codes might also be
+    // valid but they have not been seen yet.
+    assert.strictEqual(err.code, 'ECONNRESET');
+  });
+
   for (let i = 0; i < numRequests; i++) {
     client.write('GET / HTTP/1.1\r\n' +
                  'Host: some.host.name\r\n' +
