test: fix flaky test-http2-reset-flood

Set `allowHalfOpen: true` in the client. Fixes: #29802 Refs: #31806 PR-URL: #34318 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Robert Nagy <ronagy@icloud.com>
nodejs · Jul 22, 2020 · 9fa53b3 · 9fa53b3
1 parent 8023815
commit 9fa53b3
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 5 deletions.
diff --git a/test/parallel/parallel.status b/test/parallel/parallel.status
@@ -5,8 +5,6 @@ prefix parallel
 # sample-test                        : PASS,FLAKY
 
 [true] # This section applies to all platforms
-# https://github.com/nodejs/node/issues/29802
-test-http2-reset-flood: PASS,FLAKY
 
 [$system==win32]
 # https://github.com/nodejs/node/issues/32863
@@ -46,8 +44,6 @@ test-crypto-keygen: SKIP
 [$system==freebsd]
 # https://github.com/nodejs/node/issues/31727
 test-fs-stat-bigint: PASS,FLAKY
-# https://github.com/nodejs/node/issues/29802
-test-http2-reset-flood: PASS,FLAKY
 # https://github.com/nodejs/node/issues/28803
 test-stdout-close-catch: PASS,FLAKY
 # https://github.com/nodejs/node/issues/31280

diff --git a/test/parallel/test-http2-reset-flood.js b/test/parallel/test-http2-reset-flood.js
@@ -28,7 +28,7 @@ if (process.env.HAS_STARTED_WORKER) {
 process.env.HAS_STARTED_WORKER = 1;
 const worker = new Worker(__filename).on('message', common.mustCall((port) => {
   const h2header = Buffer.alloc(9);
-  const conn = net.connect(port);
+  const conn = net.connect({ port, allowHalfOpen: true });
 
   conn.write('PRI * HTTP/2.0\r\n\r\nSM\r\n\r\n');