CLJS-2332: module_deps.js doesn't process export from correctly

Author: anmonteiro

src/main/cljs/cljs/module_deps.js

@@ -20,7 +20,7 @@ let getDeps = function (src, {dynamicImport = true, parse = {sourceType: 'module
     let ast;
 
     if (typeof src === 'string') {
-        const moduleRe = /\b(require|import)\b/;
+        const moduleRe = /\b(require|import|export)\b/;
 
         if (!moduleRe.test(src)) {
             return modules;
@@ -44,11 +44,14 @@ let getDeps = function (src, {dynamicImport = true, parse = {sourceType: 'module
                         modules.expressions.push(src.slice(arg.start, arg.end));
                     }
                 }
-            } else if (path.node.type === 'ImportDeclaration') {
-                modules.strings.push(path.node.source.value);
-            } else if (path.node.type === 'ExportNamedDeclaration' && path.node.source) {
-                // this branch handles `export ... from` - David
-                modules.strings.push(path.node.source.value);
+            } else if (path.node.type === 'ImportDeclaration' ||
+                       path.node.type === 'ExportNamedDeclaration' ||
+                       path.node.type === 'ExportAllDeclaration') {
+                const source = path.node.source;
+
+                if (source != null) {
+                    modules.strings.push(path.node.source.value);
+                }
             }
         }
     });

src/test/clojure/cljs/closure_tests.clj

@@ -369,3 +369,21 @@
                  modules))))
   (.delete (io/file "package.json"))
   (test/delete-node-modules))
+
+(deftest test-cljs-2332
+  (spit (io/file "package.json") "{}")
+  (let [opts {:npm-deps {"@material/drawer" "0.5.4"}}
+        out (util/output-directory opts)]
+    (test/delete-node-modules)
+    (test/delete-out-files out)
+    (closure/maybe-install-node-deps! opts)
+    (let [modules (closure/index-node-modules ["@material/drawer"] opts)]
+      (is (true? (some (fn [module]
+                         (= module {:module-type :es6
+                                    :file (.getAbsolutePath (io/file "node_modules/@material/drawer/slidable/constants.js"))
+                                    :provides ["@material/drawer/slidable/constants.js"
+                                               "@material/drawer/slidable/constants"]}))
+                   modules))))
+    (.delete (io/file "package.json"))
+    (test/delete-node-modules)
+    (test/delete-out-files out)))