diff options
| -rw-r--r-- | changes/ticket28883 | 4 | ||||
| -rw-r--r-- | src/test/test_rebind.py | 6 |
2 files changed, 6 insertions, 4 deletions
diff --git a/changes/ticket28883 b/changes/ticket28883 new file mode 100644 index 0000000000..1d8b6cb416 --- /dev/null +++ b/changes/ticket28883 @@ -0,0 +1,4 @@ + o Minor bugfixes (testing): + - Make sure that test_rebind.py actually obeys its timeout, even + when it receives a large number of log messages. Fixes bug 28883; + bugfix on 0.3.5.4-alpha. diff --git a/src/test/test_rebind.py b/src/test/test_rebind.py index 6f759b717e..00e5a08be7 100644 --- a/src/test/test_rebind.py +++ b/src/test/test_rebind.py @@ -11,7 +11,6 @@ import time LOG_TIMEOUT = 60.0 LOG_WAIT = 0.1 -LOG_CHECK_LIMIT = LOG_TIMEOUT / LOG_WAIT def fail(msg): logging.error('FAIL') @@ -25,8 +24,8 @@ def try_connecting_to_socksport(): socks_socket.close() def wait_for_log(s): - log_checked = 0 - while log_checked < LOG_CHECK_LIMIT: + cutoff = time.time() + LOG_TIMEOUT + while time.time() < cutoff: l = tor_process.stdout.readline() l = l.decode('utf8') if s in l: @@ -37,7 +36,6 @@ def wait_for_log(s): # avoid busy-waiting if len(s) == 0: time.sleep(LOG_WAIT) - log_checked += 1 fail('Could not find "{}" in logs after {} seconds'.format(s, LOG_TIMEOUT)) def pick_random_port(): |