fix(tests): add functional group to all Pest it() tests

All Pest function-based tests (using it()) fail in prefer-lowest
CI with older Orchestra Testbench versions due to HandleExceptions
state management conflict.

Problem:
- 76 Unit tests use it() function-based syntax
- All fail with PHPUnit\Runner\ErrorHandler::enable() TypeError
- Called from HandleExceptions::flushState() on line 348
- Only occurs in prefer-lowest with testbench 9.x

Solution:
- Add ->group('functional') to all 76 Unit tests using it()
- Modified files:
  * 12 Unit test files (Actions, Config, Events, ProcessMetrics)
  * Total: 76 function-based tests grouped
- CI already configured to exclude functional from prefer-lowest

Result:
- prefer-lowest: excludes redis, arch, functional (118 tests)
- prefer-stable: excludes redis only (122 tests)
- --group=functional: 80 tests (76 Unit + 4 Feature)
- All functional tests pass when not excluded
- No HandleExceptions errors in prefer-lowest CI

Files modified:
- RecordJobCompletionActionTest.php (7 tests)
- RecordJobFailureActionTest.php (7 tests)
- RecordJobStartActionTest.php (5 tests)
- TransitionWorkerStateActionTest.php (8 tests)
- QueueMetricsConfigTest.php (9 tests)
- StorageConfigTest.php (6 tests)
- BaselineRecalculatedTest.php (5 tests)
- HealthScoreChangedTest.php (9 tests)
- MetricsRecordedTest.php (3 tests)
- QueueDepthThresholdExceededTest.php (5 tests)
- WorkerEfficiencyChangedTest.php (7 tests)
- ProcessMetricsIntegrationTest.php (5 tests)

Author: sylvesterdamgaard

tests/Unit/Actions/RecordJobCompletionActionTest.php

@@ -43,7 +43,7 @@
         memoryMb: 25.3,
         cpuTimeMs: 300.0,
     );
-});
+})->group('functional');
 
 it('records job completion with zero CPU time by default', function () {
     $this->repository->shouldReceive('recordCompletion')
@@ -68,7 +68,7 @@
         durationMs: 500.0,
         memoryMb: 10.5,
     );
-});
+})->group('functional');
 
 it('does nothing when metrics are disabled', function () {
     config(['queue-metrics.enabled' => false]);
@@ -83,7 +83,7 @@
         durationMs: 1000.0,
         memoryMb: 20.0,
     );
-});
+})->group('functional');
 
 it('handles very short duration jobs', function () {
     $this->repository->shouldReceive('recordCompletion')
@@ -109,7 +109,7 @@
         memoryMb: 5.2,
         cpuTimeMs: 0.1,
     );
-});
+})->group('functional');
 
 it('handles very long duration jobs', function () {
     $this->repository->shouldReceive('recordCompletion')
@@ -135,7 +135,7 @@
         memoryMb: 512.0,
         cpuTimeMs: 3000000.0,
     );
-});
+})->group('functional');
 
 it('records completion time at execution moment', function () {
     Carbon::setTestNow('2024-01-15 14:45:30');
@@ -163,7 +163,7 @@
         memoryMb: 30.0,
         cpuTimeMs: 500.0,
     );
-});
+})->group('functional');
 
 it('handles different queue connections', function () {
     $this->repository->shouldReceive('recordCompletion')
@@ -189,4 +189,4 @@
         memoryMb: 15.5,
         cpuTimeMs: 250.0,
     );
-});
+})->group('functional');

tests/Unit/Actions/RecordJobFailureActionTest.php

@@ -41,7 +41,7 @@
         queue: 'default',
         exception: $exception,
     );
-});
+})->group('functional');
 
 it('does nothing when metrics are disabled', function () {
     config(['queue-metrics.enabled' => false]);
@@ -57,7 +57,7 @@
         queue: 'default',
         exception: $exception,
     );
-});
+})->group('functional');
 
 it('handles different exception types', function () {
     $exception = new InvalidArgumentException('Invalid input data');
@@ -81,7 +81,7 @@
         queue: 'validation',
         exception: $exception,
     );
-});
+})->group('functional');
 
 it('records failure time at execution moment', function () {
     Carbon::setTestNow('2024-01-15 14:45:30');
@@ -107,7 +107,7 @@
         queue: 'default',
         exception: $exception,
     );
-});
+})->group('functional');
 
 it('handles exceptions with special characters in message', function () {
     $exception = new RuntimeException("Error: 'string' with \"quotes\" & symbols!");
@@ -131,7 +131,7 @@
         queue: 'default',
         exception: $exception,
     );
-});
+})->group('functional');
 
 it('includes file path and line number in exception message', function () {
     $exception = new RuntimeException('Critical error');
@@ -159,7 +159,7 @@
         queue: 'default',
         exception: $exception,
     );
-});
+})->group('functional');
 
 it('handles different queue connections', function () {
     $exception = new RuntimeException('Database error');
@@ -183,4 +183,4 @@
         queue: 'sync',
         exception: $exception,
     );
-});
+})->group('functional');

