test: ✅ adjust tests

Author: stormslowly

tests/e2e/cases/chunk/recover-error/index.test.ts

@@ -2,7 +2,7 @@ import { test, expect } from "@/fixtures";
 import path from "path";
 
 test("should error with invalid syntax", async ({ page, fileAction, rspack }) => {
-	const codePath = path.resolve(rspack.outDir, "AppIndex.js");
+	const codePath = path.resolve(rspack.outDir, "src_AppIndex_jsx.js");
 	await expect(page.locator("button")).toHaveText("count is 0");
 	await page.click("button");
 	await expect(page.locator("button")).toHaveText("count is 1");

tests/e2e/cases/css/import-empty-css-file/index.test.ts

@@ -3,5 +3,5 @@ import { test, expect } from "@/fixtures";
 test("should not throw error for importing empty css files", async ({
 	page
 }) => {
-	expect(await page.textContent("#root")).toBe("ok");
+	await expect(page.locator("#root")).toHaveText("ok");
 });

tests/e2e/cases/file/context-delete-file/index.test.ts

@@ -7,11 +7,10 @@ test("delete file should work", async ({
 	rspack
 }) => {
 	// asset page status
-	const statusText = await page.textContent("#root");
-	expect(statusText).toBe("__PAGE_RENDER__");
+	await expect(page.locator("#root")).toHaveText("__PAGE_RENDER__");
 	// asset script content
 	const response = await request.get(
-		`http://localhost:${rspack.devServer.options.port}/main.js`
+		`http://localhost:${rspack.devServer.options.port}/src_index_js.js`
 	);
 	const bodyResponse = (await response.body()).toString();
 	expect(bodyResponse).toContain("this is mod1");
@@ -21,7 +20,7 @@ test("delete file should work", async ({
 	await expect(page.locator("#root")).toHaveText("__HMR_UPDATED__");
 	// asset new script content
 	const responseAfterDelete = await request.get(
-		`http://localhost:${rspack.devServer.options.port}/main.js`
+		`http://localhost:${rspack.devServer.options.port}/src_index_js.js`
 	);
 	const bodyResponseAfterDelete = (await responseAfterDelete.body()).toString();
 	expect(bodyResponseAfterDelete).not.toContain("this is mod1");

tests/e2e/cases/hooks/asset-emitted/index.test.ts

@@ -8,7 +8,7 @@ test("asset emitted hook should only emit modified assets", async ({
 	const assets = (rspack.compiler as any).assets;
 	// reset assets
 	assets.length = 0;
-	expect(await page.textContent("#root")).toBe("__ROOT_TEXT____FOO_VALUE__");
+	await expect(page.locator("#root")).toHaveText("__ROOT_TEXT____FOO_VALUE__");
 
 	// update js file
 	fileAction.updateFile("src/index.js", content => {

tests/e2e/cases/hooks/asset-emitted/rspack.config.js

@@ -22,5 +22,6 @@ module.exports = {
 	optimization: {
 		chunkIds: "named"
 	},
-	output: { clean: true }
+	output: { clean: true },
+	lazyCompilation: false,
 };

tests/e2e/cases/html/cross-origin-loading/rspack.config.js

@@ -16,5 +16,7 @@ module.exports = {
 	},
 	devServer: {
 		port: 3000
-	}
+	},
+	// for concise assert for assets
+	lazyCompilation: false
 };

tests/e2e/cases/make/remove-dynamic-entry-with-loader/index.test.ts

@@ -2,8 +2,8 @@ import { expect, test } from "@/fixtures";
 
 test("should compile", async ({ page, fileAction, rspack }) => {
 	// rspack.compiler.__sharedObj is injected by plugin in rspack.config.js
-	await expect(page.getByText("index1")).toBeVisible();
-	await expect(page.getByText("index2")).toBeVisible();
+	await expect(page.locator("#index1")).toHaveText("index1");
+	await expect(page.locator("#index2")).toHaveText("index2");
 
 	rspack.compiler.__sharedObj.useFullEntry = false;
 	fileAction.updateFile("src/index2.js", content =>
@@ -23,6 +23,7 @@ test("should compile", async ({ page, fileAction, rspack }) => {
 		await page.reload();
 		expect(await page.locator("#index1").innerText()).toBe("index1 updated");
 	}).toPass();
+
 	await expect(page.locator("#index2")).toHaveCount(0);
 	await expect(page.locator("#webpack-dev-server-client-overlay")).toHaveCount(
 		0

tests/e2e/cases/make/remove-dynamic-entry-with-loader/rspack.config.js

@@ -34,5 +34,6 @@ module.exports = {
 	],
 	devServer: {
 		hot: true
-	}
+	},
+	lazyCompilation: false,
 };

tests/e2e/cases/make/remove-dynamic-entry/index.test.ts

@@ -1,8 +1,8 @@
 import { expect, test } from "@/fixtures";
 
 test("should compile", async ({ page, fileAction, rspack }) => {
-	await expect(page.getByText("index1")).toBeVisible();
-	await expect(page.getByText("index2")).toBeVisible();
+	await expect(page.locator("#index1")).toBeVisible();
+	await expect(page.locator("#index2")).toBeVisible();
 
 	fileAction.deleteFile("src/index2.js");
 	fileAction.updateFile("src/index1.js", content => content.replace("div.innerText = \"index1\";", "div.innerText = \"index1 updated\";"));

tests/e2e/cases/make/remove-dynamic-entry/rspack.config.js

@@ -22,4 +22,7 @@ module.exports = {
 	devServer: {
 		hot: true
 	},
+  lazyCompilation: {
+    entries: false
+  }
 };