tests expect more notifications on take offer

woodser · Dec 18, 2024 · 412b05c · 412b05c
1 parent 7b88f7d
commit 412b05c
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/HavenoClient.test.ts b/src/HavenoClient.test.ts
@@ -2909,7 +2909,7 @@ async function takeOffer(ctxP: Partial<TradeContext>): Promise<TradeInfo> {
   // maker is notified that offer is taken
   await wait(ctx.maxTimePeerNoticeMs);
   const tradeNotifications = getNotifications(makerNotifications, NotificationMessage.NotificationType.TRADE_UPDATE, takerTrade.getTradeId());
-  expect(tradeNotifications.length).toBe(1);
+  expect(tradeNotifications.length).toBeGreaterThanOrEqual(1); // TODO: client is missing notifications when deposits are confirmed and unlocked
   assert(moneroTs.GenUtils.arrayContains(["DEPOSITS_PUBLISHED", "DEPOSITS_CONFIRMED", "DEPOSITS_UNLOCKED"], tradeNotifications[0].getTrade()!.getPhase()), "Unexpected trade phase: " + tradeNotifications[0].getTrade()!.getPhase());
   expect(tradeNotifications[0].getTitle()).toEqual("Offer Taken");
   expect(tradeNotifications[0].getMessage()).toEqual("Your offer " + ctx.offerId + " has been accepted");