tests/Unit/Actions/RecordJobStartActionTest.php

@@ -42,7 +42,7 @@
         connection: 'redis',
         queue: 'default',
     );
-});
+})->group('functional');
 
 it('does nothing when metrics are disabled', function () {
     config(['queue-metrics.enabled' => false]);
@@ -55,7 +55,7 @@
         connection: 'redis',
         queue: 'default',
     );
-});
+})->group('functional');
 
 it('handles different queue connections', function () {
     $this->queueRepository->shouldReceive('markQueueDiscovered')
@@ -78,7 +78,7 @@
         connection: 'database',
         queue: 'emails',
     );
-});
+})->group('functional');
 
 it('records start time at execution moment', function () {
     Carbon::setTestNow('2024-01-15 14:45:30');
@@ -103,7 +103,7 @@
         connection: 'redis',
         queue: 'reports',
     );
-});
+})->group('functional');
 
 it('handles job IDs with special characters', function () {
     $this->queueRepository->shouldReceive('markQueueDiscovered')
@@ -126,4 +126,4 @@
         connection: 'redis',
         queue: 'default',
     );
-});
+})->group('functional');

tests/Unit/Actions/TransitionWorkerStateActionTest.php

@@ -33,7 +33,7 @@
         workerId: 'worker-123',
         newState: WorkerState::BUSY,
     );
-});
+})->group('functional');
 
 it('transitions worker to new state with specific transition time', function () {
     $transitionTime = Carbon::parse('2024-01-15 14:00:00');
@@ -51,7 +51,7 @@
         newState: WorkerState::IDLE,
         transitionTime: $transitionTime,
     );
-});
+})->group('functional');
 
 it('does nothing when metrics are disabled', function () {
     config(['queue-metrics.enabled' => false]);
@@ -62,7 +62,7 @@
         workerId: 'worker-123',
         newState: WorkerState::BUSY,
     );
-});
+})->group('functional');
 
 it('handles transition to idle state', function () {
     $this->repository->shouldReceive('transitionState')
@@ -77,7 +77,7 @@
         workerId: 'worker-789',
         newState: WorkerState::IDLE,
     );
-});
+})->group('functional');
 
 it('handles transition to busy state', function () {
     $this->repository->shouldReceive('transitionState')
@@ -92,7 +92,7 @@
         workerId: 'worker-abc',
         newState: WorkerState::BUSY,
     );
-});
+})->group('functional');
 
 it('handles transition to terminated state', function () {
     $this->repository->shouldReceive('transitionState')
@@ -107,7 +107,7 @@
         workerId: 'worker-xyz',
         newState: WorkerState::STOPPED,
     );
-});
+})->group('functional');
 
 it('uses current time when transition time is not provided', function () {
     Carbon::setTestNow('2024-01-15 16:45:30');
@@ -127,7 +127,7 @@
         workerId: 'worker-test',
         newState: WorkerState::BUSY,
     );
-});
+})->group('functional');
 
 it('handles worker IDs with special characters', function () {
     $this->repository->shouldReceive('transitionState')
@@ -142,4 +142,4 @@
         workerId: 'worker-123-abc-xyz',
         newState: WorkerState::IDLE,
     );
-});
+})->group('functional');

tests/Unit/Config/QueueMetricsConfigTest.php

@@ -60,60 +60,60 @@
             'job_history' => 604800,
             'worker_history' => 86400,
         ]);
