Merge pull request #5316 from kenjis/fix-tests-markTestSkipped

test: add $this->markTestSkipped() when extension not loaded

Author: kenjis

tests/system/Cache/Handlers/MemcachedHandlerTest.php

@@ -37,6 +37,10 @@ protected function setUp(): void
     {
         parent::setUp();
 
+        if (! extension_loaded('memcached')) {
+            $this->markTestSkipped('Memcached extension not loaded.');
+        }
+
         $this->config = new Cache();
 
         $this->handler = new MemcachedHandler($this->config);

tests/system/Cache/Handlers/RedisHandlerTest.php

@@ -36,6 +36,10 @@ protected function setUp(): void
     {
         parent::setUp();
 
+        if (! extension_loaded('redis')) {
+            $this->markTestSkipped('redis extension not loaded.');
+        }
+
         $this->config = new Cache();
 
         $this->handler = new RedisHandler($this->config);

tests/system/Images/BaseHandlerTest.php

@@ -34,8 +34,6 @@ protected function setUp(): void
     {
         if (! extension_loaded('gd')) {
             $this->markTestSkipped('The GD extension is not available.');
-
-            return;
         }
 
         // create virtual file system

tests/system/Images/GDHandlerTest.php

@@ -33,8 +33,6 @@ protected function setUp(): void
     {
         if (! extension_loaded('gd')) {
             $this->markTestSkipped('The GD extension is not available.');
-
-            return;
         }
 
         // create virtual file system

tests/system/Images/ImageMagickHandlerTest.php

@@ -34,8 +34,6 @@ protected function setUp(): void
     {
         if (! extension_loaded('imagick')) {
             $this->markTestSkipped('The ImageMagick extension is not available.');
-
-            return;
         }
 
         $this->root = WRITEPATH . 'cache/';