[patch] Add module.exports to files using es3 member literals

59naga · Oct 22, 2018 · df47a0a · df47a0a
1 parent e8e5be0
commit df47a0a
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 1 deletion.
diff --git a/src/index.js b/src/index.js
@@ -103,7 +103,10 @@ class ExportsFinder {
     }
 
     const objectName = path.get(`${property}.left.object.name`).node
-    const propertyName = path.get(`${property}.left.property.name`).node
+    // Check name of  MemberExpressions and values of StringLiterals
+    const propertyName =
+      path.get(`${property}.left.property.name`).node ||
+      path.get(`${property}.left.property.value`).node
     if (objectName === 'exports' || objectName === '_exports') {
       if (propertyName === 'default') {
         this.hasExportsDefault = true

diff --git a/test/spec.js b/test/spec.js
@@ -208,5 +208,27 @@ module.exports = [
       module: 'default-entry',
       exports: 'default-entry'
     }
+  },
+  {
+    name: 'handle a single quote string literal export',
+    code: `
+          Object.defineProperty(exports, '__esModule', {value: true});
+          exports['default'] = 'foo';
+        `,
+    expected: {
+      module: 'foo',
+      exports: 'foo'
+    }
+  },
+  {
+    name: 'handle a double quote string literal export',
+    code: `
+          Object.defineProperty(exports, '__esModule', {value: true});
+          exports["default"] = 'foo';
+        `,
+    expected: {
+      module: 'foo',
+      exports: 'foo'
+    }
   }
 ]