update web test-runner and remove temp summaryReporter workaournd (#3320)

MichaelWest22 · web-flow · commit 083dbcdd6f3b · 2025-05-22T07:17:16.000+02:00
* update web test-runner and remove temp summaryReporter workaournd

* Update error reporting to show in context of the test file and as a summary at the bottom
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -80,7 +80,7 @@
   "devDependencies": {
     "@types/node": "20.0.0",
     "@types/parse5": "^7.0.0",
-    "@web/test-runner": "^0.20.1",
+    "@web/test-runner": "^0.20.2",
     "@web/test-runner-playwright": "^0.11.0",
     "chai": "^4.5.0",
     "chai-dom": "^1.12.1",
diff --git a/web-test-runner.config.mjs b/web-test-runner.config.mjs
@@ -1,16 +1,8 @@
 import {
   summaryReporter,
-  defaultReporter,
-  dotReporter
+  defaultReporter
 } from '@web/test-runner'
 
-// There is a bug in the summaryReporter made with this PR https://github.com/modernweb-dev/web/pull/2126
-// It captures the buffered logger into cachedLogger for reuse later for the test run finished error reporting
-// But the buffered logger in finished its buffer flush before this test error reporting so these logs go nowhere
-// to resolve this for now reusing dotReporter which reports errors well but disabled its . test reporting
-const errorReporter = dotReporter()
-delete errorReporter.reportTestFileResults
-
 const config = {
   testRunnerHtml: (testFramework) => `
   <html lang="en">
@@ -69,7 +61,7 @@ const config = {
     'test/attributes/**/*.js',
     'test/core/**/*.js'
   ],
-  reporters: [summaryReporter({ flatten: false }), errorReporter, defaultReporter({ reportTestProgress: true, reportTestResults: false })]
+  reporters: [summaryReporter({ flatten: false, reportTestLogs: false, reportTestErrors: true }), defaultReporter({ reportTestProgress: true, reportTestResults: true })]
 }
 
 export default config
