test: fix test-net-persistent-keepalive for AIX
authorImran Iqbal <imrani@ca.ibm.com>
Tue, 3 Nov 2015 16:10:16 +0000 (11:10 -0500)
committerJames M Snell <jasnell@gmail.com>
Wed, 23 Dec 2015 16:38:33 +0000 (08:38 -0800)
commite85f0c7a6dfca8055aef92cb9063c3ffd3c49f9f
treea049d29ccad8da4904e093767f8fab726ba5cb3d
parentac36f4342b4f9cd1402e8cb48a9ca933bf2aee25
test: fix test-net-persistent-keepalive for AIX

Fixed an intermittent issue on AIX where the 600ms timeout was reached
before the 'connection' event was fired. This resulted in a failure as
serverConnection would be undefined and the assert.equal would throw an
error. Changed the flow of the test so that the timeout is only set
after a connection has been made.

PR-URL: https://github.com/nodejs/node/pull/3646
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
test/parallel/test-net-persistent-keepalive.js