Merge pull request #90 from alex-scott/patch-1

lrem() function - parameters order fixed
BedrockStreaming · Jan 8, 2020 · 1b7a1bc · 1b7a1bc
2 parents 057ad8e + d20232c
commit 1b7a1bc
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/src/M6Web/Component/RedisMock/RedisMock.php b/src/M6Web/Component/RedisMock/RedisMock.php
@@ -495,7 +495,7 @@ public function lindex($key, $index)
         return $this->returnPipedInfo(self::$dataValues[$this->storage][$key][$position]);
     }
 
-    public function lrem($key, $count, $value)
+    public function lrem($key, $value, $count)
     {
         if (!isset(self::$dataValues[$this->storage][$key]) || !in_array($value, self::$dataValues[$this->storage][$key]) || $this->deleteOnTtlExpired($key)) {
             return $this->returnPipedInfo(0);

diff --git a/tests/units/RedisMock.php b/tests/units/RedisMock.php
@@ -1497,11 +1497,11 @@ public function testLPushRPushLRemLTrim()
                 ->isIdenticalTo(4)
             ->array($redisMock->getData())
                 ->isEqualTo(array('test' => array('raoul', 'blabla', 'something', 'raoul')))
-            ->integer($redisMock->lrem('test', 2, 'blabla'))
+            ->integer($redisMock->lrem('test', 'blabla', 2))
                 ->isIdenticalTo(1)
             ->array($redisMock->getData())
                 ->isEqualTo(array('test' => array('raoul', 'something', 'raoul')))
-            ->integer($redisMock->lrem('test', 1, 'raoul'))
+            ->integer($redisMock->lrem('test', 'raoul', 1))
                 ->isIdenticalTo(1)
             ->array($redisMock->getData())
                 ->isEqualTo(array('test' => array('something', 'raoul')))
@@ -1513,7 +1513,7 @@ public function testLPushRPushLRemLTrim()
                 ->isIdenticalTo(5)
             ->array($redisMock->getData())
                 ->isEqualTo(array('test' => array('raoul', 'something', 'raoul', 'raoul', 'raoul')))
-            ->integer($redisMock->lrem('test', -2, 'raoul'))
+            ->integer($redisMock->lrem('test', 'raoul', -2))
                 ->isIdenticalTo(2)
             ->array($redisMock->getData())
                 ->isEqualTo(array('test' => array('raoul', 'something', 'raoul')))
@@ -1523,7 +1523,7 @@ public function testLPushRPushLRemLTrim()
                 ->isIdenticalTo(5)
             ->array($redisMock->getData())
                 ->isEqualTo(array('test' => array('raoul', 'something', 'raoul', 'raoul', 'raoul')))
-            ->integer($redisMock->lrem('test', 0, 'raoul'))
+            ->integer($redisMock->lrem('test', 'raoul', 0))
                 ->isIdenticalTo(4)
             ->array($redisMock->getData())
                 ->isEqualTo(array('test' => array('something')))
@@ -1597,13 +1597,13 @@ public function testLPushRPushLRemLTrim()
                 ->isEqualTo(1)
             ->integer($redisMock->rpush('test', 'test1'))
                 ->isEqualTo(2)
-            ->integer($redisMock->lrem('test', 1 , 'test1'))
+            ->integer($redisMock->lrem('test', 'test1', 1))
                 ->isEqualTo(1)
             ->integer($redisMock->expire('test', 1))
                 ->isEqualTo(1);
         sleep(2);
         $this->assert
-            ->integer($redisMock->lrem('test', 1 , 'test1'))
+            ->integer($redisMock->lrem('test', 'test1', 1))
                 ->isEqualTo(0);
 
         $this->assert
@@ -1667,7 +1667,7 @@ public function testPipeline()
                     ->lpush('test', 'test1')
                     ->lrange('test', 0, -1)
                     ->ltrim('test', 0, -1)
-                    ->lrem('test', 1, 'test1')
+                    ->lrem('test', 'test1', 1)
                     ->rpush('test', 'test1')
                     ->type('test')
                     ->ttl('test')