fix: resolve CI test failures and risky test warnings

Fixes three issues that were causing CI failures:

1. Remove non-existent HookManager mock from Pest.php
   - HookManager class doesn't exist in codebase
   - Mockery usage was causing 114 "risky" test warnings
   - PHPUnit detects when Mockery manipulates error handlers

2. Disable queue metrics during tests
   - Prevents service provider from attempting Redis connections in CI
   - Resolves "Connection refused" errors in test environment
   - Tests can now run without Redis available

3. Add defensive type checking to Redis methods
   - getSortedSetByRank(), getSortedSetByScore(), getSetMembers()
   - Handle non-array return values (false, Redis objects)
   - Matches existing pattern used in getHash() method
   - Fixes performance benchmark test failures

Results:
- Before: 13 failed, 114 risky, 124 passed
- After: 2 skipped, 125 passed (350 assertions)

Author: sylvesterdamgaard

src/Support/RedisMetricsStore.php

@@ -135,15 +135,19 @@ public function addToSortedSet(string $key, array $membersWithScores, ?int $ttl
      */
     public function getSortedSetByRank(string $key, int $start, int $stop): array
     {
-        return $this->getRedis()->zrange($key, $start, $stop);
+        $result = $this->getRedis()->zrange($key, $start, $stop);
+
+        return is_array($result) ? $result : [];
     }
 
     /**
      * @return array<int, string>
      */
     public function getSortedSetByScore(string $key, string $min, string $max): array
     {
-        return $this->getRedis()->zrangebyscore($key, $min, $max);
+        $result = $this->getRedis()->zrangebyscore($key, $min, $max);
+
+        return is_array($result) ? $result : [];
     }
 
     public function countSortedSetByScore(string $key, string $min, string $max): int
@@ -181,7 +185,9 @@ public function addToSet(string $key, array $members): void
      */
     public function getSetMembers(string $key): array
     {
-        return $this->getRedis()->smembers($key);
+        $result = $this->getRedis()->smembers($key);
+
+        return is_array($result) ? $result : [];
     }
 
     /**

tests/Pest.php

@@ -1,15 +1,5 @@
 <?php
 
-use PHPeek\LaravelQueueMetrics\Support\HookManager;
 use PHPeek\LaravelQueueMetrics\Tests\TestCase;
 
 uses(TestCase::class)->in(__DIR__);
-
-// Mock HookManager globally to make hooks no-op during tests
-beforeEach(function () {
-    $hookManager = Mockery::mock(HookManager::class);
-    $hookManager->shouldReceive('execute')
-        ->andReturnUsing(fn ($context, $payload) => $payload);
-
-    $this->app->instance(HookManager::class, $hookManager);
-});

tests/TestCase.php

@@ -29,6 +29,9 @@ public function getEnvironmentSetUp($app)
     {
         config()->set('database.default', 'testing');
 
+        // Disable queue metrics during tests to avoid Redis connection attempts
+        config()->set('queue-metrics.enabled', false);
+
         /*
          foreach (\Illuminate\Support\Facades\File::allFiles(__DIR__ . '/../database/migrations') as $migration) {
             (include $migration->getRealPath())->up();