fix: setup tests

Author: underfin

package.json

@@ -23,6 +23,7 @@
     "typecheck": "tsc -p scripts && tsc -p playground && tsc -p packages/plugin-react",
     "test": "pnpm run test-serve && pnpm run test-build && pnpm --filter ./packages/plugin-react-swc run test",
     "test-serve": "vitest run -c playground/vitest.config.e2e.ts",
+    "test-full-bundle-mode-serve": "VITE_TEST_FULL_BUNDLE_MODE=1 vitest run -c playground/vitest.config.e2e.ts",
     "test-build": "VITE_TEST_BUILD=1 vitest run -c playground/vitest.config.e2e.ts",
     "debug-serve": "VITE_DEBUG_SERVE=1 vitest run -c playground/vitest.config.e2e.ts",
     "debug-build": "VITE_TEST_BUILD=1 VITE_PRESERVE_BUILD_ARTIFACTS=1 vitest run -c playground/vitest.config.e2e.ts",

packages/plugin-react-oxc/src/index.ts

@@ -102,11 +102,12 @@ export default function viteReact(opts: Options = {}): PluginOption[] {
   }
 
   let skipFastRefresh = false
-
+  let base: string | undefined
   const viteRefreshWrapper: Plugin = {
     name: 'vite:react-oxc:refresh-wrapper',
     apply: 'serve',
     configResolved(config) {
+      base = config.base
       skipFastRefresh = config.isProduction || config.server.hmr === false
     },
     transform: {
@@ -135,15 +136,22 @@ export default function viteReact(opts: Options = {}): PluginOption[] {
         return { code: newCode, map: null }
       },
     },
-    transformIndexHtml(_, config) {
-      if (!skipFastRefresh)
-        return [
-          {
-            tag: 'script',
-            attrs: { type: 'module' },
-            children: getPreambleCode(config.server!.config.base),
-          },
-        ]
+    transformIndexHtml: {
+      handler() {
+        if (!skipFastRefresh)
+          return [
+            {
+              tag: 'script',
+              attrs: { type: 'module' },
+              // !!! Rolldown vite full bunlde module break changes, config.server is invalid
+              // children: getPreambleCode(config.server!.config.base),
+              children: getPreambleCode(base!),
+            },
+          ]
+      },
+      // Rolldown vite full bunlde module break changes.
+      // Changed it to make sure the inject module could be bundled
+      order: 'pre',
     },
   }
 

packages/plugin-react-swc/src/index.ts

@@ -91,6 +91,7 @@ const react = (_options?: Options): PluginOption[] => {
       _options?.useAtYourOwnRisk_mutateSwcOptions,
   }
 
+  let base: string | undefined
   return [
     {
       name: 'vite:react-swc:resolve-runtime',
@@ -118,6 +119,7 @@ const react = (_options?: Options): PluginOption[] => {
         },
       }),
       configResolved(config) {
+        base = config.base
         if (config.server.hmr === false) hmrDisabled = true
         const mdxIndex = config.plugins.findIndex(
           (p) => p.name === '@mdx-js/rollup',
@@ -132,13 +134,22 @@ const react = (_options?: Options): PluginOption[] => {
           )
         }
       },
-      transformIndexHtml: (_, config) => [
-        {
-          tag: 'script',
-          attrs: { type: 'module' },
-          children: getPreambleCode(config.server!.config.base),
+      transformIndexHtml: {
+        handler() {
+          return [
+            {
+              tag: 'script',
+              attrs: { type: 'module' },
+              // !!! Rolldown vite full bunlde module break changes, config.server is invalid
+              // children: getPreambleCode(config.server!.config.base),
+              children: getPreambleCode(base!),
+            },
+          ]
         },
-      ],
+        // Rolldown vite full bunlde module break changes.
+        // Changed it to make sure the inject module could be bundled
+        order: 'pre',
+      },
       async transform(code, _id, transformOptions) {
         const id = _id.split('?')[0]
         const refresh = !transformOptions?.ssr && !hmrDisabled

playground/vitest.config.e2e.ts

@@ -11,7 +11,20 @@ export default defineConfig({
   },
   test: {
     pool: 'forks',
-    include: ['./playground/**/*.spec.[tj]s'],
+    include: process.env.VITE_TEST_FULL_BUNDLE_MODE
+      ? [
+          './playground/class-components/**/*.spec.[tj]s',
+          './playground/compiler/**/*.spec.[tj]s',
+          './playground/compiler-react-18/**/*.spec.[tj]s',
+          './playground/mdx/**/*.spec.[tj]s',
+          // './playground/react/**/*.spec.[tj]s',
+          // './playground/react-classic/**/*.spec.[tj]s',
+          './playground/react-emotion/**/*.spec.[tj]s',
+          './playground/react-env/**/*.spec.[tj]s',
+          './playground/react-sourcemap/**/*.spec.[tj]s',
+          // './playground/ssr-react/**/*.spec.[tj]s',
+        ]
+      : ['./playground/**/*.spec.[tj]s'],
     setupFiles: ['./playground/vitestSetup.ts'],
     globalSetup: ['./playground/vitestGlobalSetup.ts'],
     testTimeout: timeout,

playground/vitestSetup.ts

@@ -214,21 +214,39 @@ async function loadConfig(configEnv: ConfigEnv) {
       // tests are flaky when `emptyOutDir` is `true`
       emptyOutDir: false,
     },
+    experimental: {
+      fullBundleMode: !!process.env.VITE_TEST_FULL_BUNDLE_MODE,
+    },
     customLogger: createInMemoryLogger(serverLogs),
   }
   return mergeConfig(options, config || {})
 }
 
 export async function startDefaultServe(): Promise<void> {
-  const { build, createBuilder, createServer, mergeConfig, preview } =
-    await importVite()
+  const {
+    build,
+    createBuilder,
+    createServer,
+    mergeConfig,
+    preview,
+    createServerWithResolvedConfig,
+  } = await importVite()
 
   setupConsoleWarnCollector(serverLogs)
 
   if (!isBuild) {
     process.env.VITE_INLINE = 'inline-serve'
     const config = await loadConfig({ command: 'serve', mode: 'development' })
-    viteServer = server = await (await createServer(config)).listen()
+
+    if (process.env.VITE_TEST_FULL_BUNDLE_MODE) {
+      const builder = await createBuilder(config, null, 'serve')
+      viteServer = server = await createServerWithResolvedConfig(builder.config)
+      await server.listen()
+      await builder.buildApp(server)
+    } else {
+      viteServer = server = await (await createServer(config)).listen()
+    }
+
     viteTestUrl = stripTrailingSlashIfNeeded(
       server.resolvedUrls.local[0],
       server.config.base,