Skip to content

Commit 7b14d23

Browse files
committed
EffectLoopingWorks: raise BusyWait timeouts to 30s
The default 10s threshold occasionally trips on the SqlServer CI runner under 12-way parallel test execution. Bumping to 30s gives the workflow restart + replay cycle more headroom without changing test semantics.
1 parent d4af08c commit 7b14d23

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

  • Core/Cleipnir.ResilientFunctions.Tests/TestTemplates/FunctionTests

Core/Cleipnir.ResilientFunctions.Tests/TestTemplates/FunctionTests/EffectTests.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1021,8 +1021,8 @@ await elms.CaptureEach(
10211021

10221022
for (var i = 0; i < 6; i++)
10231023
{
1024-
await BusyWait.Until(() => flag.Value == i);
1025-
await cp.BusyWaitUntil(c => c.Status == Status.Suspended);
1024+
await BusyWait.Until(() => flag.Value == i, maxWait: TimeSpan.FromSeconds(30));
1025+
await cp.BusyWaitUntil(c => c.Status == Status.Suspended, maxWait: TimeSpan.FromSeconds(30));
10261026
var storedEffects = await effectStore.GetEffectResults(registration.MapToStoredId(id.Instance));
10271027
storedEffects.Any(e => e.Alias == "Before").ShouldBeTrue();
10281028
storedEffects.Any(e => e.Alias == "Loop").ShouldBeTrue();
@@ -1032,7 +1032,7 @@ await elms.CaptureEach(
10321032
await messageWriter.AppendMessage(i.ToString());
10331033
}
10341034

1035-
await cp.BusyWaitUntil(c => c.Status == Status.Succeeded);
1035+
await cp.BusyWaitUntil(c => c.Status == Status.Succeeded, maxWait: TimeSpan.FromSeconds(30));
10361036

10371037
iterations.SequenceEqual([0, 1, 2, 3, 4, 5]).ShouldBeTrue();
10381038
}

0 commit comments

Comments
 (0)