summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMike Bayer <mike_mp@zzzcomputing.com>2021-04-22 14:33:36 -0400
committerMike Bayer <mike_mp@zzzcomputing.com>2021-04-22 14:33:36 -0400
commitf4e93067a116b748242e62eb524b2a87490ec18c (patch)
treea05980c895a8ec66c89ab1adadf62460bf12f95d
parent72f43a8b49803cd4d4befe91635f6691965f34fb (diff)
downloadsqlalchemy-f4e93067a116b748242e62eb524b2a87490ec18c.tar.gz
Ensure sparse backend selection sorts on driver also
Because we have --dbdriver now, we usually have lots of db drivers queued up which affect the selection of backend tests. the sort in possible_configs_for_cls() is used so that when we are only picking some backends due to the sparse flag, we get the same answer each time otherwise pytest-xdist raises an error if two workers have different tests. adding the sort for driver fixes an issue on jenkins master build where the memusage test run frequently failed due to mismatched test collection. Change-Id: I8584c1a3a4e5b759c0e5a2602a93432d9cc8168d
-rw-r--r--lib/sqlalchemy/testing/plugin/plugin_base.py1
1 files changed, 1 insertions, 0 deletions
diff --git a/lib/sqlalchemy/testing/plugin/plugin_base.py b/lib/sqlalchemy/testing/plugin/plugin_base.py
index 858814f91..6370d7ce9 100644
--- a/lib/sqlalchemy/testing/plugin/plugin_base.py
+++ b/lib/sqlalchemy/testing/plugin/plugin_base.py
@@ -688,6 +688,7 @@ def _possible_configs_for_cls(cls, reasons=None, sparse=False):
all_configs,
key=lambda cfg: (
cfg.db.name,
+ cfg.db.driver,
cfg.db.dialect.server_version_info,
),
)