From 82aef467b33484a6b5376a2564482e2588ec0ff2 Mon Sep 17 00:00:00 2001 From: Shakker Date: Fri, 8 May 2026 10:45:10 +0100 Subject: [PATCH] test: assert matrix sas notice messages --- .../matrix/src/matrix/monitor/events.test.ts | 28 +++++++++++-------- 1 file changed, 17 insertions(+), 11 deletions(-) diff --git a/extensions/matrix/src/matrix/monitor/events.test.ts b/extensions/matrix/src/matrix/monitor/events.test.ts index 7916ef80b9c..24e8250b9e8 100644 --- a/extensions/matrix/src/matrix/monitor/events.test.ts +++ b/extensions/matrix/src/matrix/monitor/events.test.ts @@ -593,8 +593,10 @@ describe("registerMatrixMonitorEvents verification routing", () => { await flushTasks(); const bodies = getSentNoticeBodies(sendMessage); - expect(bodies.some((body) => body.includes("SAS emoji:"))).toBe(true); - expect(bodies.some((body) => body.includes("SAS decimal: 6158 1986 3513"))).toBe(true); + expect(bodies).toEqual(expect.arrayContaining([expect.stringContaining("SAS emoji:")])); + expect(bodies).toEqual( + expect.arrayContaining([expect.stringContaining("SAS decimal: 6158 1986 3513")]), + ); }); it("rehydrates an in-progress DM verification before resolving SAS notices", async () => { @@ -961,7 +963,7 @@ describe("registerMatrixMonitorEvents verification routing", () => { await vi.waitFor(() => { const bodies = getSentNoticeBodies(sendMessage); - expect(bodies.some((body) => body.includes("SAS emoji:"))).toBe(true); + expect(bodies).toEqual(expect.arrayContaining([expect.stringContaining("SAS emoji:")])); }); } finally { vi.useRealTimers(); @@ -1215,11 +1217,13 @@ describe("registerMatrixMonitorEvents verification routing", () => { }); await flushTasks(); - expect( - getSentNoticeBodies(sendMessage).some((body) => body.includes("SAS decimal: 6158 1986 3513")), - ).toBe(true); + expect(getSentNoticeBodies(sendMessage)).toEqual( + expect.arrayContaining([expect.stringContaining("SAS decimal: 6158 1986 3513")]), + ); const bodies = getSentNoticeBodies(sendMessage); - expect(bodies.some((body) => body.includes("SAS decimal: 1111 2222 3333"))).toBe(false); + expect(bodies).not.toEqual( + expect.arrayContaining([expect.stringContaining("SAS decimal: 1111 2222 3333")]), + ); }); it("preserves strict-room SAS fallback when active DM inspection cannot resolve a room", async () => { @@ -1321,11 +1325,13 @@ describe("registerMatrixMonitorEvents verification routing", () => { }); await flushTasks(); - expect( - getSentNoticeBodies(sendMessage).some((body) => body.includes("SAS decimal: 6158 1986 3513")), - ).toBe(true); + expect(getSentNoticeBodies(sendMessage)).toEqual( + expect.arrayContaining([expect.stringContaining("SAS decimal: 6158 1986 3513")]), + ); const bodies = getSentNoticeBodies(sendMessage); - expect(bodies.some((body) => body.includes("SAS decimal: 1111 2222 3333"))).toBe(false); + expect(bodies).not.toEqual( + expect.arrayContaining([expect.stringContaining("SAS decimal: 1111 2222 3333")]), + ); }); it("does not emit SAS notices for cancelled verification events", async () => {