-
Notifications
You must be signed in to change notification settings - Fork 177
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bugfixes in event-sourced-domain-with-tests #344
Conversation
…ario->getEvents method: array_map only accepts an array and not an iterator. Using iterator_to_array() now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tnx for spotting this! Would you mind adding the examples directory to the phpunit.xml.dist? This way we make sure we also run those tests.
@@ -34,7 +34,7 @@ public static function invite($invitationId, $name) | |||
* | |||
* {@inheritdoc} | |||
*/ | |||
public function getAggregateRootId() | |||
public function getAggregateRootId() : string |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
apparently the CS fixers fails. Can you remove the space before the colon? "public function getAggregateRootId(): string"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done, as well as added the examples folder to phpunit.xml.dist
@@ -23,7 +23,7 @@ public function setUp() | |||
$this->generator = new Broadway\UuidGenerator\Rfc4122\Version4Generator(); | |||
} | |||
|
|||
protected function getAggregateRootClass() | |||
protected function getAggregateRootClass():string |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
protected function getAggregateRootClass(): string
sorry, the CS fixer is picky
Thank you, @renedekat ! |
Now I just have to figure how to use it in a project :) I'm new to CQRS and Event Sourcing. |
Fixed missing return types required in strict mode.
Fixed bug in Scenario->getEvents method: array_map only accepts an array and not an iterator. Using iterator_to_array() now.
Renamed InvitationTest class name to match filename.