-
Notifications
You must be signed in to change notification settings - Fork 3
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
Changed TestableAggregateRoot to be explicit on commandhandling failures #44
Conversation
Ensured TestableAggregateRoot works properly based on a starting 'given' (the actor was created in when, that is now part of given that is actually class constructor work) Added tests to cover the functionality of TestableAggregateRoot
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.
Only some small comments, but further looking good!
@@ -6,6 +6,8 @@ package io.cafienne.bounded.test | |||
|
|||
import java.util.concurrent.atomic.AtomicInteger | |||
|
|||
import io.cafienne.bounded.test.TestableAggregateRoot.{CommandHandlingException, IllegalCommandException} | |||
|
|||
import scala.reflect.ClassTag | |||
//import scala.reflect.runtime.universe |
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.
Commented import can be removed?
import io.cafienne.bounded.test.TestAggregateRoot.TestAggregateRootState | ||
import scala.collection.immutable.Seq | ||
|
||
//object DomainProtocol { |
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.
Why commented?
import io.cafienne.bounded.{BuildInfo, RuntimeInfo} | ||
import io.cafienne.bounded.aggregate.{AggregateRootId, CommandMetaData, MetaData} | ||
import io.cafienne.bounded.test.TestableAggregateRoot.{CommandHandlingException, IllegalCommandException} | ||
//import io.cafienne.bounded.test.DomainProtocol.InitialStateCreated |
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.
Clean imports plz.
Updated code to remove the issues around the comments. Commented code is removed and/or enabled. Thanks. Will merge this when all is fine. (and publish a new version) |
@olger You can merge this PR, and publish a new version. Don't forget to update the release notes! |
Ensured TestableAggregateRoot works properly based on a starting 'given' (the actor was created in when, that is now part of given that is actually class constructor work)
Added tests to cover the functionality of TestableAggregateRoot