Skip to content

Commit

Permalink
test: remove extra process.exit()
Browse files Browse the repository at this point in the history
process.exit() has a tendency to hide test failures because
it forces the process to exit. This test doesn't need the
process.exit(), so this commit removes it.

PR-URL: #29873
Reviewed-By: Jeremiah Senkpiel <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Yongsheng Zhang <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
  • Loading branch information
cjihrig authored and BridgeAR committed Oct 10, 2019
1 parent 6a5d401 commit 5fdf4a4
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion test/parallel/test-http2-connect-options.js
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,6 @@ server.listen(0, '127.0.0.1', common.mustCall(() => {
client.close();
req.close();
server.close();
process.exit();
}));
req.end();
}));

0 comments on commit 5fdf4a4

Please sign in to comment.