fix: dont run properties through accessors when serialising

Author: ryangjchandler

src/Support/ModelAttributeFormatter.php

@@ -3,24 +3,23 @@
 namespace Orbit\Support;
 
 use BackedEnum;
+use Carbon\Carbon;
 use Illuminate\Database\Eloquent\Model;
+use Illuminate\Support\Arr;
 use Orbit\Contracts\Orbit;
 
 class ModelAttributeFormatter
 {
     public static function format(Orbit&Model $model, array $attributes): array
     {
-        $formatted = [];
+        return Arr::map($attributes, static function (mixed $value, string $key) use ($model) {
+            $cast = $model->{$key};
 
-        foreach ($attributes as $key => $value) {
-            $value = $model->{$key};
-
-            $formatted[$key] = match (true) {
-                $value instanceof BackedEnum => $value->value,
+            return match (true) {
+                $cast instanceof BackedEnum => $cast->value,
+                $cast instanceof Carbon => $cast->toIso8601String(),
                 default => $value,
             };
-        }
-
-        return $formatted;
+        });
     }
 }

tests/Feature/Orbital/CreateTest.php

@@ -15,3 +15,14 @@
         title: 'Example Post'
         MD);
 });
+
+it('does not use accessors when serialising saved data', function () {
+    $post = Post::create([
+        'title' => 'hello',
+    ]);
+
+    expect(base_path("content/posts/{$post->id}.md"))
+        ->toBeFile()
+        ->and(file_get_contents(base_path("content/posts/{$post->id}.md")))
+        ->not->toContain('Hello');
+});

tests/Feature/Orbital/SoftDeleteTest.php

@@ -16,7 +16,7 @@
         ->toBeFile()
         ->and(file_get_contents(base_path("content/categories/{$category->id}.md")))
         ->toContain(<<<MD
-        deleted_at: {$category->deleted_at->toIso8601String()}
+        deleted_at: '{$category->deleted_at->toIso8601String()}'
         MD);
 });
 
@@ -34,7 +34,7 @@
         ->toBeFile()
         ->and(file_get_contents(base_path("content/categories/{$category->id}.md")))
         ->toContain(<<<MD
-        deleted_at: {$category->deleted_at->toIso8601String()}
+        deleted_at: '{$category->deleted_at->toIso8601String()}'
         MD);
 
     $category->restore();

tests/Fixtures/Models/Post.php

@@ -2,8 +2,10 @@
 
 namespace Tests\Fixtures\Models;
 
+use Illuminate\Database\Eloquent\Casts\Attribute;
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Support\Str;
 use Orbit\Concerns\Orbital;
 use Orbit\Contracts\Orbit;
 
@@ -19,4 +21,9 @@ public function schema(Blueprint $table): void
         $table->string('title');
         $table->longText('content')->nullable();
     }
+
+    public function title(): Attribute
+    {
+        return Attribute::get(fn (string $value) => Str::headline($value));
+    }
 }