fix: newExpression arguments not be traverse issue (#597)

Co-authored-by: 华志林 <linmu.hzl@raycloud.com>
umijs · Nov 3, 2021 · d21264e · d21264e
1 parent e86af59
commit d21264e
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/src/Plugin.js b/src/Plugin.js
@@ -274,7 +274,10 @@ export default class Plugin {
 
   NewExpression(path, state) {
     const { node } = path;
-    this.buildExpressionHandler(node, ['callee', 'arguments'], path, state);
+    this.buildExpressionHandler(node, ['callee'], path, state);
+
+    const argumentsProps = node.arguments.map((_, index) => index);
+    this.buildExpressionHandler(node.arguments, argumentsProps, path, state);
   }
 
   SwitchStatement(path, state) {

diff --git a/test/fixtures/as-arguments-identifier/actual.js b/test/fixtures/as-arguments-identifier/actual.js
@@ -1,2 +1,3 @@
 import { Modal } from 'antd';
 const _Modal = bind()(Modal);
+const _newModal = new AnyClass(Modal);
diff --git a/test/fixtures/as-arguments-identifier/expected.js b/test/fixtures/as-arguments-identifier/expected.js
@@ -4,4 +4,6 @@ var _modal = _interopRequireDefault(require("antd/lib/modal"));
 
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
-var _Modal = bind()(_modal.default);
+var _Modal = bind()(_modal.default);
+
+var _newModal = new AnyClass(_modal.default);