forked from akkadotnet/akka.net
-
Notifications
You must be signed in to change notification settings - Fork 2
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Added reproduction for missing
Stash.Stash
messages in Akka.,Persis…
…tence actors reproduces akkadotnet#7373
- Loading branch information
1 parent
0cbddb5
commit 35e350a
Showing
1 changed file
with
81 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,81 @@ | ||
// ----------------------------------------------------------------------- | ||
// <copyright file="Bugfix7373Specs.cs" company="Akka.NET Project"> | ||
// Copyright (C) 2009-2024 Lightbend Inc. <http://www.lightbend.com> | ||
// Copyright (C) 2013-2024 .NET Foundation <https://github.com/akkadotnet/akka.net> | ||
// </copyright> | ||
// ----------------------------------------------------------------------- | ||
|
||
using System.Threading.Tasks; | ||
using Akka.Actor; | ||
using Akka.TestKit; | ||
using Xunit; | ||
using Xunit.Abstractions; | ||
|
||
namespace Akka.Persistence.Tests; | ||
|
||
public class Bugfix7373Specs : AkkaSpec | ||
{ | ||
public Bugfix7373Specs(ITestOutputHelper output) : base(output) | ||
{ | ||
} | ||
|
||
/// <summary> | ||
/// Reproduction for https://github.com/akkadotnet/akka.net/issues/7373 | ||
/// </summary> | ||
[Fact] | ||
public async Task ShouldDeliverAllStashedMessages() | ||
{ | ||
// arrange | ||
var actor = Sys.ActorOf(Props.Create<MinimalStashingActor>()); | ||
|
||
// act | ||
var msg = new Msg(1); | ||
actor.Tell(msg); | ||
actor.Tell(msg); | ||
|
||
actor.Tell("Initialize"); | ||
|
||
// assert | ||
await ExpectMsgAsync($"Processed: {msg}"); | ||
await ExpectMsgAsync($"Processed: {msg}"); | ||
} | ||
|
||
public sealed record Msg(int Id); | ||
|
||
public class MinimalStashingActor : UntypedPersistentActor, IWithStash | ||
{ | ||
public override string PersistenceId => "minimal-stashing-actor"; | ||
|
||
protected override void OnCommand(object message) | ||
{ | ||
Sender.Tell($"Processed: {message}"); | ||
} | ||
|
||
private void Ready(object message) | ||
{ | ||
switch (message) | ||
{ | ||
case "Initialize": | ||
Persist("init", e => | ||
{ | ||
Stash.UnstashAll(); // Unstash all stashed messages | ||
Become(OnCommand); // Transition to ready state | ||
}); | ||
break; | ||
default: | ||
Stash.Stash(); // Stash messages until initialized | ||
break; | ||
} | ||
} | ||
|
||
protected override void OnRecover(object message) | ||
{ | ||
switch (message) | ||
{ | ||
case RecoveryCompleted: | ||
Become(Ready); | ||
break; | ||
} | ||
} | ||
} | ||
} |