Fix test by using a hack

vercel · Nov 18, 2024 · a9959e6 · a9959e6
1 parent 903a1d8
commit a9959e6
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/turbopack/crates/turbopack-ecmascript/src/references/mod.rs b/turbopack/crates/turbopack-ecmascript/src/references/mod.rs
@@ -57,6 +57,7 @@ use turbopack_core::{
     compile_time_info::{
         CompileTimeInfo, DefineableNameSegment, FreeVarReference, FreeVarReferences,
     },
+    context::AssetContext,
     environment::Rendering,
     error::PrettyPrintError,
     issue::{analyze::AnalyzeIssue, IssueExt, IssueSeverity, IssueSource, StyledString},
@@ -124,7 +125,7 @@ use crate::{
         top_level_await::has_top_level_await,
         ConstantNumber, ConstantString, JsValueUrlKind, RequireContextValue,
     },
-    chunk::EcmascriptExports,
+    chunk::{EcmascriptChunkPlaceable, EcmascriptExports},
     code_gen::{CodeGen, CodeGenerateable, CodeGenerateableWithAsyncModuleInfo, CodeGenerateables},
     magic_identifier,
     parse::parse,
@@ -591,6 +592,10 @@ pub(crate) async fn analyse_ecmascript_module_internal(
         set_handler_and_globals(&handler, globals, || create_graph(program, eval_context));
 
     let mut evaluation_references = Vec::new();
+    let side_effect_free_packages = module.asset_context().side_effect_free_packages();
+    let is_side_effect_free = *module
+        .is_marked_as_side_effect_free(side_effect_free_packages)
+        .await?;
 
     for (i, r) in eval_context.imports.references().enumerate() {
         let r = EsmAssetReference::new(
@@ -607,7 +612,9 @@ pub(crate) async fn analyse_ecmascript_module_internal(
                     }
                     ImportedSymbol::Symbol(name) => Some(ModulePart::export((&**name).into())),
                     ImportedSymbol::PartEvaluation(part_id) => {
-                        evaluation_references.push(i);
+                        if !is_side_effect_free {
+                            evaluation_references.push(i);
+                        }
                         Some(ModulePart::internal_evaluation(*part_id))
                     }
                     ImportedSymbol::Part(part_id) => Some(ModulePart::internal(*part_id)),