From 3a2263fe7c244325748820840fb3084630f75705 Mon Sep 17 00:00:00 2001 From: Mike Bayer Date: Wed, 06 Mar 2019 08:18:26 -0500 Subject: [PATCH] Loosen SingletonThreadPoolTest This test is non-critical as the SingletonThreadPool is not production-level robust under multithreaded scenarios in any case. Fixes: #4527 Change-Id: Ie19ebd69438c97b2d8adb571f8f1b2c56894f7fb --- diff --git a/test/engine/test_pool.py b/test/engine/test_pool.py index 1aadd47..5979388 100644 --- a/test/engine/test_pool.py +++ b/test/engine/test_pool.py @@ -15,6 +15,7 @@ from sqlalchemy.testing import fixtures from sqlalchemy.testing import is_ from sqlalchemy.testing import is_not_ +from sqlalchemy.testing import is_true from sqlalchemy.testing.engines import testing_engine from sqlalchemy.testing.mock import ANY from sqlalchemy.testing.mock import call @@ -1809,7 +1810,9 @@ threads.append(th) for th in threads: th.join(join_timeout) - eq_(len(p._all_conns), 3) + + lp = len(p._all_conns) + is_true(3 <= lp <=4) if strong_refs: still_opened = len([c for c in sr if not c.close.call_count])