Orders, die beim Broker liegen, gelten nicht mehr als gescheitert

Am geschlossenen Markt gemessen: IBKR nimmt eine Order an und merkt sie fuer die naechste Eroeffnung vor. Dazu schickt es Code 399 - eine Mitteilung, deren Text mit "Warning" beginnt. Der Adapter las jede Meldung aus dem Fehler-Callback als Ablehnung, meldete "Order abgelehnt", und das Kapitalmodell gab die Bindung frei. Auf dem Testkonto lagen dadurch 67 Orders ueber 306 BAS, 140 SAP und 43 ALV beim Broker, vorgemerkt fuer Montag 09:00, ohne eine einzige Reservierung dahinter. OrderResult kannte nur "erfolgreich" und "gescheitert". Der haeufigste Fall passte in keines von beiden. Es gibt jetzt einen dritten Ausgang - Working: abgesendet, Ausgang offen. Die Trennlinie ist, ob BEWIESEN ist, dass beim Broker nichts liegt; ohne Beweis bleibt das Kapital gebunden. Eine Bindung, die zu lange steht, kostet Spielraum und faellt auf. Eine, die zu frueh verschwindet, kostet Geld und faellt nicht auf. - 399/404 sind Ordermeldungen, keine Ablehnungen; nur 201/202/203 u. a. beenden eine Order beweisbar. Ein Verbindungsabbruch beendet unsere Sicht, nicht die Order. - IsDeadStatus ist enger als IsTerminalStatus: "Inactive" ist kein Beweis. - ExecutionService haelt die Bindung bei Working (MarkSubmittedAsync statt CancelReservationAsync) und journalisiert Pending/OrderWorking statt Failed. - Neu lesbar: das Orderbuch des Brokers (reqAllOpenOrders) samt Storno. --open-orders stellt Orderbuch und eigene Bindungen gegenueber, --cancel-order <ID>|all --confirm nimmt zurueck (Sammelstorno nur im Paper-Modus). - Der Wiederanlauf-Abgleich lief bisher gegen eine leere Orderliste und haette ab jetzt jede gesunde Bindung in Quarantaene gestellt. Er sitzt jetzt im ReservationWatchdogWorker, hinter dem Beweis, dass der Broker antwortet. 457 Tests gruen. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-09-06 12:27:32 +02:00
parent 1dac482cc7
commit e7e3970dec
18 changed files with 837 additions and 39 deletions
@@ -299,4 +299,79 @@ public class ExecutionServiceTests
            Arg.Any<string>(), Arg.Any<InstrumentRef>(), Arg.Any<TradeSide>(),
            Arg.Any<int>(), Arg.Any<decimal>(), Arg.Any<string>(), Arg.Any<string?>(), Arg.Any<CancellationToken>());
    }
+
+    // ─── Die Order, die beim Broker liegt (#489) ─────────────────────────────
+
+    /// <summary>
+    /// <b>Der Test, um den es bei #489 geht.</b> Am 2026-09-06 wurden 67 Orders bei geschlossener
+    /// Börse angenommen und für Montag vorgemerkt, während der Adapter sie als abgelehnt meldete und
+    /// jede Bindung freigegeben wurde. Das Geld war wieder im Topf, die Orders lebten weiter.
+    /// </summary>
+    [Fact]
+    public async Task WorkingOrder_KeepsTheCapitalBound()
+    {
+        ArrangeHappyPath();
+        _broker.PlaceOrderAsync(Arg.Any<OrderRequest>(), Arg.Any<CancellationToken>())
+               .Returns(Task.FromResult(OrderResult.Working("O-77", "PreSubmitted", "wartet auf Eröffnung")));
+
+        var result = await CreateSut().ExecuteAsync(BuySignal);
+
+        result.Action.Should().Be("PENDING");
+        result.IsPending.Should().BeTrue();
+        result.Executed.Should().BeFalse("nichts ist ausgeführt – PENDING ist kein Erfolg");
+
+        await _capital.Received(1).MarkSubmittedAsync(1, "O-77", Arg.Any<CancellationToken>());
+        await _capital.DidNotReceive().CancelReservationAsync(
+            Arg.Any<long>(), Arg.Any<string>(), Arg.Any<CancellationToken>());
+    }
+
+    [Fact]
+    public async Task WorkingOrder_BooksNothing()
+    {
+        // Eine Position zu buchen, bevor ausgeführt wurde, wäre der Fehler in die andere Richtung.
+        ArrangeHappyPath();
+        _broker.PlaceOrderAsync(Arg.Any<OrderRequest>(), Arg.Any<CancellationToken>())
+               .Returns(Task.FromResult(OrderResult.Working("O-77", "PreSubmitted", "wartet")));
+
+        await CreateSut().ExecuteAsync(BuySignal);
+
+        await _portfolio.DidNotReceive().RecordFillAsync(
+            Arg.Any<string>(), Arg.Any<InstrumentRef>(), Arg.Any<TradeSide>(),
+            Arg.Any<int>(), Arg.Any<decimal>(), Arg.Any<string>(), Arg.Any<string?>(), Arg.Any<CancellationToken>());
+        await _capital.DidNotReceive().SettleReservationAsync(
+            Arg.Any<long>(), Arg.Any<int>(), Arg.Any<decimal>(), Arg.Any<string>(), Arg.Any<CancellationToken>());
+    }
+
+    [Fact]
+    public async Task RejectedOrder_StillReleasesTheCapital()
+    {
+        // Die Gegenprobe: Bei einer beweisbar abgelehnten Order MUSS das Geld zurück. Sonst hätte
+        // der Fix aus #489 nur die eine Klemme gegen die andere getauscht.
+        ArrangeHappyPath();
+        _broker.PlaceOrderAsync(Arg.Any<OrderRequest>(), Arg.Any<CancellationToken>())
+               .Returns(Task.FromResult(OrderResult.Fail("Order abgelehnt")));
+
+        await CreateSut().ExecuteAsync(BuySignal);
+
+        await _capital.Received(1).CancelReservationAsync(
+            1, Arg.Any<string>(), Arg.Any<CancellationToken>());
+        await _capital.DidNotReceive().MarkSubmittedAsync(
+            Arg.Any<long>(), Arg.Any<string>(), Arg.Any<CancellationToken>());
+    }
+
+    [Fact]
+    public async Task WorkingOrder_IsJournalledAsPending_NotAsFailure()
+    {
+        // Das Journal ist die Grundlage der Supervisor-Forensik. Stuende dort "Failed", suchte
+        // später jemand einen Fehler, den es nicht gab - und uebersaehe die offene Order.
+        ArrangeHappyPath();
+        _broker.PlaceOrderAsync(Arg.Any<OrderRequest>(), Arg.Any<CancellationToken>())
+               .Returns(Task.FromResult(OrderResult.Working("O-77", "PreSubmitted", "wartet")));
+
+        await CreateSut().ExecuteAsync(BuySignal);
+
+        _journal.Received().Write(Arg.Is<CoreDecisionRecord>(r =>
+            r.Decision == TradeDecision.Pending && r.Reason == DecisionReason.OrderWorking));
+        _orderLog.Received().Write(Arg.Is<CoreOrderEvent>(e => e.EventType == OrderEventType.Placed));
+    }
 }