Let the O/S supply a port if none of the default ports can be used.

This should make the tests more robust at the expense of allowing
tests to be sloppier by not requiring them to cleanup after themselves.
(It will legitamitely help when running two test suites simultaneously
or if another process is already using one of the predefined ports.)

Also simplifies (slightLy) the exception handling elsewhere.
This commit is contained in:
Neal Norwitz 2007-10-14 00:18:40 +00:00
parent 5d743fd187
commit 880605aafd
1 changed files with 12 additions and 4 deletions

View File

@ -100,10 +100,19 @@ def bind_port(sock, host='', preferred_port=54321):
tests and we don't try multiple ports, the test can fails. This
makes the test more robust."""
# some random ports that hopefully no one is listening on.
for port in [preferred_port, 9907, 10243, 32999]:
# Find some random ports that hopefully no one is listening on.
# Ideally each test would clean up after itself and not continue listening
# on any ports. However, this isn't the case. The last port (0) is
# a stop-gap that asks the O/S to assign a port. Whenever the warning
# message below is printed, the test that is listening on the port should
# be fixed to close the socket at the end of the test.
# Another reason why we can't use a port is another process (possibly
# another instance of the test suite) is using the same port.
for port in [preferred_port, 9907, 10243, 32999, 0]:
try:
sock.bind((host, port))
if port == 0:
port = sock.getsockname()[1]
return port
except socket.error, (err, msg):
if err != errno.EADDRINUSE:
@ -535,8 +544,7 @@ def _run_suite(suite):
elif len(result.failures) == 1 and not result.errors:
err = result.failures[0][1]
else:
msg = "errors occurred; run in verbose mode for details"
raise TestFailed(msg)
err = "errors occurred; run in verbose mode for details"
raise TestFailed(err)