From 0b0370f8841c4c399ddeddea1c63ff80137971da 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. PR-URL: https://github.com/nodejs/node/pull/21145 Reviewed-By: Rich Trott --- 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(); }));