test(webpack-test): add back two test case

.github/actions/pnpm-cache/action.yml

@@ -28,8 +28,7 @@ runs:
       if: ${{ inputs.node-version != '16' }}
       shell: bash
       run: |
-        corepack enable
-
+        npm i -g --force corepack && which -a corepack && corepack enable
     # https://pnpm.io/continuous-integration#github-actions
     # Uses `packageManagement` field from package.json
     - name: Install pnpm

tests/webpack-test/Compiler.test.js

@@ -12,16 +12,15 @@ const { normalizeFilteredTestName, FilteredStatus } = require("./lib/util/filter
 
 describe("Compiler", () => {
 	jest.setTimeout(20000);
-	function compile(entry, options, callback) {
+	function compile(entry, options, callback, done = () => {}) {
 		const noOutputPath = !options.output || !options.output.path;
 		const webpack = require("@rspack/core");
 		options = webpack.config.getNormalizedWebpackOptions(options);
 		if (!options.mode) options.mode = "production";
 		options.entry = entry;
 		options.context = path.join(__dirname, "fixtures");
 		if (noOutputPath) options.output.path = "/";
-		// CHANGE: The pathinfo is currently not supported in rspack
-		// options.output.pathinfo = true;
+		options.output.pathinfo = true;
 		options.optimization = {
 			minimize: false
 		};
@@ -81,7 +80,11 @@ describe("Compiler", () => {
 			stats.logs = logs;
 			c.close(err => {
 				if (err) return callback(err);
-				callback(stats, files, compilation);
+				try {
+					callback(stats, files, compilation);
+				} catch (e) {
+					done(e);
+				}
 			});
 		});
 	}
@@ -117,6 +120,7 @@ describe("Compiler", () => {
 	});
 
 	// CHANGE: skip due to Rspack defaults to numerical module ids, unlike webpack's string-based ids
+	// TODO: ModuleInfoHeaderPlugin
 	it.skip(normalizeFilteredTestName(FilteredStatus.TODO, "should compile a single file"), done => {
 		compile("./c", {}, (stats, files) => {
 			expect(Object.keys(files)).toEqual(["/main.js"]);
@@ -132,56 +136,74 @@ describe("Compiler", () => {
 			expect(bundle).not.toMatch("jsonp");
 			expect(bundle).not.toMatch("fixtures");
 			done();
-		});
+			},
+			done
+		);
 	});
 
 	// CHANGE: skip with custom test name for tracking alignment status
+	// TODO: ModuleInfoHeaderPlugin
 	it.skip(normalizeFilteredTestName(FilteredStatus.TODO, "should compile a complex file"), done => {
-		compile("./main1", {}, (stats, files) => {
-			expect(Object.keys(files)).toEqual(["/main.js"]);
-			const bundle = files["/main.js"];
-			expect(bundle).toMatch("function __webpack_require__(");
-			expect(bundle).toMatch("__webpack_require__(/*! ./a */");
-			expect(bundle).toMatch("./main1.js");
-			expect(bundle).toMatch("./a.js");
-			expect(bundle).toMatch("./b.js");
-			expect(bundle).toMatch("./node_modules/m1/a.js");
-			expect(bundle).toMatch("This is a");
-			expect(bundle).toMatch("This is b");
-			expect(bundle).toMatch("This is m1/a");
-			expect(bundle).not.toMatch("4: function(");
-			expect(bundle).not.toMatch("window");
-			expect(bundle).not.toMatch("jsonp");
-			expect(bundle).not.toMatch("fixtures");
-			done();
-		});
+		compile(
+			"./main1",
+			{},
+			(stats, files) => {
+				expect(Object.keys(files)).toEqual(["/main.js"]);
+				const bundle = files["/main.js"];
+				expect(bundle).toMatch("function __webpack_require__(");
+				expect(bundle).toMatch("__webpack_require__(/*! ./a */");
+				expect(bundle).toMatch("./main1.js");
+				expect(bundle).toMatch("./a.js");
+				expect(bundle).toMatch("./b.js");
+				expect(bundle).toMatch("./node_modules/m1/a.js");
+				expect(bundle).toMatch("This is a");
+				expect(bundle).toMatch("This is b");
+				expect(bundle).toMatch("This is m1/a");
+				expect(bundle).not.toMatch("4: function(");
+				expect(bundle).not.toMatch("window");
+				expect(bundle).not.toMatch("jsonp");
+				expect(bundle).not.toMatch("fixtures");
+				done();
+			},
+			done
+		);
 	});
 
 	// CHANGE: skip with custom test name for tracking alignment status
-	it.skip(normalizeFilteredTestName(FilteredStatus.TODO, "should compile a file with transitive dependencies"), done => {
-		compile("./abc", {}, (stats, files) => {
-			expect(Object.keys(files)).toEqual(["/main.js"]);
-			const bundle = files["/main.js"];
-			expect(bundle).toMatch("function __webpack_require__(");
-			expect(bundle).toMatch("__webpack_require__(/*! ./a */");
-			expect(bundle).toMatch("__webpack_require__(/*! ./b */");
-			expect(bundle).toMatch("__webpack_require__(/*! ./c */");
-			expect(bundle).toMatch("./abc.js");
-			expect(bundle).toMatch("./a.js");
-			expect(bundle).toMatch("./b.js");
-			expect(bundle).toMatch("./c.js");
-			expect(bundle).toMatch("This is a");
-			expect(bundle).toMatch("This is b");
-			expect(bundle).toMatch("This is c");
-			expect(bundle).not.toMatch("4: function(");
-			expect(bundle).not.toMatch("window");
-			expect(bundle).not.toMatch("jsonp");
-			expect(bundle).not.toMatch("fixtures");
-			done();
-		});
+	// TODO: ModuleInfoHeaderPlugin.js
+	it.skip(normalizeFilteredTestName(
+		FilteredStatus.TODO,
+		"should compile a file with transitive dependencies"
+	), done => {
+		compile(
+			"./abc",
+			{},
+			(stats, files) => {
+				expect(Object.keys(files)).toEqual(["/main.js"]);
+				const bundle = files["/main.js"];
+				expect(bundle).toMatch("function __webpack_require__(");
+				expect(bundle).toMatch("__webpack_require__(/*! ./a */");
+				expect(bundle).toMatch("__webpack_require__(/*! ./b */");
+				expect(bundle).toMatch("__webpack_require__(/*! ./c */");
+				expect(bundle).toMatch("./abc.js");
+				expect(bundle).toMatch("./a.js");
+				expect(bundle).toMatch("./b.js");
+				expect(bundle).toMatch("./c.js");
+				expect(bundle).toMatch("This is a");
+				expect(bundle).toMatch("This is b");
+				expect(bundle).toMatch("This is c");
+				expect(bundle).not.toMatch("4: function(");
+				expect(bundle).not.toMatch("window");
+				expect(bundle).not.toMatch("jsonp");
+				expect(bundle).not.toMatch("fixtures");
+				done();
+			},
+			done
+		);
 	});
 
 	// CHANGE: skip with custom test name for tracking alignment status
+	// TODO: ModuleInfoHeaderPlugin CodeSplitting
 	it.skip(normalizeFilteredTestName(FilteredStatus.TODO, "should compile a file with multiple chunks"), done => {
 		compile("./chunks", {}, (stats, files) => {
 			expect(stats.chunks).toHaveLength(2);
@@ -699,7 +721,7 @@ describe("Compiler", () => {
 	});
 	// CHANGE: skip with custom test name for tracking alignment status
 	// CHANGE: skip due to panic occurred at runtime
-	it.skip(normalizeFilteredTestName(FilteredStatus.TODO, "should call afterDone hook after other callbacks (watch)"), done => {
+	it("should call afterDone hook after other callbacks (watch)", done => {
 		const webpack = require("@rspack/core");
 		compiler = webpack({
 			context: __dirname,
@@ -788,9 +810,7 @@ describe("Compiler", () => {
 			});
 		});
 	});
-	// CHANGE: skip with custom test name for tracking alignment status
-	// CHANGE: skip due to panic occurred at runtime
-	it.skip(normalizeFilteredTestName(FilteredStatus.TODO, "should use cache on second run call"), done => {
+	it("should use cache on second run call", done => {
 		const webpack = require("@rspack/core");
 		compiler = webpack({
 			context: __dirname,