[6.x] Fix the return type of spop for phpredis (#36106)

* PhpRedis return value of spop compatibility

* fix style

* use func_get_args() instead of [$key, $count]

* Update PhpRedisConnection.php

Co-authored-by: Taylor Otwell <taylor@laravel.com>

Author: zingimmick

src/Illuminate/Redis/Connections/PhpRedisConnection.php

@@ -197,7 +197,7 @@ public function brpop(...$arguments)
      */
     public function spop($key, $count = 1)
     {
-        return $this->command('spop', [$key, $count]);
+        return $this->command('spop', func_get_args());
     }
 
     /**

tests/Redis/RedisConnectionTest.php

@@ -690,6 +690,33 @@ public function testItSscansForKeys()
         }
     }
 
+    public function testItSPopsForKeys()
+    {
+        foreach ($this->connections() as $redis) {
+            $members = ['test:spop:1', 'test:spop:2', 'test:spop:3', 'test:spop:4'];
+
+            foreach ($members as $member) {
+                $redis->sadd('set', $member);
+            }
+
+            $result = $redis->spop('set');
+            $this->assertIsNotArray($result);
+            $this->assertContains($result, $members);
+
+            $result = $redis->spop('set', 1);
+
+            $this->assertIsArray($result);
+            $this->assertCount(1, $result);
+
+            $result = $redis->spop('set', 2);
+
+            $this->assertIsArray($result);
+            $this->assertCount(2, $result);
+
+            $redis->flushAll();
+        }
+    }
+
     public function testPhpRedisScanOption()
     {
         foreach ($this->connections() as $redis) {