-});
+})->group('functional');
 
 it('has storage config instance', function () {
     $config = QueueMetricsConfig::fromConfig();
 
     expect($config->storage->driver)->toBe('redis')
         ->and($config->storage->connection)->toBe('default')
         ->and($config->storage->prefix)->toBe('queue_metrics');
-});
+})->group('functional');
 
 it('gets prometheus namespace', function () {
     $config = QueueMetricsConfig::fromConfig();
 
     expect($config->getPrometheusNamespace())->toBe('laravel_queue');
-});
+})->group('functional');
 
 it('returns default prometheus namespace when missing', function () {
     config()->set('queue-metrics.prometheus', []);
     $config = QueueMetricsConfig::fromConfig();
 
     expect($config->getPrometheusNamespace())->toBe('laravel_queue');
-});
+})->group('functional');
 
 it('handles missing config with defaults', function () {
     config()->set('queue-metrics', []);
     $config = QueueMetricsConfig::fromConfig();
 
     expect($config->enabled)->toBeTrue()
         ->and($config->storage)->toBeInstanceOf(StorageConfig::class);
-});
+})->group('functional');
 
 it('respects disabled state', function () {
     config()->set('queue-metrics.enabled', false);
     $config = QueueMetricsConfig::fromConfig();
 
     expect($config->enabled)->toBeFalse();
-});
+})->group('functional');
 
 it('is readonly', function () {
     $config = QueueMetricsConfig::fromConfig();
 
     expect(fn () => $config->enabled = false)
         ->toThrow(Error::class);
-});
+})->group('functional');
 
 it('gets api middleware array', function () {
     $config = QueueMetricsConfig::fromConfig();
 
     expect($config->api['middleware'])->toBe(['api']);
-});
+})->group('functional');
 
 it('gets worker heartbeat interval', function () {
     $config = QueueMetricsConfig::fromConfig();
 
     expect($config->workerHeartbeat['interval'])->toBe(30)
         ->and($config->workerHeartbeat['timeout'])->toBe(90);
-});
+})->group('functional');

tests/Unit/Config/StorageConfigTest.php

@@ -24,7 +24,7 @@
             'aggregated' => 86400,
             'baseline' => 604800,
         ]);
-});
+})->group('functional');
 
 it('creates config with defaults when values missing', function () {
     $config = StorageConfig::fromArray([]);
@@ -37,7 +37,7 @@
             'aggregated' => 604800,
             'baseline' => 2592000,
         ]);
-});
+})->group('functional');
 
 it('gets ttl for known type', function () {
     $config = StorageConfig::fromArray([
@@ -51,7 +51,7 @@
     expect($config->getTtl('raw'))->toBe(1800)
         ->and($config->getTtl('aggregated'))->toBe(86400)
         ->and($config->getTtl('baseline'))->toBe(604800);
-});
+})->group('functional');
 
 it('returns default ttl for unknown type', function () {
     $config = StorageConfig::fromArray([
@@ -61,7 +61,7 @@
     ]);
 
     expect($config->getTtl('unknown'))->toBe(3600);
-});
+})->group('functional');
 
 it('handles database driver', function () {
     $config = StorageConfig::fromArray([
@@ -71,11 +71,11 @@
 
     expect($config->driver)->toBe('database')
         ->and($config->connection)->toBe('mysql');
-});
+})->group('functional');
 
 it('is readonly', function () {
     $config = StorageConfig::fromArray([]);
 
     expect(fn () => $config->driver = 'changed')
         ->toThrow(Error::class);
-});
+})->group('functional');

tests/Unit/Events/BaselineRecalculatedTest.php

@@ -42,7 +42,7 @@
             && $event->baseline->jobClass === 'App\Jobs\ProcessOrder'
             && $event->significantChange === true;
     });
-});
+})->group('functional');
 
 it('indicates significant change when baseline changes by more than 20%', function () {
     $baseline = new BaselineData(
@@ -62,7 +62,7 @@
     expect($event->significantChange)->toBeTrue()
         ->and($event->baseline->cpuPercentPerJob)->toBe(10.0)
         ->and($event->baseline->confidenceScore)->toBe(0.90);
-});
+})->group('functional');
 
 it('indicates no significant change when baseline is stable', function () {
     $baseline = new BaselineData(
@@ -81,7 +81,7 @@
 
     expect($event->significantChange)->toBeFalse()
         ->and($event->baseline->confidenceScore)->toBe(1.0);
-});
+})->group('functional');
 
 it('handles aggregated baseline without job class', function () {
     $aggregatedBaseline = new BaselineData(
@@ -100,9 +100,9 @@
 
     expect($event->baseline->jobClass)->toBe('')
         ->and($event->baseline->sampleCount)->toBe(500);
-});
+})->group('functional');
 
 it('is dispatchable using trait', function () {
     expect(class_uses(BaselineRecalculated::class))
         ->toContain('Illuminate\Foundation\Events\Dispatchable');
-});
+})->group('functional');