Set alarms on child processes in process_test.
These alarms used to be there, but got dropped in a previous change because I had assumed the child processes would inherit the parent's alarm.
This commit is contained in:
parent
efcab59b0b
commit
3451019e2b
|
@ -57,9 +57,11 @@ class ProcessTest(LogTrapTestCase):
|
|||
return "http://127.0.0.1:%d%s" % (port, path)
|
||||
sockets = bind_sockets(port, "127.0.0.1")
|
||||
# ensure that none of these processes live too long
|
||||
signal.alarm(5)
|
||||
signal.alarm(5) # master process
|
||||
try:
|
||||
id = fork_processes(3, max_restarts=3)
|
||||
assert id is not None
|
||||
signal.alarm(5) # child processes
|
||||
except SystemExit, e:
|
||||
# if we exit cleanly from fork_processes, all the child processes
|
||||
# finished with status 0
|
||||
|
|
Loading…
Reference in New Issue