test: extend a delay in testing after spawning server processes

awwad · awwad · commit e99ff85b2c33 · 2018-11-05T16:28:59.000-05:00
This adds to PR #799; please see that for details. Signed-off-by: Sebastien Awwad <sebastien.awwad@gmail.com>
diff --git a/tests/test_multiple_repositories_integration.py b/tests/test_multiple_repositories_integration.py
@@ -148,9 +148,13 @@ def setUp(self):
     self.url = 'http://localhost:' + str(self.SERVER_PORT) + os.path.sep
     self.url2 = 'http://localhost:' + str(self.SERVER_PORT2) + os.path.sep
 
-    # NOTE: Following error is raised if a delay is not applied:
+    # NOTE: Following error is raised if a delay is not long enough:
     # <urlopen error [Errno 111] Connection refused>
-    time.sleep(.3)
+    # or, on Windows:
+    # Failed to establish a new connection: [Errno 111] Connection refused'
+    # 0.3s led to occasional failures in automated builds, primarily on
+    # AppVeyor, so increasing this to 2s, sadly.
+    time.sleep(2)
 
     url_prefix = 'http://localhost:' + str(self.SERVER_PORT)
     url_prefix2 = 'http://localhost:' + str(self.SERVER_PORT2)
