Implemented new treatment for old export declaration (module.exports)…

… in ReactForwardedRefOrMemo components. (#1) * Implemented new treatment for old export declaration (module.exports) in ReactForwardedRefOrMemo components * fix test of moduleExportsInMemoOrForwardRefComponent * removed displayName of module.exports and fix version of babel * fix to remove displayName in module.exports case * changed fix to ignore MemberExpression node type Co-authored-by: Lucas Victor <[email protected]>
geekie · Aug 20, 2021 · d7d9abf · d7d9abf
1 parent aa2914f
commit d7d9abf
Show file tree

Hide file tree

Showing 4 changed files with 35 additions and 3 deletions.
diff --git a/index.js b/index.js
@@ -60,7 +60,8 @@ function transform (babel) {
           }
 
           var id = path.node.arguments[0].id || findCandidateNameForExpression(path);
-          if (id) {
+
+          if (id && id.type !== 'MemberExpression') {
             setDisplayNameUsingClosure(path.get('arguments.0'), id, babel.types);
           }
         }

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@geekie/babel-plugin-add-react-displayname",
-  "version": "0.0.5",
+  "version": "0.1.1",
   "description": "Automatically add displayName to all your components",
   "main": "index.js",
   "dependencies": {},
@@ -22,7 +22,8 @@
   },
   "author": "Ron Cohen",
   "contributors": [
-    "Rodrigo Almeida"
+    "Rodrigo Almeida",
+    "Lucas Victor"
   ],
   "license": "MIT",
   "bugs": {

diff --git a/test/fixtures/reactModuleExportsInMemoForwardRef/expected.js b/test/fixtures/reactModuleExportsInMemoForwardRef/expected.js
@@ -0,0 +1,15 @@
+import React from "react";
+
+let TestComponent = class TestComponent extends React.Component {
+  constructor(props) {
+    super(props);
+  }
+
+  render() {
+    return React.createElement("div", null);
+  }
+};
+TestComponent.displayName = "TestComponent";
+
+
+module.exports = React.forwardRef((props, state) => React.createElement(TestComponent, null));
diff --git a/test/fixtures/reactModuleExportsInMemoForwardRef/input.js b/test/fixtures/reactModuleExportsInMemoForwardRef/input.js
@@ -0,0 +1,15 @@
+import React from "react";
+
+class TestComponent extends React.Component {
+  constructor(props) {
+    super(props);
+  }
+
+  render() {
+    return (
+      <div></div>
+    )
+  }
+}
+
+module.exports = React.forwardRef((props, state) => <TestComponent />);