root
/
webpack
mirror of https://github.com/webpack/webpack.git
1
0
Fork 0
Code Issues Actions Packages Projects Releases Wiki Activity

fix: bug

This commit is contained in:
alexander.akait 2023-04-18 15:02:05 +03:00
parent 98b976e8bf
commit 65d17428da
2 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/NormalModuleFactory.js
View File

@ -1022,9 +1022,9 @@ If changing the source code is not an option there is also a resolve options cal
const parsedResult = this._parseResourceWithoutFragment(result); const parsedResult = this._parseResourceWithoutFragment(result);
const type = /\.mjs$/i.test(parsedResult.loader) const type = /\.mjs$/i.test(parsedResult.path)
? "module" ? "module"
: /\.cjs$/i.test(parsedResult.loader) : /\.cjs$/i.test(parsedResult.path)
? "commonjs" ? "commonjs"
: resolveRequest.descriptionFileData === undefined : resolveRequest.descriptionFileData === undefined
? undefined ? undefined

8
test/cases/loaders/cjs-loader-type/index.js
View File

@ -1,11 +1,11 @@
it("should pass package.json type to loader", function () { it("should pass package.json type to loader", function () {
// expect(require("cjs/loader.js!")).toBe("commonjs"); expect(require("cjs/loader.js!")).toBe("commonjs");
// expect(require("./loader.js!")).toBe("undefined"); expect(require("./loader.js!")).toBe("undefined");
}); });
it("should pass 'commonjs' type to loader for .cjs", function () { it("should pass 'commonjs' type to loader for .cjs", function () {
// expect(require("cjs/loader.cjs!")).toBe("commonjs"); expect(require("cjs/loader.cjs!")).toBe("commonjs");
expect(require("./loader.cjs!")).toBe("undefined"); expect(require("./loader.cjs!")).toBe("commonjs");
// TODO need fix in v8 https://github.com/nodejs/node/issues/35889 // TODO need fix in v8 https://github.com/nodejs/node/issues/35889
// TODO otherwise this test case cause segment fault // TODO otherwise this test case cause segment fault
// expect(require("esm/loader.cjs!")).toBe("commonjs"); // expect(require("esm/loader.cjs!")).toBe("commonjs");