Skip to content

Commit

Permalink
test: move test-http-max-sockets to parallel
Browse files Browse the repository at this point in the history
PR-URL: #54977
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Jake Yuesong Li <jake.yuesong@gmail.com>
  • Loading branch information
lpinca authored and targos committed Oct 4, 2024
1 parent 22b4139 commit 5b012f5
Showing 1 changed file with 5 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
// USE OR OTHER DEALINGS IN THE SOFTWARE.

'use strict';
const common = require('../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
Expand All @@ -40,13 +40,14 @@ const server = http.createServer(function(req, res) {
res.end('Hello World\n');
});

const addrString = agent.getName({ host: '127.0.0.1', port: common.PORT });
server.listen(0, '127.0.0.1', function() {
const { port } = server.address();
const addrString = agent.getName({ host: '127.0.0.1', port });

server.listen(common.PORT, '127.0.0.1', function() {
for (let i = 0; i < N; i++) {
const options = {
host: '127.0.0.1',
port: common.PORT
port
};

const req = http.get(options, function(res) {
Expand Down

0 comments on commit 5b012f5

Please sign in to comment.