Adds test and fix for issue #82

eslint · Jul 18, 2013 · d816b8f · d816b8f
1 parent 9637601
commit d816b8f
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: {} };