fix: separate Redis integration tests from unit tests

Addresses CI failures by properly segregating Redis-dependent tests.

Changes:
1. **Workflow separation**:
   - test job: Excludes redis group (runs without Redis)
   - test-with-redis job: Only runs redis group tests
   - Fixes "redis-cli: command not found" by using PHP Redis extension

2. **Test grouping**:
   - Performance tests: Added 'redis' group (require Redis)
   - EventListenersTest: Added @group redis annotation
   - Unit tests: Run without Redis via config disable

3. **Redis connection check**:
   - Replaced bash redis-cli with PHP Redis extension check
   - More reliable and consistent with test environment

Expected results:
- test job: ~119 tests (unit tests only)
- test-with-redis job: ~8 tests (integration/performance)
- ProcessMetricsIntegrationTest errors unrelated to Redis

Author: sylvesterdamgaard

.github/workflows/run-tests.yml

@@ -57,7 +57,7 @@ jobs:
         run: composer show -D
 
       - name: Execute tests
-        run: vendor/bin/pest --ci
+        run: vendor/bin/pest --ci --exclude-group=redis
 
   test-with-redis:
     runs-on: ubuntu-latest
@@ -118,10 +118,26 @@ jobs:
 
       - name: Wait for Redis
         run: |
-          timeout 30 bash -c 'until redis-cli -h 127.0.0.1 -p 6379 ping; do sleep 1; done'
+          php -r "
+            \$redis = new Redis();
+            \$start = time();
+            while (time() - \$start < 30) {
+              try {
+                if (\$redis->connect('127.0.0.1', 6379, 1)) {
+                  echo 'Redis is ready\n';
+                  exit(0);
+                }
+              } catch (Exception \$e) {
+                sleep(1);
+              }
+            }
+            echo 'Timeout waiting for Redis\n';
+            exit(1);
+          "
 
       - name: Execute integration tests
-        run: vendor/bin/pest --ci
+        run: vendor/bin/pest --ci --group=redis
         env:
           REDIS_HOST: 127.0.0.1
           REDIS_PORT: 6379
+          REDIS_AVAILABLE: true

tests/Feature/EventListenersTest.php

@@ -11,6 +11,9 @@
 use PHPeek\LaravelQueueMetrics\Repositories\Contracts\JobMetricsRepository;
 use PHPeek\LaravelQueueMetrics\Tests\Feature\Support\TestJob;
 
+/**
+ * @group redis
+ */
 final class EventListenersTest extends TestCase
 {
     use RefreshDatabase;

tests/Performance/BaselineCalculationBenchmarkTest.php

@@ -23,7 +23,7 @@
 
     // Assert: Should complete within 5 seconds even with 1000 samples
     expect($duration)->toBeLessThan(5.0);
-})->group('performance', 'slow');
+})->group('performance', 'slow', 'redis');
 
 test('batch baseline fetching is faster than sequential', function () {
     $baselineRepository = app(BaselineRepository::class);
@@ -51,7 +51,7 @@
     // Assert: Batch should not be significantly slower (allow 2x tolerance for overhead)
     // With empty data, batch operations may have pipeline overhead
     expect($batchDuration)->toBeLessThan($sequentialDuration * 2);
-})->group('performance', 'slow');
+})->group('performance', 'slow', 'redis');
 
 test('key scanning performance is acceptable for large datasets', function () {
     $keyScanner = app(\PHPeek\LaravelQueueMetrics\Services\RedisKeyScannerService::class);
@@ -75,7 +75,7 @@
 
     // Assert: Scanning should complete within 2 seconds
     expect($duration)->toBeLessThan(2.0);
-})->group('performance');
+})->group('performance', 'redis');
 
 test('overview query aggregation completes within acceptable time', function () {
     $overviewService = app(\PHPeek\LaravelQueueMetrics\Services\OverviewQueryService::class);
@@ -91,7 +91,7 @@
     expect($duration)->toBeLessThan(3.0);
     expect($overview)->toBeArray();
     expect($overview)->toHaveKeys(['queues', 'jobs', 'servers', 'workers', 'baselines', 'metadata']);
-})->group('performance', 'slow');
+})->group('performance', 'slow', 'redis');
 
 test('redis transaction is faster or same speed as pipeline for critical mutations', function () {
     $jobMetricsRepository = app(\PHPeek\LaravelQueueMetrics\Repositories\Contracts\JobMetricsRepository::class);
@@ -127,7 +127,7 @@
 
     // Assert: 10 transactions should complete within 1 second
     expect($duration)->toBeLessThan(1.0);
-})->group('performance');
+})->group('performance', 'redis');
 
 test('memory usage stays reasonable during large batch operations', function () {
     $overviewService = app(\PHPeek\LaravelQueueMetrics\Services\OverviewQueryService::class);
@@ -141,4 +141,4 @@
 
     // Assert: Memory increase should be less than 50MB for overview query
     expect($memoryIncrease)->toBeLessThan(50);
-})->group('performance');
+})->group('performance', 'redis');