From 0ece7cc227040f769f015fb28e8b30bfdbc589ee Mon Sep 17 00:00:00 2001
From: Ujjwal Sharma <usharma1998@gmail.com>
Date: Tue, 6 Mar 2018 20:00:38 +0530
Subject: [PATCH] test: rename test-regress-GH-877.js

Rename the test appropriately alongside mentioning the subsystem
Also, make a few basic changes to make sure the test conforms
to the standard test structure

Refs: https://github.com/nodejs/node/issues/19105
Refs: https://github.com/nodejs/node/blob/master/doc/guides/writing-tests.md#test-structure

PR-URL: https://github.com/nodejs/node/pull/19161
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
---
 .../{test-regress-GH-877.js => test-http-max-sockets.js}      | 4 ++++
 1 file changed, 4 insertions(+)
 rename test/sequential/{test-regress-GH-877.js => test-http-max-sockets.js} (94%)

diff --git a/test/sequential/test-regress-GH-877.js b/test/sequential/test-http-max-sockets.js
similarity index 94%
rename from test/sequential/test-regress-GH-877.js
rename to test/sequential/test-http-max-sockets.js
index c717c2dd74f444..24629448f7ac77 100644
--- a/test/sequential/test-regress-GH-877.js
+++ b/test/sequential/test-http-max-sockets.js
@@ -21,6 +21,10 @@
 
 'use strict';
 const common = require('../common');
+
+// Make sure http server doesn't wait for socket pool to establish connections
+// https://github.com/nodejs/node-v0.x-archive/issues/877
+
 const http = require('http');
 const assert = require('assert');