Merge pull request #2083 from umbraco/chore/fix-test-runner-log-filter

madsrasmussen · web-flow · commit 1e85b2241c71 · 2024-07-03T12:11:20.000+02:00
Chore: Fix test runner log filter
diff --git a/web-test-runner.config.mjs b/web-test-runner.config.mjs
@@ -12,6 +12,8 @@ if (!['dev', 'prod'].includes(mode)) {
 	throw new Error(`MODE must be "dev" or "prod", was "${mode}"`);
 }
 
+const silencedLogs = ['Lit is in dev mode.', 'Multiple versions of Lit loaded.'];
+
 /** @type {import('@web/dev-server').DevServerConfig} */
 export default {
 	rootDir: '.',
@@ -38,6 +40,14 @@ export default {
 		}),
 		esbuildPlugin({ ts: true, tsconfig: './tsconfig.json', target: 'auto', json: true }),
 	],
+	filterBrowserLogs(log) {
+		for (const arg of log.args) {
+			if (typeof arg === 'string' && silencedLogs.some((l) => arg.includes(l))) {
+				return false;
+			}
+		}
+		return true;
+	},
 	testRunnerHtml: (testFramework, devMode) =>
 		`<html lang="en-us">
 			<head>
@@ -59,19 +69,6 @@ export default {
 			</head>
       <body>
         <script type="module" src="${testFramework}"></script>
-				<script type="module">
-					/* Hack to disable Lit dev mode warnings */
-					const systemWarn = window.console.warn;
-					window.console.warn = (...args) => {
-						if (args[0].indexOf('Lit is in dev mode.') === 0) {
-							return;
-						}
-						if (args[0].indexOf('Multiple versions of Lit loaded.') === 0) {
-							return;
-						}
-						systemWarn(...args);
-					};
-				</script>
         <script type="module">
 					import 'element-internals-polyfill';
 					import '@umbraco-ui/uui';
