fix: modify lazy compilation tests (#11349)

JSerFeng · web-flow · commit 396e59ac573a · 2025-08-13T12:48:37.000+08:00
* fix: modify lazy-compilation tests

* fix: should use specific runtime to get target url
diff --git a/packages/rspack-test-tools/src/plugin/lazy-compilation-test-plugin.ts b/packages/rspack-test-tools/src/plugin/lazy-compilation-test-plugin.ts
@@ -22,12 +22,12 @@ export class LazyCompilationTestPlugin {
 							: `${protocol}://${addr.address}:${addr.port}`;
 				if (compiler instanceof MultiCompiler) {
 					for (const c of compiler.compilers) {
-						if (c.options.experiments.lazyCompilation) {
-							c.options.experiments.lazyCompilation.serverUrl = urlBase;
+						if (c.options.lazyCompilation) {
+							c.options.lazyCompilation.serverUrl = urlBase;
 						}
 					}
-				} else if (compiler.options.experiments.lazyCompilation) {
-					compiler.options.experiments.lazyCompilation.serverUrl = urlBase;
+				} else if (compiler.options.lazyCompilation) {
+					compiler.options.lazyCompilation.serverUrl = urlBase;
 				}
 				middleware = experiments.lazyCompilationMiddleware(compiler);
 
diff --git a/packages/rspack-test-tools/src/processor/hot.ts b/packages/rspack-test-tools/src/processor/hot.ts
@@ -175,7 +175,7 @@ export class HotProcessor<T extends ECompilerType> extends BasicProcessor<T> {
 			};
 		}
 
-		if (options.experiments?.lazyCompilation) {
+		if ((options as TCompilerOptions<ECompilerType.Rspack>).lazyCompilation) {
 			(options as TCompilerOptions<ECompilerType.Rspack>).plugins!.push(
 				new LazyCompilationTestPlugin()
 			);
diff --git a/packages/rspack-test-tools/tests/hotCases/lazy-compilation/context/webpack.config.js b/packages/rspack-test-tools/tests/hotCases/lazy-compilation/context/webpack.config.js
@@ -2,11 +2,9 @@
 
 /** @type {import("@rspack/core").Configuration} */
 module.exports = {
-	experiments: {
-		lazyCompilation: {
-			cacheable: false,
-			entries: false,
-			imports: true
-		}
+	lazyCompilation: {
+		cacheable: false,
+		entries: false,
+		imports: true
 	}
 };
diff --git a/packages/rspack-test-tools/tests/hotCases/lazy-compilation/module-test/webpack.config.js b/packages/rspack-test-tools/tests/hotCases/lazy-compilation/module-test/webpack.config.js
@@ -2,11 +2,9 @@
 
 /** @type {import("@rspack/core").Configuration} */
 module.exports = {
-	experiments: {
-		lazyCompilation: {
-			entries: false,
-			cacheable: false,
-			test: /moduleA/
-		}
+	lazyCompilation: {
+		entries: false,
+		cacheable: false,
+		test: /moduleA/
 	}
 };
diff --git a/packages/rspack-test-tools/tests/hotCases/lazy-compilation/only-entries/webpack.config.js b/packages/rspack-test-tools/tests/hotCases/lazy-compilation/only-entries/webpack.config.js
@@ -2,10 +2,8 @@
 
 /** @type {import("@rspack/core").Configuration} */
 module.exports = {
-	experiments: {
-		lazyCompilation: {
-			entries: false,
-			imports: false
-		}
+	lazyCompilation: {
+		entries: false,
+		imports: false
 	}
 };
diff --git a/packages/rspack-test-tools/tests/hotCases/lazy-compilation/simple/webpack.config.js b/packages/rspack-test-tools/tests/hotCases/lazy-compilation/simple/webpack.config.js
@@ -2,10 +2,8 @@
 
 /** @type {import("@rspack/core").Configuration} */
 module.exports = {
-	experiments: {
-		lazyCompilation: {
-			entries: false,
-			cacheable: false
-		}
+	lazyCompilation: {
+		entries: false,
+		cacheable: false
 	}
 };
diff --git a/packages/rspack-test-tools/tests/hotCases/lazy-compilation/unrelated/webpack.config.js b/packages/rspack-test-tools/tests/hotCases/lazy-compilation/unrelated/webpack.config.js
@@ -2,9 +2,7 @@
 
 /** @type {import("@rspack/core").Configuration} */
 module.exports = {
-	experiments: {
-		lazyCompilation: {
-			entries: false
-		}
+	lazyCompilation: {
+		entries: false
 	}
 };
diff --git a/tests/e2e/cases/lazy-compilation/css-update/rspack.config.js b/tests/e2e/cases/lazy-compilation/css-update/rspack.config.js
@@ -16,8 +16,8 @@ module.exports = {
 	plugins: [new rspack.HtmlRspackPlugin(), new rspack.CssExtractRspackPlugin()],
 	experiments: {
 		css: false,
-		lazyCompilation: true
 	},
+	lazyCompilation: true,
 	optimization: {
 		splitChunks: {
 			minSize: 0,
diff --git a/tests/e2e/cases/lazy-compilation/custom-prefix/rspack.config.js b/tests/e2e/cases/lazy-compilation/custom-prefix/rspack.config.js
@@ -9,14 +9,12 @@ module.exports = {
   stats: "none",
   mode: "development",
   plugins: [new rspack.HtmlRspackPlugin()],
-  experiments: {
-    lazyCompilation: {
-      entries: true,
-      imports: true,
-      // Set custom prefix for lazy compilation
-      prefix: "/custom-lazy-endpoint-"
-    }
-  },
+	lazyCompilation: {
+		entries: true,
+		imports: true,
+		// Set custom prefix for lazy compilation
+		prefix: "/custom-lazy-endpoint-"
+	},
   devServer: {
     hot: true
   }
diff --git a/tests/e2e/cases/lazy-compilation/default-prefix/index.test.ts b/tests/e2e/cases/lazy-compilation/default-prefix/index.test.ts
@@ -1,28 +1,30 @@
 import { expect, test } from "@/fixtures";
 
 test("should use default prefix for lazy compilation", async ({ page }) => {
-  // Wait for a request with default prefix
-  const responsePromise = page.waitForResponse(
-    response =>
-      response.url().includes("/lazy-compilation-using-") &&
-      response.request().method() === "GET",
-    { timeout: 5000 }
-  );
+	// Wait for a request with default prefix
+	const responsePromise = page.waitForResponse(
+		response => {
+			console.log(`server url: ${response.url()}`)
+			return response.url().includes("/lazy-compilation-using-") &&
+				response.request().method() === "GET"
+		},
+		{ timeout: 5000 }
+	);
 
-  // Click the button that triggers dynamic import
-  await page.getByText("Click me").click();
+	// Click the button that triggers dynamic import
+	await page.getByText("Click me").click();
 
-  // Wait for response with default prefix
-  const response = await responsePromise;
-  expect(response.status()).toBe(200);
+	// Wait for response with default prefix
+	const response = await responsePromise;
+	expect(response.status()).toBe(200);
 
-  // Wait for the component to appear with a more reliable wait
-  await page.waitForSelector('div:has-text("Component")', { timeout: 5000 });
+	// Wait for the component to appear with a more reliable wait
+	await page.waitForSelector('div:has-text("Component")', { timeout: 5000 });
 
-  // Check that the component was loaded and displayed
-  const component_count = await page.getByText("Component").count();
-  expect(component_count).toBe(1);
+	// Check that the component was loaded and displayed
+	const component_count = await page.getByText("Component").count();
+	expect(component_count).toBe(1);
 
-  // Verify that the request was made using the default prefix
-  expect(response.url()).toContain("/lazy-compilation-using-");
+	// Verify that the request was made using the default prefix
+	expect(response.url()).toContain("/lazy-compilation-using-");
 });
diff --git a/tests/e2e/cases/lazy-compilation/default-prefix/rspack.config.js b/tests/e2e/cases/lazy-compilation/default-prefix/rspack.config.js
@@ -9,14 +9,16 @@ module.exports = {
   stats: "none",
   mode: "development",
   plugins: [new rspack.HtmlRspackPlugin()],
-  experiments: {
-    lazyCompilation: {
-      entries: true,
-      imports: true
-      // Using default prefix (not specifying prefix option)
-    }
-  },
+	lazyCompilation: {
+		entries: true,
+		imports: true
+		// Using default prefix (not specifying prefix option)
+	},
+	devtool: false,
   devServer: {
-    hot: true
+    hot: true,
+		devMiddleware: {
+			writeToDisk: true
+		}
   }
 };
diff --git a/tests/e2e/cases/lazy-compilation/ignore-entry/rspack.config.js b/tests/e2e/cases/lazy-compilation/ignore-entry/rspack.config.js
@@ -24,8 +24,8 @@ module.exports = {
 	plugins: [new rspack.HtmlRspackPlugin(), new rspack.CssExtractRspackPlugin()],
 	experiments: {
 		css: false,
-		lazyCompilation: true
 	},
+	lazyCompilation: true,
 	devServer: {
 		hot: true
 	}
diff --git a/tests/e2e/cases/lazy-compilation/persistent-cache/rspack.config.js b/tests/e2e/cases/lazy-compilation/persistent-cache/rspack.config.js
@@ -10,8 +10,8 @@ module.exports = {
 	mode: "production",
 	plugins: [new rspack.HtmlRspackPlugin()],
 	cache: true,
+	lazyCompilation: true,
 	experiments: {
-		lazyCompilation: true,
 		cache: {
 			type: "persistent"
 		}
diff --git a/tests/e2e/fixtures/rspack.ts b/tests/e2e/fixtures/rspack.ts
@@ -61,7 +61,7 @@ class Rspack {
 			}
 		});
 		const DevServerConstructor = RspackDevServer;
-		if (compiler.options.experiments.lazyCompilation) {
+		if (compiler.options.lazyCompilation) {
 			const middleware = experiments.lazyCompilationMiddleware(compiler);
 			compiler.options.devServer ??= {};
 			const setupMiddlewares = compiler.options.devServer.setupMiddlewares;
@@ -102,47 +102,47 @@ export const rspackFixtures = (): RspackFixtures => {
 	const rspackFixture = (
 		incremental: boolean
 	): RspackFixtures["rspack"] | RspackFixtures["rspackIncremental"] => [
-		async ({ page, pathInfo, defaultRspackConfig }, use, { workerIndex }) => {
-			const { tempProjectDir } = pathInfo;
-			const port = (incremental ? 8200 : 8000) + workerIndex;
-			const rspack = new Rspack(tempProjectDir, config => {
-				// rewrite port
-				if (!config.devServer) {
-					config.devServer = {};
-				}
-				config.devServer.port = port;
+			async ({ page, pathInfo, defaultRspackConfig }, use, { workerIndex }) => {
+				const { tempProjectDir } = pathInfo;
+				const port = (incremental ? 8200 : 8000) + workerIndex;
+				const rspack = new Rspack(tempProjectDir, config => {
+					// rewrite port
+					if (!config.devServer) {
+						config.devServer = {};
+					}
+					config.devServer.port = port;
 
-				// set default context
-				if (!config.context) {
-					config.context = tempProjectDir;
-				}
+					// set default context
+					if (!config.context) {
+						config.context = tempProjectDir;
+					}
 
-				if (incremental) {
-					config.experiments ??= {};
-					config.experiments.incremental = true;
-					const cache = config.experiments.cache;
-					if (typeof cache === "object" && cache.type === "persistent") {
-						cache.storage = {
-							type: "filesystem",
-							...cache.storage,
-							//rewrite directory
-							directory: "node_modules/.cache/incremental"
-						};
+					if (incremental) {
+						config.experiments ??= {};
+						config.experiments.incremental = true;
+						const cache = config.experiments.cache;
+						if (typeof cache === "object" && cache.type === "persistent") {
+							cache.storage = {
+								type: "filesystem",
+								...cache.storage,
+								//rewrite directory
+								directory: "node_modules/.cache/incremental"
+							};
+						}
 					}
-				}
 
-				return defaultRspackConfig.handleConfig(config);
-			});
-			await rspack.start();
+					return defaultRspackConfig.handleConfig(config);
+				});
+				await rspack.start();
 
-			await page.goto(`http://localhost:${port}`);
+				await page.goto(`http://localhost:${port}`);
 
-			await use(rspack);
+				await use(rspack);
 
-			await rspack.stop();
-		},
-		{ auto: true }
-	];
+				await rspack.stop();
+			},
+			{ auto: true }
+		];
 	return {
 		defaultRspackConfig: [{ handleConfig: c => c }, { option: true }],
 		rspack: rspackFixture(false),
diff --git a/tests/webpack-examples/lazy-compilation/README.md b/tests/webpack-examples/lazy-compilation/README.md
@@ -53,9 +53,7 @@ module.exports = {
 		type: "filesystem",
 		idleTimeout: 5000
 	},
-	experiments: {
-		lazyCompilation: true
-	},
+	lazyCompilation: true,
 	devServer: {
 		hot: true,
 		devMiddleware: {
diff --git a/tests/webpack-examples/lazy-compilation/webpack.config.js b/tests/webpack-examples/lazy-compilation/webpack.config.js
@@ -9,9 +9,7 @@ module.exports = {
 		type: "filesystem",
 		idleTimeout: 5000
 	},
-	experiments: {
-		lazyCompilation: true
-	},
+	lazyCompilation: true,
 	devServer: {
 		hot: true,
 		devMiddleware: {
diff --git a/tests/webpack-test/hotCases/lazy-compilation/context/webpack.config.js b/tests/webpack-test/hotCases/lazy-compilation/context/webpack.config.js
@@ -2,15 +2,13 @@
 
 /** @type {import("@rspack/core").Configuration} */
 module.exports = {
-	experiments: {
-		lazyCompilation: {
-			cacheable: false,
-			entries: false,
-			imports: true,
-			backend: {
-				listen: {
-					host: "127.0.0.1"
-				}
+	lazyCompilation: {
+		cacheable: false,
+		entries: false,
+		imports: true,
+		backend: {
+			listen: {
+				host: "127.0.0.1"
 			}
 		}
 	}
diff --git a/tests/webpack-test/hotCases/lazy-compilation/https/webpack.config.js b/tests/webpack-test/hotCases/lazy-compilation/https/webpack.config.js
@@ -5,15 +5,13 @@ const path = require("path");
 
 /** @type {import("@rspack/core").Configuration} */
 module.exports = {
-	experiments: {
-		lazyCompilation: {
-			cacheable: false,
-			entries: false,
-			backend: {
-				server: {
-					key: fs.readFileSync(path.join(__dirname, "key.pem")),
-					cert: fs.readFileSync(path.join(__dirname, "cert.pem"))
-				}
+	lazyCompilation: {
+		cacheable: false,
+		entries: false,
+		backend: {
+			server: {
+				key: fs.readFileSync(path.join(__dirname, "key.pem")),
+				cert: fs.readFileSync(path.join(__dirname, "cert.pem"))
 			}
 		}
 	}
diff --git a/tests/webpack-test/hotCases/lazy-compilation/module-test/webpack.config.js b/tests/webpack-test/hotCases/lazy-compilation/module-test/webpack.config.js
@@ -2,11 +2,9 @@
 
 /** @type {import("@rspack/core").Configuration} */
 module.exports = {
-	experiments: {
-		lazyCompilation: {
-			entries: false,
-			cacheable: false,
-			test: /moduleA/
-		}
+	lazyCompilation: {
+		entries: false,
+		cacheable: false,
+		test: /moduleA/
 	}
 };
diff --git a/tests/webpack-test/hotCases/lazy-compilation/only-entries/webpack.config.js b/tests/webpack-test/hotCases/lazy-compilation/only-entries/webpack.config.js
diff --git a/tests/webpack-test/hotCases/lazy-compilation/simple/webpack.config.js b/tests/webpack-test/hotCases/lazy-compilation/simple/webpack.config.js
diff --git a/tests/webpack-test/hotCases/lazy-compilation/unrelated/webpack.config.js b/tests/webpack-test/hotCases/lazy-compilation/unrelated/webpack.config.js

Original file line number	Diff line number	Diff line change
`@@ -22,12 +22,12 @@ export class LazyCompilationTestPlugin {`
`22`	`22`	: `${protocol}://${addr.address}:${addr.port}`;
`23`	`23`	`if (compiler instanceof MultiCompiler) {`
`24`	`24`	`for (const c of compiler.compilers) {`
`25`		`- if (c.options.experiments.lazyCompilation) {`
`26`		`- c.options.experiments.lazyCompilation.serverUrl = urlBase;`
	`25`	`+ if (c.options.lazyCompilation) {`
	`26`	`+ c.options.lazyCompilation.serverUrl = urlBase;`
`27`	`27`	`}`
`28`	`28`	`}`
`29`		`- } else if (compiler.options.experiments.lazyCompilation) {`
`30`		`- compiler.options.experiments.lazyCompilation.serverUrl = urlBase;`
	`29`	`+ } else if (compiler.options.lazyCompilation) {`
	`30`	`+ compiler.options.lazyCompilation.serverUrl = urlBase;`
`31`	`31`	`}`
`32`	`32`	`middleware = experiments.lazyCompilationMiddleware(compiler);`
`33`	`33`
Original file line number	Diff line number	Diff line change
`@@ -175,7 +175,7 @@ export class HotProcessor<T extends ECompilerType> extends BasicProcessor<T> {`
`175`	`175`	`};`
`176`	`176`	`}`
`177`	`177`
`178`		`- if (options.experiments?.lazyCompilation) {`
	`178`	`+ if ((options as TCompilerOptions<ECompilerType.Rspack>).lazyCompilation) {`
`179`	`179`	`(options as TCompilerOptions<ECompilerType.Rspack>).plugins!.push(`
`180`	`180`	`new LazyCompilationTestPlugin()`
`181`	`181`	`);`