chore: refactor test setup (#19789)

Author: sapphi-red

playground/vitestSetup.ts

@@ -67,8 +67,6 @@ export const serverLogs: string[] = []
 export const browserLogs: string[] = []
 export const browserErrors: Error[] = []
 
-export let resolvedConfig: ResolvedConfig = undefined!
-
 export let page: Page = undefined!
 export let browser: Browser = undefined!
 export let viteTestUrl: string = ''
@@ -107,15 +105,6 @@ beforeAll(async (s) => {
   browser = await chromium.connect(wsEndpoint)
   page = await browser.newPage()
 
-  const globalConsole = global.console
-  const warn = globalConsole.warn
-  globalConsole.warn = (msg, ...args) => {
-    // suppress @vue/reactivity-transform warning
-    if (msg.includes('@vue/reactivity-transform')) return
-    if (msg.includes('Generated an empty chunk')) return
-    warn.call(globalConsole, msg, ...args)
-  }
-
   try {
     page.on('console', (msg) => {
       // ignore favicon request in headed browser
@@ -226,9 +215,6 @@ async function loadConfig(configEnv: ConfigEnv) {
         usePolling: true,
         interval: 100,
       },
-      fs: {
-        strict: !isBuild,
-      },
     },
     build: {
       // esbuild do not minify ES lib output since that would remove pure annotations and break tree-shaking
@@ -249,13 +235,14 @@ export async function startDefaultServe(): Promise<void> {
     process.env.VITE_INLINE = 'inline-serve'
     const config = await loadConfig({ command: 'serve', mode: 'development' })
     viteServer = server = await (await createServer(config)).listen()
-    viteTestUrl = server.resolvedUrls.local[0]
-    if (server.config.base === '/') {
-      viteTestUrl = viteTestUrl.replace(/\/$/, '')
-    }
+    viteTestUrl = stripTrailingSlashIfNeeded(
+      server.resolvedUrls.local[0],
+      server.config.base,
+    )
     await page.goto(viteTestUrl)
   } else {
     process.env.VITE_INLINE = 'inline-build'
+    let resolvedConfig: ResolvedConfig
     // determine build watch
     const resolvedPlugin: () => PluginOption = () => ({
       name: 'vite-plugin-watcher',
@@ -294,7 +281,10 @@ export async function startDefaultServe(): Promise<void> {
     const previewServer = await preview(previewConfig)
     // prevent preview change NODE_ENV
     process.env.NODE_ENV = _nodeEnv
-    viteTestUrl = previewServer.resolvedUrls.local[0]
+    viteTestUrl = stripTrailingSlashIfNeeded(
+      previewServer.resolvedUrls.local[0],
+      previewServer.config.base,
+    )
     await page.goto(viteTestUrl)
   }
 }
@@ -362,6 +352,13 @@ export function slash(p: string): string {
   return p.replace(/\\/g, '/')
 }
 
+function stripTrailingSlashIfNeeded(url: string, base: string): string {
+  if (base === '/') {
+    return url.replace(/\/$/, '')
+  }
+  return url
+}
+
 declare module 'vite' {
   export interface UserConfig {
     /**

vitest.config.e2e.ts

@@ -20,14 +20,6 @@ export default defineConfig({
       // Prevent Vitest from running the workspace packages in Vite's SSR runtime
       moduleDirectories: ['node_modules', 'packages'],
     },
-    onConsoleLog(log) {
-      if (
-        log.match(
-          /experimental|jit engine|emitted file|tailwind|The CJS build of Vite/i,
-        )
-      )
-        return false
-    },
   },
   esbuild: {
     target: 'node18',