Fix tests and remove logs from output (#196)

forge-42 · Feb 18, 2025 · aed054e · aed054e
1 parent 53069ac
commit aed054e
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 1 deletion.
diff --git a/src/server/utils.test.ts b/src/server/utils.test.ts
@@ -161,6 +161,7 @@ describe("analyzeCache Test suite", () => {
 	})
 
 	it("should log the cache headers set by the server with a custom max age and s-maxage", () => {
+		vi.clearAllMocks()
 		const loggerSpy = vi.spyOn(console, "log")
 		const config = {
 			logs: {
@@ -171,6 +172,7 @@ describe("analyzeCache Test suite", () => {
 		const headers = new Headers()
 		headers.set("Cache-Control", "max-age=3600, s-maxage=600, private")
 		analyzeCache("test", config, headers)
+
 		expect(loggerSpy).toHaveBeenCalledTimes(2)
 		expect(loggerSpy).nthCalledWith(
 			1,

diff --git a/test/console.ts b/test/console.ts
@@ -0,0 +1,2 @@
+vi.spyOn(console, "log").mockImplementation(() => undefined)
+vi.spyOn(console, "error").mockImplementation(() => undefined)
diff --git a/vitest.workspace.ts b/vitest.workspace.ts
@@ -11,7 +11,7 @@ export default defineWorkspace([
 			globals: true,
 			environment: "happy-dom",
 			exclude: ["**/node_modules/**", "**/dist/**", "**/docs/**", "**/public/**", "**/test-apps/**"],
-			setupFiles: ["../../test/setup.tsx"],
+			setupFiles: ["../../test/setup.tsx", "../../test/console.ts"],
 			css: true,
 			root: "./src/client",
 			name: "react-router-devtools/client",
@@ -35,6 +35,7 @@ export default defineWorkspace([
 		test: {
 			globals: true,
 			exclude: ["**/node_modules/**", "**/dist/**", "**/docs/**", "**/public/**", "**/test-apps/**"],
+			setupFiles: ["../../test/console.ts"],
 			environment: "happy-dom",
 			root: "./src/server",
 			name: "react-router-devtools/server",
@@ -58,6 +59,7 @@ export default defineWorkspace([
 		test: {
 			globals: true,
 			exclude: ["**/node_modules/**", "**/dist/**", "**/docs/**", "**/public/**", "**/test-apps/**"],
+			setupFiles: ["../../test/console.ts"],
 			environment: "happy-dom",
 			root: "./src/vite",
 			name: "react-router-devtools/vite",
@@ -81,6 +83,7 @@ export default defineWorkspace([
 		test: {
 			globals: true,
 			exclude: ["**/node_modules/**", "**/dist/**", "**/docs/**", "**/public/**", "**/test-apps/**"],
+			setupFiles: ["../../test/console.ts"],
 			environment: "node",
 			root: "./src/shared",
 			name: "react-router-devtools/shared",
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		vi.spyOn(console, "log").mockImplementation(() => undefined)
		vi.spyOn(console, "error").mockImplementation(() => undefined)