fix(cr): introduce CacheManagerException and update error handling in…

… CacheManagerBase

ProcessMaker/Cache/CacheManagerBase.php

@@ -45,7 +45,7 @@ public function __construct()
     public function getKeysByPattern(string $pattern): array
     {
         if (!in_array($this->connection, self::AVAILABLE_CONNECTIONS)) {
-            throw new Exception('`getKeysByPattern` method only supports Redis connections.');
+            throw new CacheManagerException('`getKeysByPattern` method only supports Redis connections.');
         }
 
         try {
@@ -54,7 +54,7 @@ public function getKeysByPattern(string $pattern): array
             // Filter keys by pattern
             return array_filter($keys, fn ($key) => preg_match('/' . $pattern . '/', $key));
         } catch (Exception $e) {
-            Log::info('CacheABC' . $e->getMessage());
+            Log::info('CacheManagerBase: ' . $e->getMessage());
         }
 
         return [];

ProcessMaker/Cache/CacheManagerException.php

@@ -0,0 +1,9 @@
+<?php
+
+namespace ProcessMaker\Cache;
+
+use Exception;
+
+class CacheManagerException extends Exception
+{
+}

ProcessMaker/Cache/Settings/SettingCacheManager.php

@@ -150,6 +150,7 @@ public function clear(): bool
      */
     public function clearBy(string $pattern): void
     {
+        dump($this->connection);
         if ($this->connection !== 'cache_settings') {
             throw new SettingCacheException('The cache driver must be Redis.');
         }

tests/Feature/Cache/CacheManagerBaseTest.php

@@ -6,6 +6,7 @@
 use Illuminate\Support\Facades\Log;
 use Illuminate\Support\Facades\Redis;
 use ProcessMaker\Cache\CacheManagerBase;
+use ProcessMaker\Cache\CacheManagerException;
 use Tests\TestCase;
 
 class CacheManagerBaseTest extends TestCase
@@ -75,7 +76,7 @@ public function testGetKeysByPatternWithInvalidConnection()
 
         $this->cacheManagerBase = $this->getMockForAbstractClass(CacheManagerBase::class);
 
-        $this->expectException(Exception::class);
+        $this->expectException(CacheManagerException::class);
         $this->expectExceptionMessage('`getKeysByPattern` method only supports Redis connections.');
 
         $this->cacheManagerBase->getKeysByPattern('pattern');
@@ -97,7 +98,7 @@ public function testGetKeysByPatternWithExceptionDuringKeyRetrieval()
             ->andThrow(new Exception('Redis error'));
 
         Log::shouldReceive('info')
-            ->with('CacheABC' . 'Redis error')
+            ->with('CacheManagerBase: ' . 'Redis error')
             ->once();
 
         $result = $this->cacheManagerBase->getKeysByPattern($pattern);

tests/Feature/Cache/SettingCacheTest.php

@@ -215,7 +215,7 @@ public function testTryClearByPatternWithNonRedisDriver()
     {
         config()->set('cache.default', 'array');
 
-        $this->expectException(\Exception::class);
+        $this->expectException(SettingCacheException::class);
         $this->expectExceptionMessage('The cache driver must be Redis.');
 
         \SettingCache::clearBy('pattern');