Issue #3468612 by berdir: Fixing tests

tests/src/Functional/Lock/RedisLockFunctionalTest.php

@@ -61,7 +61,7 @@ class RedisLockFunctionalTest extends LockFunctionalTest {
   /**
    * {@inheritdoc}
    */
-  public function testLockAcquire() {
+  public function testLockAcquire(): void {
     $redis_interface = self::getRedisInterfaceEnv();
     $lock = $this->container->get('lock');
     $this->assertInstanceOf('\Drupal\redis\Lock\\' . $redis_interface, $lock);
@@ -77,7 +77,7 @@ class RedisLockFunctionalTest extends LockFunctionalTest {
   /**
    * {@inheritdoc}
    */
-  public function testPersistentLock() {
+  public function testPersistentLock(): void {
     $redis_interface = self::getRedisInterfaceEnv();
     $persistent_lock = $this->container->get('lock.persistent');
     $this->assertInstanceOf('\Drupal\redis\PersistentLock\\' . $redis_interface, $persistent_lock);

tests/src/Kernel/RedisLockTest.php

@@ -49,7 +49,7 @@ class RedisLockTest extends LockTest {
   /**
    * {@inheritdoc}
    */
-  public function testBackendLockRelease() {
+  public function testBackendLockRelease(): void {
     $redis_interface = self::getRedisInterfaceEnv();
     // Verify that the correct lock backend is being instantiated by the
     // factory.

tests/src/Kernel/RedisQueueTest.php

@@ -25,7 +25,7 @@ class RedisQueueTest extends CoreQueueTest {
   /**
    * Tests Redis non-blocking queue.
    */
-  public function testRedisNonBlockingQueue() {
+  public function testRedisNonBlockingQueue(): void {
     self::setUpSettings();
     $client_factory = \Drupal::service('redis.factory');
     $settings = ['reserve_timeout' => NULL];
@@ -59,7 +59,7 @@ class RedisQueueTest extends CoreQueueTest {
   /**
    * Tests Redis blocking queue.
    */
-  public function testRedisBlockingQueue() {
+  public function testRedisBlockingQueue(): void {
     self::setUpSettings();
     // Create two queues.
     $client_factory = \Drupal::service('redis.factory');
@@ -82,7 +82,7 @@ class RedisQueueTest extends CoreQueueTest {
    *
    * We override tests from core class we extend to prevent them from running.
    */
-  public function testSystemQueue() {
+  public function testSystemQueue(): void {
     $this->markTestSkipped();
   }
 
@@ -91,7 +91,7 @@ class RedisQueueTest extends CoreQueueTest {
    *
    * We override tests from core class we extend to prevent them from running.
    */
-  public function testMemoryQueue() {
+  public function testMemoryQueue(): void {
     $this->markTestSkipped();
   }