From 6462779230598c5e92ab6bdfb1b60cd38d1db80d Mon Sep 17 00:00:00 2001 From: Martin Vrachev Date: Thu, 22 Oct 2020 16:41:02 +0300 Subject: [PATCH] Remove sleep from test_slow_retrieval_attack.py Now, after we can use wait_for_server and the retry mechanism of TestServerProcess in utils.py we no longer need to use sleep in this test file. Signed-off-by: Martin Vrachev --- tests/test_slow_retrieval_attack.py | 8 -------- 1 file changed, 8 deletions(-) diff --git a/tests/test_slow_retrieval_attack.py b/tests/test_slow_retrieval_attack.py index a1dddaae..4b9943bf 100755 --- a/tests/test_slow_retrieval_attack.py +++ b/tests/test_slow_retrieval_attack.py @@ -172,14 +172,6 @@ def setUp(self): logger.info('Slow Retrieval Server process started.') - # NOTE: Following error is raised if a delay is not long enough: - # - # or, on Windows: - # Failed to establish a new connection: [Errno 111] Connection refused' - # 1s led to occasional failures in automated builds on AppVeyor, so - # increasing this to 3s, sadly. - time.sleep(3) - url_prefix = 'http://localhost:' \ + str(self.server_process_handler.port) + repository_basepath