Skip to content

Commit

Permalink
test: fix test-http2-socket-close.js
Browse files Browse the repository at this point in the history
Fixes: #54819
PR-URL: #54900
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
  • Loading branch information
huseyinacacak-janea authored Sep 14, 2024
1 parent efbba60 commit c6f514c
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 0 additions & 2 deletions test/parallel/parallel.status
Original file line number Diff line number Diff line change
Expand Up @@ -30,8 +30,6 @@ test-runner-watch-mode-complex: PASS, FLAKY
test-async-context-frame: PASS, FLAKY
# https://github.com/nodejs/node/issues/54534
test-runner-run-watch: PASS, FLAKY
# https://github.com/nodejs/node/issues/54819
test-http2-socket-close: PASS, FLAKY

# Windows on ARM
[$system==win32 && $arch==arm64]
Expand Down
2 changes: 2 additions & 0 deletions test/parallel/test-http2-socket-close.js
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@ netServer.listen(0, common.mustCall(() => {
rejectUnauthorized: false
});

proxyClient.on('error', () => {});
proxyClient.on('close', common.mustCall(() => {
netServer.close();
}));
Expand All @@ -51,6 +52,7 @@ netServer.listen(0, common.mustCall(() => {
':path': '/'
});

req.on('error', () => {});
req.on('response', common.mustCall((response) => {
assert.strictEqual(response[':status'], 200);

Expand Down

0 comments on commit c6f514c

Please sign in to comment.