Add Tests and fix style

Author: sebastiandittrich

src/StoredEvents/Repositories/EloquentStoredEventRepository.php

@@ -93,10 +93,9 @@ public function persist(ShouldBeStored $event, string $uuid = null): StoredEvent
             throw new InvalidStoredEvent();
         }
 
+        $serializerClass = EventSerializer::class;
         if ($serializerAttribute = $reflectionClass->getAttributes(EventSerializerAttribute::class)[0] ?? null) {
-            $serializerClass = ($serializerAttribute->newInstance())->serializerClass;
-        } else {
-            $serializerClass = EventSerializer::class;
+            $serializerClass = $serializerAttribute->newInstance()->serializerClass;
         }
 
         $eloquentStoredEvent->setRawAttributes([

tests/EloquentStoredEventRepositoryTest.php

@@ -8,6 +8,7 @@
 use Spatie\EventSourcing\StoredEvents\Repositories\EloquentStoredEventRepository;
 use Spatie\EventSourcing\Tests\TestClasses\AggregateRoots\AccountAggregateRoot;
 use Spatie\EventSourcing\Tests\TestClasses\AggregateRoots\StorableEvents\MoneyAdded;
+use Spatie\EventSourcing\Tests\TestClasses\Events\EventWithCustomSerializer;
 
 it('can get the latest version id for a given aggregate uuid', function () {
     $eloquentStoredEventRepository = new EloquentStoredEventRepository();
@@ -37,3 +38,13 @@
 
     assertSame($originalEvent, $storedEvent->event);
 });
+
+it('uses the custom serializer if one is set', function () {
+    $eloquentStoredEventRepository = app(EloquentStoredEventRepository::class);
+
+    $originalEvent = new EventWithCustomSerializer('default message');
+    $storedEvent = $eloquentStoredEventRepository->persist($originalEvent, 'uuid-1', 1);
+
+    $eventFromDatabase = $eloquentStoredEventRepository->find($storedEvent->id)->event;
+    assertSame('message set by custom serializer', $eventFromDatabase->message);
+});

tests/TestClasses/EventSerializer/DummySerializer.php

@@ -0,0 +1,15 @@
+<?php
+
+namespace Spatie\EventSourcing\Tests\TestClasses\EventSerializer;
+
+use DateTimeZone;
+use Spatie\EventSourcing\EventSerializers\JsonEventSerializer;
+use Spatie\EventSourcing\StoredEvents\ShouldBeStored;
+
+class DummySerializer extends JsonEventSerializer
+{
+    public function serialize(ShouldBeStored $event): string
+    {
+        return '{"message":"message set by custom serializer"}';
+    }
+}

tests/TestClasses/Events/EventWithCustomSerializer.php

@@ -0,0 +1,18 @@
+<?php
+
+namespace Spatie\EventSourcing\Tests\TestClasses\Events;
+
+use Spatie\EventSourcing\Attributes\EventSerializer;
+use Spatie\EventSourcing\StoredEvents\ShouldBeStored;
+use Spatie\EventSourcing\Tests\TestClasses\EventSerializer\DummySerializer;
+
+#[EventSerializer(DummySerializer::class)]
+class EventWithCustomSerializer extends ShouldBeStored
+{
+    public string $message;
+
+    public function __construct(string $message)
+    {
+        $this->message = $message;
+    }
+}