-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
add more tests for receiving PoisonPill in various states, #24687 #26577
Conversation
Test PASSed. |
@@ -123,7 +123,8 @@ private[akka] final class ReplayingEvents[C, E, S]( | |||
private def onCommand(cmd: InternalProtocol): Behavior[InternalProtocol] = { | |||
// during recovery, stash all incoming commands | |||
if (state.receivedPoisonPill) { | |||
if (setup.settings.logOnStashing) setup.log.debug("Discarding message [{}], because actor is to be stopped", cmd) | |||
if (setup.settings.logOnStashing) | |||
setup.log.debug("Discarding message [{}], because actor is to be stopped", cmd) |
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.
setup.log.debug("Discarding message [{}], because actor is to be stopped", cmd) | |
setup.log.debug("Discarding message [{}] because actor is to be stopped.", cmd) |
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.
fixed, and a few more of the same kind
else onCommand(cmd) | ||
if (receivedPoisonPill) { | ||
if (setup.settings.logOnStashing) | ||
setup.log.debug("Discarding message [{}], because actor is to be stopped", cmd) |
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.
setup.log.debug("Discarding message [{}], because actor is to be stopped", cmd) | |
setup.log.debug("Discarding message [{}] because actor is to be stopped.", cmd) |
@@ -68,14 +73,16 @@ object EventSourcedBehaviorStashSpec { | |||
|
|||
final case class State(value: Int, active: Boolean) | |||
|
|||
def counter(persistenceId: PersistenceId): Behavior[Command[_]] = | |||
def counter(persistenceId: PersistenceId, signalProbe: Option[ActorRef[String]] = None): Behavior[Command[_]] = |
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 ok with the default param because it's in a test.
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.
LGTM with 2 minor changes
60bac1c
to
980891c
Compare
will merge after validation |
Test FAILed. |
PLS BUILD |
Test PASSed. |
980891c
to
e6775d7
Compare
Test PASSed. |
Refs #24687