Perform actual connection (+ increase test coverage)

rq/scripts/__init__.py

@@ -56,5 +56,5 @@ def setup_redis(args):
         redis_conn = redis.StrictRedis.from_url(args.url, db=args.db)
     else:
         redis_conn = redis.StrictRedis(host=args.host, port=args.port, db=args.db,
-            password=args.password)
+            password=args.password, unix_socket_path=args.socket)
     use_connection(redis_conn)

tests/test_scripts.py

@@ -10,9 +10,12 @@ class uFaking(object):
         for attr in faking:
             setattr(self, attr, None)
 
+def fake_args():
+    return uFaking('host', 'port', 'url', 'socket', 'db', 'queues', 'password')
+
 def conf_pair(**faked_settings):
-    return (faked_settings,
+    return faked_settings, fake_args()
-            uFaking('host', 'port', 'socket', 'db', 'queues', 'password'))
+
 
 class TestScripts(TestCase):
     def test_config_file(self):
@@ -40,3 +43,13 @@ class TestScripts(TestCase):
         add_standard_arguments(args)
         socket_parms = [e for e in caught if e[0] == '--socket']
         self.assertEqual([('--socket', '-s')], socket_parms)
+
+    def test_redis_socket_whiteboxish(self):
+        args = fake_args()
+        args.socket = '/no/such/path'
+        setup_redis(args)
+
+        from rq import connections
+        conn = connections.pop_connection()
+        from redis.connection import UnixDomainSocketConnection
+        self.assertEqual(conn.connection_pool.connection_kwargs['path'], '/no/such/path')