Merge pull request #19 from Vendic/feature/redis-mandatory

Redis Connection added enforce_redis constructor param

Author: Tjitse-E

Checks/RedisConnection.php

@@ -19,7 +19,8 @@ class RedisConnection implements CheckInterface
     public function __construct(
         private DeploymentConfig $deploymentConfig,
         private Cache $cache,
-        private CheckResultFactory $checkResultFactory
+        private CheckResultFactory $checkResultFactory,
+        private bool $enforceRedis = false
     ) {
     }
 
@@ -41,7 +42,9 @@ public function run(): CheckResultInterface
         );
 
         if ($this->checkisRedisEnabled($deploymentConfig) === false) {
-            $checkResult->setStatus(CheckStatus::STATUS_SKIPPED);
+            $checkResult->setStatus(
+                $this->enforceRedis ? CheckStatus::STATUS_FAILED : CheckStatus::STATUS_SKIPPED
+            );
             $checkResult->setShortSummary('Redis disabled');
             $checkResult->setNotificationMessage('Redis is not enabled');
             return $checkResult;

Test/Integration/Checks/RedisConnectionTest.php

@@ -25,4 +25,20 @@ public function testRedisConnectionWarning(): void
             'Redis connection check should be skipped when Redis is not enabled'
         );
     }
+
+    public function testRedisConnectionEnforceRedis(): void
+    {
+        /** @var RedisConnection $redisConnectionCheck */
+        $redisConnectionCheck = Bootstrap::getObjectManager()->create(RedisConnection::class, [
+            'enforceRedis' => true,
+        ]);
+
+        $output = $redisConnectionCheck->run();
+        $this->assertEquals('redis_connection', $output->getName());
+        $this->assertEquals(
+            CheckStatus::STATUS_FAILED,
+            $output->getStatus(),
+            'Redis connection check should be skipped when Redis is not enabled'
+        );
+    }
 }