fix concatenated version of reexport dependency for sideEffects
fixes #9159
This commit is contained in:
parent
6ca9167e9e
commit
fec26a9503
|
@ -521,7 +521,7 @@ class ConcatenatedModule extends Module {
|
|||
dep instanceof HarmonyExportImportedSpecifierDependency
|
||||
) {
|
||||
const exportName = dep.name;
|
||||
const importName = dep.id;
|
||||
const importName = dep._id;
|
||||
const importedModule = dep._module;
|
||||
if (exportName && importName) {
|
||||
if (!reexportMap.has(exportName)) {
|
||||
|
@ -1390,12 +1390,12 @@ class HarmonyExportImportedSpecifierDependencyConcatenatedTemplate {
|
|||
|
||||
getExports(dep) {
|
||||
const importModule = dep._module;
|
||||
if (dep.id) {
|
||||
if (dep._id) {
|
||||
// export { named } from "module"
|
||||
return [
|
||||
{
|
||||
name: dep.name,
|
||||
id: dep.id,
|
||||
id: dep._id,
|
||||
module: importModule
|
||||
}
|
||||
];
|
||||
|
|
|
@ -0,0 +1,5 @@
|
|||
it("should reexport from side-effect-free scope-hoisted module", () => {
|
||||
const m = require("./reexport");
|
||||
expect(m.value).toBe(42);
|
||||
expect(m.ns.default).toBe(42);
|
||||
});
|
|
@ -0,0 +1,4 @@
|
|||
export { default as value } from "./module";
|
||||
|
||||
import * as ns from "./module";
|
||||
export { ns };
|
|
@ -0,0 +1 @@
|
|||
export default 42;
|
|
@ -0,0 +1,3 @@
|
|||
{
|
||||
"sideEffects": false
|
||||
}
|
|
@ -0,0 +1 @@
|
|||
export { value, ns } from "./package";
|
Loading…
Reference in New Issue