From 605c6f14f47ada8d36a55f2f5b60a83410245901 Mon Sep 17 00:00:00 2001 From: "Jeremy D. Miller" Date: Tue, 19 May 2026 05:44:28 -0500 Subject: [PATCH] Marten#4466: unskip Bug_4441_force_catch_up_with_outbox MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit JasperFx PR #285 ("Stop daemon-internal cancellations from leaking into CatchUpAsync") shipped in JasperFx.Events 2.0.0-alpha.11 and is consumed by Marten's current alpha.13 pin (#4475 Phase 2). The daemon-internal cancellation flake that motivated the re-skip in #4462/#4463 is gone — confirmed locally 5/5 on both net9.0 and net10.0. Test-only change. Closes #4466. Co-Authored-By: Claude Opus 4.7 (1M context) --- .../Bugs/Bug_4441_force_catch_up_with_outbox.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/EventSourcingTests/Bugs/Bug_4441_force_catch_up_with_outbox.cs b/src/EventSourcingTests/Bugs/Bug_4441_force_catch_up_with_outbox.cs index cdfd5d7c8e..4823f17434 100644 --- a/src/EventSourcingTests/Bugs/Bug_4441_force_catch_up_with_outbox.cs +++ b/src/EventSourcingTests/Bugs/Bug_4441_force_catch_up_with_outbox.cs @@ -81,7 +81,7 @@ public async Task force_catch_up_returns_for_async_daemon_without_side_effects() exceptions.ShouldBeEmpty(); } - [Fact(Timeout = 60000, Skip = "Re-skipped after the #4463 fix didn't fully stabilize this in CI — see https://github.com/JasperFx/marten/issues/4462. The remaining flake is a daemon-internal cancellation that surfaces as an AggregateException with an OCE inside well before the test's CTS would fire (failing run terminated in ~176ms with the test CTS set to 45s). The next step is the deeper-fix path in #4462 — auditing the GroupedProjectionExecution / per-shard internal CTS contract so daemon-lifecycle cancellations don't leak into ForceAllMartenDaemonActivityToCatchUpAsync's exceptions list. Passes locally 5/5.")] + [Fact(Timeout = 60000)] public async Task force_catch_up_invokes_message_batch_lifecycle_with_custom_outbox() { var outbox = new RecordingOutbox();