From 7ede2e0fd98f27aa6186dec1410ca95e23179471 Mon Sep 17 00:00:00 2001
From: Eunchong Yu <kroisse@gmail.com>
Date: Sun, 10 Mar 2013 12:12:57 +0900
Subject: [PATCH] Fail the testing fast when the Redis server is unreachable

---
 sidertests/env.py | 20 ++++++++++++++------
 1 file changed, 14 insertions(+), 6 deletions(-)

diff --git a/sidertests/env.py b/sidertests/env.py
index 02743ff..4c2a1ee 100644
--- a/sidertests/env.py
+++ b/sidertests/env.py
@@ -1,7 +1,7 @@
 import os
 import datetime
 import pytest
-from redis.client import StrictRedis
+from redis.client import StrictRedis, ConnectionError
 from sider.session import Session
 from sider.types import Integer
 
@@ -10,11 +10,20 @@ def get_client(cls=StrictRedis):
     host = os.environ.get('SIDERTEST_HOST', 'localhost')
     port = int(os.environ.get('SIDERTEST_PORT', 6379))
     db = int(os.environ.get('SIDERTEST_DB', 0))
-    return cls(host=host, port=port, db=db)
+    try:
+        client = cls(host=host, port=port, db=db)
+        # To connect on the server forcibly,
+        # send a ping explicitly at this line.
+        client.ping()
+        return client
+    except ConnectionError as e:
+        pytest.fail(str(e), pytrace=False)
 
 
-def get_session():
-    session = Session(get_client())
+def get_session(client=None):
+    if client is None:
+        client = get_client()
+    session = Session(client)
     session.verbose_transaction_error = True
     return session
 
@@ -30,8 +39,7 @@ def key(key):
 @pytest.fixture
 def session(request):
     client = get_client()
-    session = Session(client)
-    session.verbose_transaction_error = True
+    session = get_session(client)
 
     @request.addfinalizer
     def fin():