Add ability to pass password, add ability to pass redis instance.

Author: makasim

Diff for: PRedis.php

@@ -2,11 +2,17 @@
 
 namespace Enqueue\Redis;
 
+use Predis\Client;
 use Predis\ClientInterface;
 use Predis\Response\ServerException as PRedisServerException;
 
 class PRedis implements Redis
 {
+    /**
+     * @var array
+     */
+    private $config;
+
     /**
      * @var ClientInterface
      */
@@ -15,9 +21,19 @@ class PRedis implements Redis
     /**
      * @param ClientInterface $redis
      */
-    public function __construct(ClientInterface $redis)
+    public function __construct(array $config)
     {
-        $this->redis = $redis;
+        $this->config = $this->config = array_replace([
+            'host' => null,
+            'port' => null,
+            'pass' => null,
+            'user' => null,
+            'timeout' => null,
+            'reserved' => null,
+            'retry_interval' => null,
+            'persisted' => false,
+            'database' => 0,
+        ], $config);
     }
 
     /**
@@ -63,6 +79,12 @@ public function rpop($key)
      */
     public function connect()
     {
+        $this->redis = new Client($this->config, ['exceptions' => true]);
+
+        if ($this->config['pass']) {
+            $this->redis->auth($this->config['pass']);
+        }
+
         $this->redis->connect();
     }
 

Diff for: PhpRedis.php

@@ -22,6 +22,8 @@ public function __construct(array $config)
         $this->config = array_replace([
             'host' => null,
             'port' => null,
+            'pass' => null,
+            'user' => null,
             'timeout' => null,
             'reserved' => null,
             'retry_interval' => null,
@@ -82,12 +84,8 @@ public function connect()
                 );
             }
 
-            if (array_key_exists('pass', $this->config)) {
-                $this->config['auth'] = $this->config['pass'];
-            }
-
-            if (array_key_exists('auth', $this->config)) {
-                $this->redis->auth($this->config['auth']);
+            if ($this->config['pass']) {
+                $this->redis->auth($this->config['pass']);
             }
 
             $this->redis->select($this->config['database']);

Diff for: RedisConnectionFactory.php

@@ -3,7 +3,6 @@
 namespace Enqueue\Redis;
 
 use Interop\Queue\PsrConnectionFactory;
-use Predis\Client;
 
 class RedisConnectionFactory implements PsrConnectionFactory
 {
@@ -29,17 +28,30 @@ class RedisConnectionFactory implements PsrConnectionFactory
      *  'persisted' => bool, Whether it use single persisted connection or open a new one for every context
      *  'lazy' => the connection will be performed as later as possible, if the option set to true
      *  'database' => Database index to select when connected (default value: 0)
+     *   user - The user name to use.
+     *   pass - Password.
      * ].
      *
      * or
      *
      * redis:
      * redis:?vendor=predis
      *
-     * @param array|string|null $config
+     * or
+     *
+     * instance of Enqueue\Redis
+     *
+     * @param array|string|Redis|null $config
      */
     public function __construct($config = 'redis:')
     {
+        if ($config instanceof  Redis) {
+            $this->redis = $config;
+            $this->config = $this->defaultConfig();
+
+            return;
+        }
+
         if (empty($config) || 'redis:' === $config) {
             $config = [];
         } elseif (is_string($config)) {
@@ -88,7 +100,7 @@ private function createRedis()
             }
 
             if ('predis' == $this->config['vendor'] && false == $this->redis) {
-                $this->redis = new PRedis(new Client($this->config, ['exceptions' => true]));
+                $this->redis = new PRedis($this->config);
             }
 
             if ('custom' == $this->config['vendor'] && false == $this->redis) {

Diff for: Tests/RedisConnectionFactoryConfigTest.php

@@ -2,6 +2,7 @@
 
 namespace Enqueue\Redis\Tests;
 
+use Enqueue\Redis\Redis;
 use Enqueue\Redis\RedisConnectionFactory;
 use Enqueue\Test\ClassExtensionTrait;
 use PHPUnit\Framework\TestCase;
@@ -45,6 +46,17 @@ public function testThrowIfVendorIsInvalid()
         new RedisConnectionFactory(['vendor' => 'invalidVendor']);
     }
 
+    public function testCouldBeCreatedWithRedisInstance()
+    {
+        $redisMock = $this->createMock(Redis::class);
+
+        $factory = new RedisConnectionFactory($redisMock);
+        $this->assertAttributeSame($redisMock, 'redis', $factory);
+
+        $context = $factory->createContext();
+        $this->assertSame($redisMock, $context->getRedis());
+    }
+
     /**
      * @dataProvider provideConfigs
      *
@@ -141,5 +153,24 @@ public static function provideConfigs()
                 'redis' => null,
             ],
         ];
+
+        // heroku redis
+        yield [
+            'redis://h:[email protected]:111',
+            [
+                'host' => 'ec2-111-1-1-1.compute-1.amazonaws.com',
+                'port' => 111,
+                'timeout' => null,
+                'reserved' => null,
+                'retry_interval' => null,
+                'vendor' => 'phpredis',
+                'persisted' => false,
+                'lazy' => false,
+                'database' => 0,
+                'redis' => null,
+                'user' => 'h',
+                'pass' => 'asdfqwer1234asdf',
+            ],
+        ];
     }
 }