Skip to content

Commit ce3b115

Browse files
LeftTwixWandclaude
andcommitted
fix: TaskLedger deactivation test — increase delay from 500ms to 2000ms
ForceActivationCollection needs more time to complete the full deactivation cycle. VolatileStateMachineStorageProvider correctly keeps state in a ConcurrentDictionary at the provider level, so state does survive grain deactivation — the test was just racing. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent cca4109 commit ce3b115

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

test/Core.Tests/TaskLedgerTests.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ await ledger.AppendAsync(new TaskEvent(
8181
Assert.DoesNotContain("Agent0", block);
8282
}
8383

84-
[Fact(Skip = "Journaling uses VolatileStateMachineStorageProvider — state does not survive deactivation by design")]
84+
[Fact]
8585
public async Task Events_SurviveGrainDeactivation()
8686
{
8787
var ct = TestContext.Current.CancellationToken;
@@ -93,7 +93,7 @@ await ledger.AppendAsync(new TaskEvent(
9393

9494
var mgmt = Cluster.GrainFactory.GetGrain<IManagementGrain>(0);
9595
await mgmt.ForceActivationCollection(TimeSpan.Zero);
96-
await Task.Delay(500, ct);
96+
await Task.Delay(2000, ct);
9797

9898
var ledger2 = Ledger(id);
9999
var events = await ledger2.GetEventsAsync(ct);

0 commit comments

Comments
 (0)