Skip to content

Commit eec44bd

Browse files
committed
Moved ssl argument handling to cover cluster case
1 parent 47e6c7a commit eec44bd

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

tests/conftest.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
from tests.ssl_utils import get_ssl_filename
1818

1919
REDIS_INFO = {}
20-
default_redis_url = "redis://localhost:6379/0"
20+
default_redis_url = "redis://localhost:16379/0"
2121
default_protocol = "2"
2222
default_redismod_url = "redis://localhost:6479"
2323

@@ -322,10 +322,10 @@ def _get_client(
322322
kwargs["protocol"] = request.config.getoption("--protocol")
323323

324324
cluster_mode = REDIS_INFO["cluster_enabled"]
325+
ssl = kwargs.pop("ssl", False)
325326
if not cluster_mode:
326327
url_options = parse_url(redis_url)
327328
connection_class = Connection
328-
ssl = kwargs.pop("ssl", False)
329329
if ssl:
330330
connection_class = SSLConnection
331331
kwargs["ssl_certfile"] = get_ssl_filename("client-cert.pem")

0 commit comments

Comments
 (0)