Merge pull request #28 from stickeegreg/main

Fix ScheduleWatcherOptions

Author: plunkettscott

.gitignore

@@ -3,4 +3,4 @@
 /vendor
 composer.lock
 /phpunit.xml
-.phpunit.result.cache
+/.phpunit.cache

src/Watchers/ScheduleWatcherOptions.php

@@ -2,13 +2,13 @@
 
 namespace PlunkettScott\LaravelOpenTelemetry\Watchers;
 
-readonly class ScheduleWatcherOptions extends WatcherOptions
+final readonly class ScheduleWatcherOptions extends WatcherOptions
 {
     /**
      * @param bool $record_output When true, the output of the command will be recorded as an event on the span. This can be useful for debugging, but can be a risk if the output contains sensitive information.
      */
     public function __construct(
-        bool $record_output = false,
+        public bool $record_output = false,
     ) {
     }
 

tests/Watchers/ScheduleWatcherTest.php

@@ -0,0 +1,17 @@
+<?php
+
+use Illuminate\Console\Scheduling\Schedule;
+use Illuminate\Support\Facades\Artisan;
+use PlunkettScott\LaravelOpenTelemetry\Watchers\ScheduleWatcher;
+
+it('watches schedules', function () {
+    $fake = $this->withFakeSpan();
+    $this->enableWatcher(ScheduleWatcher::class, [
+        'record_output' => true,
+    ]);
+
+    app(Schedule::class)->call(function () {})->everyMinute();
+    Artisan::call('schedule:run');
+
+    $fake->assertAttributeEquals(0, 'task.exit_code');
+});