Commit be90817
test: remove common.port from test-tls-securepair-client
OpenSSL s_server accepts port 0 as an indicator to use an open port
provided by the operating system. Use that instead of common.PORT in the
test.
Remove 500ms delay added in 8e46167.
Hopefully the race condition in OpenSSL s_server has been fixed and/or
the change to port 0 means that the server is listening by the time
the ACCEPT text is printed and the setTimeout() is no longer necessary.
PR-URL: #32024
Reviewed-By: Ben Coe <bencoe@gmail.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>1 parent 6b44df2 commit be90817Copy full SHA for be90817
File tree
Expand file treeCollapse file tree
1 file changed
+7
-6
lines changedOpen diff view settings
Filter options
- test/sequential
Expand file treeCollapse file tree
1 file changed
+7
-6
lines changedOpen diff view settings
Collapse file
test/sequential/test-tls-securepair-client.js
Copy file name to clipboardExpand all lines: test/sequential/test-tls-securepair-client.js+7-6Lines changed: 7 additions & 6 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
62 | 62 | |
63 | 63 | |
64 | 64 | |
65 | | - |
| 65 | + |
66 | 66 | |
67 | 67 | |
68 | 68 | |
| ||
78 | 78 | |
79 | 79 | |
80 | 80 | |
81 | | - |
82 | | - |
83 | | - |
| 81 | + |
| 82 | + |
| 83 | + |
84 | 84 | |
| 85 | + |
85 | 86 | |
86 | 87 | |
87 | 88 | |
| ||
117 | 118 | |
118 | 119 | |
119 | 120 | |
120 | | - |
| 121 | + |
121 | 122 | |
122 | 123 | |
123 | 124 | |
| ||
131 | 132 | |
132 | 133 | |
133 | 134 | |
134 | | - |
| 135 | + |
135 | 136 | |
136 | 137 | |
137 | 138 | |
|
0 commit comments