Skip to content

Commit

Permalink
test: fix flaky test-child-process-fork-net
Browse files Browse the repository at this point in the history
Reduce client connections from 10 to 4 in a test that is causing issues
on Raspberry Pi 2 devices in CI.

Fixes: #5122
PR-URL: #6138
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
  • Loading branch information
Trott authored and Myles Borins committed May 18, 2016
1 parent a97a6a9 commit 69dcbb6
Showing 1 changed file with 8 additions and 8 deletions.
16 changes: 8 additions & 8 deletions test/parallel/test-child-process-fork-net.js
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
'use strict';
var assert = require('assert');
var common = require('../common');
var fork = require('child_process').fork;
var net = require('net');
const assert = require('assert');
const common = require('../common');
const fork = require('child_process').fork;
const net = require('net');

// progress tracker
function ProgressTracker(missing, callback) {
Expand Down Expand Up @@ -76,9 +76,9 @@ if (process.argv[2] === 'child') {
server.close();
});

// we expect 10 connections and close events
var connections = new ProgressTracker(10, progress.done.bind(progress));
var closed = new ProgressTracker(10, progress.done.bind(progress));
// we expect 4 connections and close events
var connections = new ProgressTracker(4, progress.done.bind(progress));
var closed = new ProgressTracker(4, progress.done.bind(progress));

// create server and send it to child
var server = net.createServer();
Expand All @@ -99,7 +99,7 @@ if (process.argv[2] === 'child') {
if (msg.what === 'listening') {
// make connections
var socket;
for (var i = 0; i < 10; i++) {
for (var i = 0; i < 4; i++) {
socket = net.connect(common.PORT, function() {
console.log('CLIENT: connected');
});
Expand Down

0 comments on commit 69dcbb6

Please sign in to comment.