[patch] Align Redis and MinIO connect() idempotency #37

Merged
brian merged 2 commits from cursor/connect-idempotency-alignment into main 2026-07-09 16:38:06 +02:00
2 changed files with 20 additions and 0 deletions
Showing only changes of commit b8d88e6703 - Show all commits
+10
View File
@@ -75,6 +75,16 @@ def test_connect_with_injected_client_raises_on_failure() -> None:
adapter.connect()
def test_connect_with_injected_client_skips_validation_when_client_cleared() -> None:
mock_client = MagicMock(spec=Minio)
adapter = MinioAdapter(config=TEST_MINIO_CONFIG, client=mock_client)
adapter.disconnect()
adapter.connect()
mock_client.list_buckets.assert_not_called()
def test_connect_disconnects_before_reconnect(
monkeypatch: pytest.MonkeyPatch,
) -> None:
+10
View File
@@ -99,6 +99,16 @@ def test_connect_with_injected_client_raises_on_redis_error() -> None:
adapter.connect()
def test_connect_with_injected_client_skips_validation_when_client_cleared() -> None:
mock_client = MagicMock(spec=redis.Redis)
adapter = RedisAdapter(config=TEST_REDIS_CONFIG, client=mock_client)
adapter.disconnect()
adapter.connect()
mock_client.ping.assert_not_called()
def test_connect_reconnects_when_existing_client_unhealthy(
monkeypatch: pytest.MonkeyPatch,
) -> None: