diff --git a/packages/excalidraw/element/flowchart.test.tsx b/packages/excalidraw/element/flowchart.test.tsx index 629fb1297..f832b20e4 100644 --- a/packages/excalidraw/element/flowchart.test.tsx +++ b/packages/excalidraw/element/flowchart.test.tsx @@ -36,7 +36,7 @@ describe("flow chart creation", () => { height: 100, }); - h.elements = [rectangle]; + API.setElements([rectangle]); API.setSelectedElements([rectangle]); }); @@ -166,7 +166,7 @@ describe("flow chart navigation", () => { height: 100, }); - h.elements = [rectangle]; + API.setElements([rectangle]); API.setSelectedElements([rectangle]); Keyboard.withModifierKeys({ ctrl: true }, () => { @@ -234,7 +234,7 @@ describe("flow chart navigation", () => { height: 100, }); - h.elements = [rectangle]; + API.setElements([rectangle]); API.setSelectedElements([rectangle]); Keyboard.withModifierKeys({ ctrl: true }, () => { @@ -325,7 +325,7 @@ describe("flow chart navigation", () => { height: 100, }); - h.elements = [rectangle]; + API.setElements([rectangle]); API.setSelectedElements([rectangle]); Keyboard.withModifierKeys({ ctrl: true }, () => { diff --git a/packages/excalidraw/tests/helpers/colorize.ts b/packages/excalidraw/tests/helpers/colorize.ts new file mode 100644 index 000000000..ca9c39d72 --- /dev/null +++ b/packages/excalidraw/tests/helpers/colorize.ts @@ -0,0 +1,2 @@ +export const yellow = (str: string) => `\u001b[33m${str}\u001b[0m`; +export const red = (str: string) => `\u001b[31m${str}\u001b[0m`; diff --git a/setupTests.ts b/setupTests.ts index 1e03e6c47..0b2ad48dc 100644 --- a/setupTests.ts +++ b/setupTests.ts @@ -5,6 +5,7 @@ import fs from "fs"; import { vi } from "vitest"; import polyfill from "./packages/excalidraw/polyfill"; import { testPolyfills } from "./packages/excalidraw/tests/helpers/polyfills"; +import { yellow } from "./packages/excalidraw/tests/helpers/colorize"; Object.assign(globalThis, testPolyfills); @@ -98,18 +99,20 @@ const element = document.createElement("div"); element.id = "root"; document.body.appendChild(element); -const logger = console.error.bind(console); +const _consoleError = console.error.bind(console); console.error = (...args) => { // the react's act() warning usually doesn't contain any useful stack trace // so we're catching the log and re-logging the message with the test name, // also stripping the actual component stack trace as it's not useful if (args[0]?.includes("act(")) { - logger( - `<<< WARNING: test "${ - expect.getState().currentTestName - }" does not wrap some state update in act() >>>`, + _consoleError( + yellow( + `<<< WARNING: test "${ + expect.getState().currentTestName + }" does not wrap some state update in act() >>>`, + ), ); } else { - logger(...args); + _consoleError(...args); } };