Merge pull request #101 from Xon/patch-1

Test against redis v4 & test phpredis v4 (breaking API change for exists())

Author: colinmollenhour

.travis.yml

@@ -13,10 +13,10 @@ allow_failures:
 
 install:
   - yes '' | pecl install -f redis
-  - wget http://download.redis.io/releases/redis-3.2.11.tar.gz
-  - tar -xzf redis-3.2.11.tar.gz
-  - make -s -C redis-3.2.11 -j4
-  - export PATH=$PATH:$PWD/redis-3.2.11/src/
+  - wget http://download.redis.io/releases/redis-4.0.8.tar.gz
+  - tar -xzf redis-4.0.8.tar.gz
+  - make -s -C redis-4.0.8 -j4
+  - export PATH=$PATH:$PWD/redis-4.0.8/src/
 
 script:
   - phpunit

Client.php

@@ -1186,6 +1186,10 @@ public function __call($name, $args)
                         throw new CredisException($error);
                     }
                     break;
+                case 'exists':
+                    // smooth over phpredis-v4 vs earlier difference to match documented credis return results
+                    $response = (int) $response;
+                    break;
                 default:
                     $error = $this->redis->getLastError();
                     $this->redis->clearLastError();

tests/CredisTest.php

@@ -65,6 +65,10 @@ public function testScalars()
         $this->assertEquals('FOO', $this->credis->get('foo'));
         $this->assertFalse($this->credis->get('nil'));
 
+        // exists support
+        $this->assertEquals($this->credis->exists('foo'), 1);
+        $this->assertEquals($this->credis->exists('nil'), 0);
+
         // Empty string
         $this->credis->set('empty','');
         $this->assertEquals('', $this->credis->get('empty'));