From 9deb68536c8694e86d63e19519d89da5efd0cc61 Mon Sep 17 00:00:00 2001 From: Jon Date: Fri, 21 Feb 2025 13:19:27 +0100 Subject: [PATCH 1/2] feat: make front tests fail if they show a warning or error on console --- test/vitest/setup-file.js | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/test/vitest/setup-file.js b/test/vitest/setup-file.js index 288f80beb..1d5165397 100644 --- a/test/vitest/setup-file.js +++ b/test/vitest/setup-file.js @@ -1 +1,16 @@ // This file will be run before each test file, don't delete or vitest will not work. +import { vi, beforeEach, afterEach } from 'vitest'; + +beforeEach(() => { + vi.spyOn(console, 'warn').mockImplementation((message) => { + throw new Error(`Test failed due to console.warn: ${message}`); + }); + + vi.spyOn(console, 'error').mockImplementation((message) => { + throw new Error(`Test failed due to console.error: ${message}`); + }); +}); + +afterEach(() => { + vi.restoreAllMocks(); +}); -- 2.40.1 From 992b64e6e3e72c227101d29d4588b0753b2405a1 Mon Sep 17 00:00:00 2001 From: Jon Date: Tue, 1 Apr 2025 08:57:45 +0200 Subject: [PATCH 2/2] refactor: throw error only in warnings --- test/vitest/setup-file.js | 4 ---- 1 file changed, 4 deletions(-) diff --git a/test/vitest/setup-file.js b/test/vitest/setup-file.js index 1d5165397..4e494552f 100644 --- a/test/vitest/setup-file.js +++ b/test/vitest/setup-file.js @@ -5,10 +5,6 @@ beforeEach(() => { vi.spyOn(console, 'warn').mockImplementation((message) => { throw new Error(`Test failed due to console.warn: ${message}`); }); - - vi.spyOn(console, 'error').mockImplementation((message) => { - throw new Error(`Test failed due to console.error: ${message}`); - }); }); afterEach(() => { -- 2.40.1