Skip to content

Commit ee498cd

Browse files
ezelkow1zwoop
authored andcommitted
Add ready checks for the microserver and ATS test instances. (#6625)
This fixed failing autest runs for nghttp on 8.1.x, adding to master first (cherry picked from commit 21f6a70)
1 parent 8da08d1 commit ee498cd

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

tests/gold_tests/h2/nghttp.test.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@
8080
tr.Processes.Default.Command = "nghttp -v --no-dep 'https://127.0.0.1:{0}/post' --trailer 'foo: bar' -d 'post_body'".format(
8181
ts.Variables.ssl_port)
8282
tr.Processes.Default.ReturnCode = 0
83-
tr.Processes.Default.StartBefore(microserver)
84-
tr.Processes.Default.StartBefore(Test.Processes.ts)
83+
tr.Processes.Default.StartBefore(microserver, ready=When.PortOpen(microserver.Variables.Port))
84+
tr.Processes.Default.StartBefore(Test.Processes.ts, ready=When.PortOpen(ts.Variables.ssl_port))
8585
tr.Processes.Default.Streams.stdout = "gold/nghttp_0_stdout.gold"
8686
tr.StillRunningAfter = microserver

0 commit comments

Comments
 (0)