Update: 调整测试

NHZEX · Mar 29, 2024 · b18cb36 · b18cb36
1 parent 20d4050
commit b18cb36
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/.github/actions/ci-prepare/action.yml b/.github/actions/ci-prepare/action.yml
@@ -17,6 +17,7 @@ runs:
         docker compose -f ./.github/docker-compose.yml up -d ${{ inputs.env }} redis-new
         docker exec ${{ inputs.env }} php -v
         docker exec ${{ inputs.env }} php -m
+        docker exec ${{ inputs.env }} php --ri redis
         docker exec ${{ inputs.env }} php --ri swoole
         docker exec ${{ inputs.env }} php -r "echo 'opcache config: '; print_r(opcache_get_status(false));"
         docker exec ${{ inputs.env }} composer -V

diff --git a/src/Components/redis/tests/Tests/PhpRedisTest.php b/src/Components/redis/tests/Tests/PhpRedisTest.php
@@ -168,9 +168,18 @@ public function testEvalScriptClass(IRedisHandler $redis): void
     #[Depends('testGetDrive')]
     public function testEvalScriptReadOnly(IRedisHandler $redis): void
     {
-        if (version_compare($redis->getServerVersion(), '7.0', '<')) {
+        if (version_compare($redis->getServerVersion(), '7.0', '<'))
+        {
             $this->markTestSkipped(sprintf('Redis version %s does not support read-only script', $redis->getServerVersion()));
         }
+        if (($redis instanceof PhpRedisHandler || $redis instanceof PhpRedisClusterHandler) && version_compare($redis->getClientVersion(), '6.0', '<'))
+        {
+            $this->markTestSkipped(sprintf('PhpRedis version %s does not support read-only script', $redis->getClientVersion()));
+        }
+        if ($redis instanceof PredisClusterHandler && version_compare($redis->getVersion(), '2.2', '<'))
+        {
+            $this->markTestSkipped(sprintf('Predis (Cluster) version %s does not support read-only script', $redis->getClientVersion()));
+        }
 
         $this->flushLuaScript($redis);