move redis limit test

Author: mvorisek

tests/Mutex/RedisMutexTest.php

@@ -11,6 +11,7 @@
 use Malkusch\Lock\Mutex\RedisMutex;
 use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\Attributes\RequiresPhpExtension;
+use PHPUnit\Framework\Constraint\IsType;
 use PHPUnit\Framework\TestCase;
 
 if (\PHP_MAJOR_VERSION >= 8) {
@@ -57,6 +58,19 @@ public function set($key, $value, $options = null)
     }
 }
 
+interface PredisClientInterfaceWithSetAndEvalMethods extends PredisClientInterface
+{
+    /**
+     * @return mixed
+     */
+    public function eval();
+
+    /**
+     * @return mixed
+     */
+    public function set();
+}
+
 /**
  * These tests require the environment variable:
  *
@@ -207,6 +221,25 @@ public function testEvalScriptFails(): void
         });
     }
 
+    public function testAcquireExpireTimeoutLimit(): void
+    {
+        $client = $this->createMock(PredisClientInterfaceWithSetAndEvalMethods::class);
+
+        $this->mutex = new RedisMutex($client, 'test');
+
+        $client->expects(self::once())
+            ->method('set')
+            ->with('php-malkusch-lock:test', new IsType(IsType::TYPE_STRING), 'PX', 31_557_600_000_000, 'NX')
+            ->willReturnSelf();
+
+        $client->expects(self::once())
+            ->method('eval')
+            ->with(self::anything(), 1, 'php-malkusch-lock:test', new IsType(IsType::TYPE_STRING))
+            ->willReturn(true);
+
+        $this->mutex->synchronized(static function () {});
+    }
+
     /**
      * @param \Redis::SERIALIZER_*  $serializer
      * @param \Redis::COMPRESSION_* $compressor

tests/Mutex/RedisMutexWithPredisTest.php

@@ -13,7 +13,7 @@
 use Predis\ClientInterface as PredisClientInterface;
 use Predis\PredisException;
 
-interface PredisClientInterfaceWithSetAndEvalMethods extends PredisClientInterface
+interface PredisClientInterfaceWithSetAndEvalMethods2 extends PredisClientInterface
 {
     /**
      * @return mixed
@@ -39,7 +39,7 @@ protected function setUp(): void
     {
         parent::setUp();
 
-        $this->client = $this->createMock(PredisClientInterfaceWithSetAndEvalMethods::class);
+        $this->client = $this->createMock(PredisClientInterfaceWithSetAndEvalMethods2::class);
 
         $this->mutex = new RedisMutex($this->client, 'test', 2.5, 3.5);
     }
@@ -98,23 +98,6 @@ public function testWorksNormally(): void
         self::assertTrue($executed);
     }
 
-    public function testAcquireExpireTimeoutLimit(): void
-    {
-        $this->mutex = new RedisMutex($this->client, 'test');
-
-        $this->client->expects(self::once())
-            ->method('set')
-            ->with('php-malkusch-lock:test', new IsType(IsType::TYPE_STRING), 'PX', 31_557_600_000_000, 'NX')
-            ->willReturnSelf();
-
-        $this->client->expects(self::once())
-            ->method('eval')
-            ->with(self::anything(), 1, 'php-malkusch-lock:test', new IsType(IsType::TYPE_STRING))
-            ->willReturn(true);
-
-        $this->mutex->synchronized(static function () {});
-    }
-
     /**
      * Tests evalScript() fails.
      */