Skip unreadable filesystems (#392)

* Skip unreadable filesystems

* Formatting

* Rescue exception

* Use `Pulse::rescue`

* Reset handler

---------

Co-authored-by: Tim MacDonald <[email protected]>

Author: 27pchrisl

src/Recorders/Servers.php

@@ -86,6 +86,7 @@ public function record(SharedBeat $event): void
                 'memory_used' => $memoryUsed,
                 'memory_total' => $memoryTotal,
                 'storage' => collect($this->config->get('pulse.recorders.'.self::class.'.directories')) // @phpstan-ignore argument.templateType, argument.templateType
+                    ->filter(fn (string $directory) => ($this->pulse->rescue(fn () => disk_total_space($directory)) ?? false) !== false)
                     ->map(fn (string $directory) => [
                         'directory' => $directory,
                         'total' => $total = intval(round(disk_total_space($directory) / 1024 / 1024)), // MB

tests/Feature/Recorders/ServersTest.php

@@ -62,3 +62,19 @@
     Servers::detectCpuUsing(null);
     Servers::detectMemoryUsing(null);
 });
+
+it('skips missing filesystems when recording events', function () {
+    Pulse::handleExceptionsUsing(function () {
+        //
+    });
+    Config::set('pulse.recorders.'.Servers::class . '.directories', ['/', '/nonexistent']);
+    Date::setTestNow(Date::now()->startOfMinute());
+
+    event(new SharedBeat(CarbonImmutable::now(), 'instance-id'));
+    Pulse::ingest();
+
+    $value = Pulse::ignore(fn () => DB::table('pulse_values')->sole());
+
+    $payload = json_decode($value->value);
+    expect($payload->storage)->toHaveCount(1);
+});