fix: fix module graph UI on html reporter with headless browser mode (#9219)

hi-ogawa · web-flow · commit 60642b30482c · 2025-12-10T09:50:13.000+01:00
diff --git a/packages/ui/node/reporter.ts b/packages/ui/node/reporter.ts
@@ -101,7 +101,7 @@ export default class HTMLReporter implements Reporter {
       processAttachments(file)
       const projectName = file.projectName || ''
       const resolvedConfig = this.ctx.getProjectByName(projectName).config
-      const browser = resolvedConfig.browser.enabled && resolvedConfig.browser.ui
+      const browser = resolvedConfig.browser.enabled
       result.moduleGraph[projectName] ??= {}
       result.moduleGraph[projectName][file.filepath] = await getModuleGraph(
         this.ctx,
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/test/reporters/package.json b/test/reporters/package.json
@@ -6,6 +6,7 @@
     "test": "vitest"
   },
   "devDependencies": {
+    "@vitest/browser-playwright": "workspace:*",
     "@vitest/runner": "workspace:*",
     "flatted": "^3.3.3",
     "pkg-reporter": "./reportPkg/",
diff --git a/test/reporters/tests/html.test.ts b/test/reporters/tests/html.test.ts
@@ -1,10 +1,10 @@
 import fs from 'node:fs'
 import zlib from 'node:zlib'
+import { playwright } from '@vitest/browser-playwright'
 import { parse } from 'flatted'
 import { resolve } from 'pathe'
 import { describe, expect, it } from 'vitest'
-
-import { runVitest } from '../../test-utils'
+import { runInlineTests, runVitest } from '../../test-utils'
 
 describe('html reporter', async () => {
   const vitestRoot = resolve(import.meta.dirname, '../../..')
@@ -100,3 +100,33 @@ describe('html reporter', async () => {
     expect(indexHtml).toMatch('window.METADATA_PATH="html.meta.json.gz"')
   }, 120000)
 })
+
+it('browser mode headless', async () => {
+  const result = await runInlineTests({
+    'basic.test.ts': /* ts */`
+import { test } from "vitest";
+test('basic', () => {});
+`,
+  }, {
+    reporters: ['default', 'html'],
+  }, {}, {
+    test: {
+      browser: {
+        enabled: true,
+        provider: playwright(),
+        headless: true,
+        instances: [
+          { browser: 'chromium' as const },
+        ],
+      },
+    },
+  })
+  expect(result.errorTree()).toMatchInlineSnapshot(`
+    {
+      "basic.test.ts": {
+        "basic": "passed",
+      },
+    }
+  `)
+  expect(result.fs.statFile('html/index.html').isFile()).toBe(true)
+})
