From 6654796859c38599eb9069539f5c9eee77db73e1 Mon Sep 17 00:00:00 2001 From: Anatoli Papirovski Date: Mon, 4 Jun 2018 17:38:23 +0000 Subject: [PATCH] test: remove unref in http2 test The bug referenced in this TODO was fixed and this test no longer requires this code to pass. --- test/parallel/test-stream-pipeline.js | 6 ------ 1 file changed, 6 deletions(-) diff --git a/test/parallel/test-stream-pipeline.js b/test/parallel/test-stream-pipeline.js index b52d60529b0332..12733d88a7ac85 100644 --- a/test/parallel/test-stream-pipeline.js +++ b/test/parallel/test-stream-pipeline.js @@ -281,12 +281,6 @@ common.crashOnUnhandledRejection(); }); pipeline(rs, req, common.mustCall((err) => { - // TODO: this is working around an http2 bug - // where the client keeps the event loop going - // (replacing the rs.destroy() with req.end() - // exits it so seems to be a destroy bug there - client.unref(); - server.close(); client.close(); }));