Commit f3594e7
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: #3646
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>1 parent 325c4c7 commit f3594e7Copy full SHA for f3594e7
File tree
Expand file treeCollapse file tree
1 file changed
+10
-10
lines changedOpen diff view settings
Filter options
- test/parallel
Expand file treeCollapse file tree
1 file changed
+10
-10
lines changedOpen diff view settings
Collapse file
test/parallel/test-net-persistent-keepalive.js
Copy file name to clipboardExpand all lines: test/parallel/test-net-persistent-keepalive.js+10-10Lines changed: 10 additions & 10 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
4 | 4 | |
5 | 5 | |
6 | 6 | |
| 7 | + |
7 | 8 | |
8 | 9 | |
| 10 | + |
| 11 | + |
| 12 | + |
| 13 | + |
| 14 | + |
| 15 | + |
| 16 | + |
| 17 | + |
9 | 18 | |
10 | 19 | |
11 | 20 | |
| ||
15 | 24 | |
16 | 25 | |
17 | 26 | |
18 | | - |
| 27 | + |
19 | 28 | |
20 | 29 | |
21 | 30 | |
22 | 31 | |
23 | 32 | |
24 | 33 | |
25 | | - |
26 | | - |
27 | | - |
28 | | - |
29 | | - |
30 | | - |
31 | | - |
32 | | - |
33 | | - |
34 | 34 | |
0 commit comments