support parsing null dependencies. fix #2

spmjs · Apr 19, 2013 · cacb749 · cacb749
1 parent 424dfd8
commit cacb749
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/lib/ast.js b/lib/ast.js
@@ -200,8 +200,12 @@ function getDefine(node) {
             return el.getValue();
           }
         });
+        dependencyNode = secondChild;
+      } else if (secondChild instanceof UglifyJS.AST_Null) {
+        if (factory instanceof UglifyJS.AST_Function) {
+          dependencies = getRequires(factory);
+        }
       }
-      dependencyNode = secondChild;
     }
   }
   return {

diff --git a/tests/ast.test.js b/tests/ast.test.js
@@ -129,6 +129,15 @@ describe('ast.parse', function() {
     ].join('\n');
     ast.parseFirst(code).id.should.equal('jquery');
   });
+
+  it('can parse null dependencies', function() {
+    var code = [
+      "define('id', null, function(require) {",
+      "  require('jquery');",
+      "})"
+    ].join('\n');
+    ast.parseFirst(code).dependencies.should.eql(['jquery']);
+  });
 });
 
 describe('ast.modify', function() {