Merge pull request #8 from patchlevel/fix-throwing-when-creating

Handle throwing exceptions when creating the aggregate better.

Author: DanielBadura

src/Test/AggregateRootTestCase.php

@@ -118,6 +118,8 @@ final public function assert(): self
             }
         } catch (Throwable $throwable) {
             $this->handleException($throwable);
+
+            return $this;
         }
 
         if (!$aggregate instanceof AggregateRoot) {

tests/Unit/Fixture/CreateProfileWithFailure.php

@@ -0,0 +1,14 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Patchlevel\EventSourcing\PhpUnit\Tests\Unit\Fixture;
+
+final readonly class CreateProfileWithFailure
+{
+    public function __construct(
+        public ProfileId $id,
+        public Email $email,
+    ) {
+    }
+}

tests/Unit/Fixture/Profile.php

@@ -37,6 +37,13 @@ public static function createProfile(CreateProfile $createProfile): self
         return $self;
     }
 
+    #[Handle]
+    public static function createProfileException(CreateProfileWithFailure $createProfile): void
+    {
+        $self = new self();
+        $self->throwException();
+    }
+
     #[Handle]
     public function visitProfile(VisitProfile $visitProfile, string|null $token = null): void
     {

tests/Unit/Test/AggregateRootTestCaseTest.php

@@ -11,6 +11,7 @@
 use Patchlevel\EventSourcing\PhpUnit\Test\NoAggregateCreated;
 use Patchlevel\EventSourcing\PhpUnit\Test\NoWhenProvided;
 use Patchlevel\EventSourcing\PhpUnit\Tests\Unit\Fixture\CreateProfile;
+use Patchlevel\EventSourcing\PhpUnit\Tests\Unit\Fixture\CreateProfileWithFailure;
 use Patchlevel\EventSourcing\PhpUnit\Tests\Unit\Fixture\Email;
 use Patchlevel\EventSourcing\PhpUnit\Tests\Unit\Fixture\Profile;
 use Patchlevel\EventSourcing\PhpUnit\Tests\Unit\Fixture\ProfileCreated;
@@ -40,7 +41,7 @@ public function testException(): void
             ->expectsException(ProfileError::class);
 
         $test->assert();
-        self::assertSame(2, $test::getCount());
+        self::assertSame(1, $test::getCount());
     }
 
     public function testExceptionMessage(): void
@@ -60,7 +61,7 @@ public function testExceptionMessage(): void
             ->expectsExceptionMessage('throwing so that you can catch it!');
 
         $test->assert();
-        self::assertSame(2, $test::getCount());
+        self::assertSame(1, $test::getCount());
     }
 
     public function testExceptionAndMessage(): void
@@ -81,7 +82,7 @@ public function testExceptionAndMessage(): void
             ->expectsExceptionMessage('throwing so that you can catch it!');
 
         $test->assert();
-        self::assertSame(3, $test::getCount());
+        self::assertSame(2, $test::getCount());
     }
 
     public function testExceptionUncatched(): void
@@ -100,6 +101,24 @@ public function testExceptionUncatched(): void
             );
 
         $this->expectException(ProfileError::class);
+        $test->assert();
+        self::assertSame(1, $test::getCount());
+    }
+
+    public function testExceptionWhenCreating(): void
+    {
+        $test = $this->getTester();
+
+        $test
+            ->when(
+                new CreateProfileWithFailure(
+                    ProfileId::fromString('1'),
+                    Email::fromString('hq@patchlevel.de'),
+                ),
+            )
+            ->expectsException(ProfileError::class)
+            ->expectsExceptionMessage('throwing so that you can catch it!');
+
         $test->assert();
         self::assertSame(2, $test::getCount());
     }