Merge pull request #83 from mamacdon/new-parens-func-fix

Adds test and fixes #82
eslint · Jul 20, 2013 · 934d766 · 934d766
2 parents 8d4bce1 + d816b8f
commit 934d766
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/lib/rules/new-parens.js b/lib/rules/new-parens.js
@@ -16,7 +16,7 @@ module.exports = function(context) {
             var prenticesTokens = tokens.filter(function(token) {
                 return token.value === "(" || token.value === ")";
             });
-            if (prenticesTokens.length !== 2) {
+            if (prenticesTokens.length < 2) {
                 context.report(node, "Missing '()' invoking a constructor");
             }
         }

diff --git a/tests/lib/rules/new-parens.js b/tests/lib/rules/new-parens.js
@@ -46,6 +46,21 @@ vows.describe(RULE_ID).addBatch({
 
         topic: "var a = new Date();",
 
+        "should not report a violation": function(topic) {
+
+            var config = { rules: {} };
+            config.rules[RULE_ID] = 1;
+
+            var messages = eslint.verify(topic, config);
+
+            assert.equal(messages.length, 0);
+        }
+    },
+
+	"when evaluating 'var a = new Date(function() {})'": {
+
+        topic: "var a = new Date(function() {});",
+
         "should not report a violation": function(topic) {
 
             var config = { rules: {} };