Added support for new container command approach of NO-EVICT subcommand (#1346)

vladvildanov · web-flow · commit a5cc48d85fdf · 2023-07-26T16:33:20.000+03:00
diff --git a/.github/workflows/linters.yml b/.github/workflows/linters.yml
@@ -7,6 +7,7 @@ on:
     branches:
       - main
       - v2.**
+      - v3.**
   pull_request: null
 
 permissions: {}
diff --git a/.github/workflows/stack.yml b/.github/workflows/stack.yml
@@ -5,6 +5,7 @@ on:
     branches:
       - main
       - v2.**
+      - v3.**
   pull_request:
 
 jobs:
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -5,6 +5,7 @@ on:
     branches:
       - main
       - v2.**
+      - v3.**
   pull_request:
 
 concurrency:
diff --git a/src/Command/Container/CLIENT.php b/src/Command/Container/CLIENT.php
@@ -18,6 +18,7 @@
  * @method string getName()
  * @method Status kill(...$arguments)
  * @method string list(string $type = null, int ...$clientId)
+ * @method Status noEvict(bool $enable = null)
  * @method Status noTouch(bool $enable = null)
  * @method Status setInfo(string $modifier = null, string $value = null)
  * @method Status setName(string $connectionName)
diff --git a/src/Command/Redis/CLIENT.php b/src/Command/Redis/CLIENT.php
@@ -36,6 +36,10 @@ public function setArguments(array $arguments)
             case 'LIST':
                 $this->setListArguments($arguments);
                 break;
+            case 'NOEVICT':
+                $arguments[0] = 'NO-EVICT';
+                $this->setNoTouchArguments($arguments);
+                break;
             case 'NOTOUCH':
                 $arguments[0] = 'NO-TOUCH';
                 $this->setNoTouchArguments($arguments);
diff --git a/tests/Predis/Command/Redis/CLIENT_Test.php b/tests/Predis/Command/Redis/CLIENT_Test.php
@@ -229,6 +229,17 @@ public function testSetsNameOfConnection(): void
         $this->assertEquals($expectedConnectionName, $redis->client->getName());
     }
 
+    /**
+     * @group connected
+     * @requiresRedisVersion >= 7.0.0
+     */
+    public function testNoEvictTurnEnableEvictionMode(): void
+    {
+        $redis = $this->getClient();
+
+        $this->assertEquals('OK', $redis->client->noEvict(true));
+    }
+
     /**
      * @group connected
      * @requiresRedisVersion >= 7.2.0
