fix: should handle require ensure with error function (#12054)

Author: LingyuCoder

crates/rspack_plugin_javascript/src/parser_plugin/require_ensure_dependencies_block_parse_plugin.rs

@@ -75,16 +75,12 @@ impl JavascriptParserPlugin for RequireEnsureDependenciesBlockParserPlugin {
     let chunk_name = match expr
       .args
       .get(3)
-      .or(error_expr.as_ref().and(None)) // !errorExpression
-      .or(expr.args.get(2))
+      .or_else(|| if error_expr.is_some() { None } else { expr.args.get(2) }) // !errorExpression
     {
-      Some(arg) => {
-        let chunk_name_expr = parser.evaluate_expression(&arg.expr);
-        match chunk_name_expr.as_string() {
-          Some(chunk_name_expr) => Some(chunk_name_expr),
-          None => return None,
-        }
-      }
+      Some(arg) => match parser.evaluate_expression(&arg.expr).as_string() {
+        Some(chunk_name) => Some(chunk_name),
+        None => return None,
+      },
       None => None,
     };
 

tests/rspack-test/configCases/parsing/dead-code-elimination-require-ensure/test.filter.js

@@ -1 +1 @@
-module.exports = () => "FIXME: build failed";
+module.exports = () => "TODO: support parser.scope.terminated"