Kapitalmodell rechnet Waehrungen um, statt sie zu addieren
Solange jedes Instrument in USD notierte, war die Frage gegenstandslos. Mit waehrungsfaehigen Instrumenten ist sie es nicht mehr: CapitalService summierte BoundCapital ueber alle Positionen unabhaengig von deren Waehrung und verglich das Ergebnis mit NetLiquidation in Kontobasiswaehrung. Dasselbe galt fuer die Konzentration je Basiswert und fuer die offenen Reservierungen, die von einer Groesse in Basiswaehrung abgezogen wurden. Auf dem Testkonto ist die Basis EUR und USD steht bei 0,8611: jede USD-Position ging bisher rund 16 Prozent zu hoch in das verteilbare Kapital ein. Die Richtung des Fehlers wechselt mit dem Kurs - bei einer starken Fremdwaehrung wird die Auslastung unterschaetzt und ein Buch darf mehr aufmachen als es soll. Die Kurse kommen aus dem Ledger des Brokers ($LEDGER:ALL, Tag ExchangeRate), nicht aus einer eigenen Quelle. Das ist Absicht: NetLiquidation und TotalCashValue sind mit genau diesen Kursen gebildet. Mit einem fremden Kurs verglichen wir den Positionswert gegen eine Kontogroesse, die anders gerechnet wurde. Fuer die Buchhaltung gilt das Gegenteil - dort ist der amtliche Tageskurs der richtige, das ist ein anderer Zweck (#321). Fehlt ein Kurs, wird nicht geraten: der Snapshot meldet FxComplete=false, der Positionswert gilt als Untergrenze, und der Kauf wird angehalten. Schliessen bleibt moeglich. Eine Reservierung ohne Kurs wird dagegen roh angesetzt statt uebersprungen - sie zu ignorieren hiesse, gebundenes Geld als frei zu fuehren. Gemessen am Paper-Gateway: "Konto: Umrechnungskurse EUR 1, USD 0.8611 (Basis EUR)". 388 Tests gruen. Behebt #480. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
@@ -269,4 +269,37 @@ public class ReservationServiceTests
|
||||
|
||||
(await sut.RecoverAsync([])).Should().BeEmpty();
|
||||
}
|
||||
|
||||
// ─── Offene Betraege je Waehrung (#480) ──────────────────────────────────────
|
||||
|
||||
[Fact]
|
||||
public async Task OpenAmounts_AreGroupedByCurrency_NotSummedAcrossThem()
|
||||
{
|
||||
var sut = CreateSut(out _);
|
||||
var sap = InstrumentRef.ForStock("SAP", "EUR");
|
||||
|
||||
await sut.ReserveAsync(Ow, Aapl, TradeSide.Buy, 10, "USD", "sig-1", Bind(1_000m));
|
||||
await sut.ReserveAsync(Ow, sap, TradeSide.Buy, 5, "EUR", "sig-2", Bind( 500m));
|
||||
await sut.ReserveAsync(Ow, Aapl, TradeSide.Buy, 2, "USD", "sig-3", Bind( 200m));
|
||||
|
||||
var byCurrency = await sut.GetOpenAmountByCurrencyAsync();
|
||||
|
||||
byCurrency.Should().HaveCount(2);
|
||||
byCurrency["USD"].Should().Be(1_200m);
|
||||
byCurrency["EUR"].Should().Be(500m);
|
||||
|
||||
// Die alte Gesamtsumme addiert weiterhin ueber Waehrungen hinweg - genau deshalb darf
|
||||
// sie nicht mehr als Geldbetrag verwendet werden.
|
||||
(await sut.GetOpenAmountAsync()).Should().Be(1_700m);
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public async Task ClosedReservations_DoNotShowUpPerCurrency()
|
||||
{
|
||||
var sut = CreateSut(out _);
|
||||
var result = await sut.ReserveAsync(Ow, Aapl, TradeSide.Buy, 10, "USD", "sig-1", Bind(1_000m));
|
||||
await sut.CancelAsync(result.Reservation!.Id, "Test");
|
||||
|
||||
(await sut.GetOpenAmountByCurrencyAsync()).Should().BeEmpty();
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user