From 5d48844670b6b26a83c4ace6ae606ab8248d97f8 Mon Sep 17 00:00:00 2001 From: Evan You Date: Fri, 9 Feb 2024 09:02:52 +0800 Subject: [PATCH] test: fix unexpected warning check when asserting same message for both conditions --- scripts/setup-vitest.ts | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/scripts/setup-vitest.ts b/scripts/setup-vitest.ts index d5c65f0d3c0..53e7f5fff56 100644 --- a/scripts/setup-vitest.ts +++ b/scripts/setup-vitest.ts @@ -4,9 +4,9 @@ vi.stubGlobal('MathMLElement', class MathMLElement {}) expect.extend({ toHaveBeenWarned(received: string) { - asserted.add(received) const passed = warn.mock.calls.some(args => args[0].includes(received)) if (passed) { + asserted.add(received) return { pass: true, message: () => `expected "${received}" not to have been warned.`, @@ -25,10 +25,10 @@ expect.extend({ }, toHaveBeenWarnedLast(received: string) { - asserted.add(received) const passed = warn.mock.calls[warn.mock.calls.length - 1][0].includes(received) if (passed) { + asserted.add(received) return { pass: true, message: () => `expected "${received}" not to have been warned last.`, @@ -44,7 +44,6 @@ expect.extend({ }, toHaveBeenWarnedTimes(received: string, n: number) { - asserted.add(received) let found = 0 warn.mock.calls.forEach(args => { if (args[0].includes(received)) { @@ -53,6 +52,7 @@ expect.extend({ }) if (found === n) { + asserted.add(received) return { pass: true, message: () => `expected "${received}" to have been warned ${n} times.`,