From 40a9c109ecb16c1b23107a70ae4d6bd44c555c1c Mon Sep 17 00:00:00 2001 From: Aaron Stannard Date: Fri, 8 May 2026 16:35:37 +0000 Subject: [PATCH] fix: add pipeline init gate to wrong-requester approval tests The two wrong-requester tests were missing the CapturedOptions initialization wait that all other tests in the file use. On Windows CI, pipeline materialization can exceed the 3s default AwaitAssertAsync timeout, causing the first assertion to consume the entire budget before the actor even finishes initializing. --- .../Channels/Contracts/SessionBindingContractTests.cs | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/Netclaw.Actors.Tests/Channels/Contracts/SessionBindingContractTests.cs b/src/Netclaw.Actors.Tests/Channels/Contracts/SessionBindingContractTests.cs index abf3f698..c5a977d7 100644 --- a/src/Netclaw.Actors.Tests/Channels/Contracts/SessionBindingContractTests.cs +++ b/src/Netclaw.Actors.Tests/Channels/Contracts/SessionBindingContractTests.cs @@ -603,6 +603,8 @@ public async Task Button_approval_from_wrong_requester_posts_warning() ]); var actor = CreateBindingActor(sid, pipeline, detector); + await AwaitAssertAsync(() => Assert.NotNull(pipeline.CapturedOptions), + cancellationToken: ct); await AwaitAssertAsync(() => { var texts = GetPostedTexts(); @@ -650,6 +652,8 @@ public async Task Text_approval_from_wrong_requester_posts_warning() ]); var actor = CreateBindingActor(sid, pipeline, detector); + await AwaitAssertAsync(() => Assert.NotNull(pipeline.CapturedOptions), + cancellationToken: ct); await AwaitAssertAsync(() => { var texts = GetPostedTexts();