Fix broken Redis mocks

derrabus · derrabus · commit 505631b290a7 · 2021-09-14T19:36:29.000+02:00
Signed-off-by: Alexander M. Turek &lt;me@derrabus.de&gt;
diff --git a/src/Symfony/Component/Messenger/Bridge/Redis/Tests/Transport/ConnectionTest.php b/src/Symfony/Component/Messenger/Bridge/Redis/Tests/Transport/ConnectionTest.php
@@ -78,7 +78,7 @@ public function testFromDsnWithTls()
         $redis->expects($this->once())
             ->method('connect')
             ->with('tls://127.0.0.1', 6379)
-            ->willReturn(null);
+            ->willReturn(true);
 
         Connection::fromDsn('redis://127.0.0.1?tls=1', [], $redis);
     }
@@ -92,7 +92,7 @@ public function testFromDsnWithTlsOption()
         $redis->expects($this->once())
             ->method('connect')
             ->with('tls://127.0.0.1', 6379)
-            ->willReturn(null);
+            ->willReturn(true);
 
         Connection::fromDsn('redis://127.0.0.1', ['tls' => true], $redis);
     }
@@ -103,7 +103,7 @@ public function testFromDsnWithRedissScheme()
         $redis->expects($this->once())
             ->method('connect')
             ->with('tls://127.0.0.1', 6379)
-            ->willReturn(null);
+            ->willReturn(true);
 
         Connection::fromDsn('rediss://127.0.0.1', [], $redis);
     }
@@ -315,7 +315,7 @@ public function testMaxEntries()
 
         $redis->expects($this->exactly(1))->method('xadd')
             ->with('queue', '*', ['message' => '{"body":"1","headers":[]}'], 20000, true)
-            ->willReturn(1);
+            ->willReturn('1');
 
         $connection = Connection::fromDsn('redis://localhost/queue?stream_max_entries=20000', [], $redis); // 1 = always
         $connection->add('1', []);
@@ -327,10 +327,10 @@ public function testDeleteAfterAck()
 
         $redis->expects($this->exactly(1))->method('xack')
             ->with('queue', 'symfony', ['1'])
-            ->willReturn(1);
+            ->willReturn('1');
         $redis->expects($this->exactly(1))->method('xdel')
             ->with('queue', ['1'])
-            ->willReturn(1);
+            ->willReturn('1');
 
         $connection = Connection::fromDsn('redis://localhost/queue?delete_after_ack=true', [], $redis); // 1 = always
         $connection->ack('1');
@@ -342,10 +342,10 @@ public function testDeleteAfterReject()
 
         $redis->expects($this->exactly(1))->method('xack')
             ->with('queue', 'symfony', ['1'])
-            ->willReturn(1);
+            ->willReturn('1');
         $redis->expects($this->exactly(1))->method('xdel')
             ->with('queue', ['1'])
-            ->willReturn(1);
+            ->willReturn('1');
 
         $connection = Connection::fromDsn('redis://localhost/queue?delete_after_reject=true', [], $redis); // 1 = always
         $connection->reject('1');
@@ -355,8 +355,8 @@ public function testLastErrorGetsCleared()
     {
         $redis = $this->createMock(\Redis::class);
 
-        $redis->expects($this->once())->method('xadd')->willReturn(0);
-        $redis->expects($this->once())->method('xack')->willReturn(0);
+        $redis->expects($this->once())->method('xadd')->willReturn('0');
+        $redis->expects($this->once())->method('xack')->willReturn('0');
 
         $redis->method('getLastError')->willReturnOnConsecutiveCalls('xadd error', 'xack error');
         $redis->expects($this->exactly(2))->method('clearLastError');
