From d816b8f3faacf955d94d4e9736674286deecaf2f Mon Sep 17 00:00:00 2001 From: Mark Macdonald Date: Thu, 18 Jul 2013 13:38:05 -0400 Subject: [PATCH] Adds test and fix for issue #82 --- lib/rules/new-parens.js | 2 +- tests/lib/rules/new-parens.js | 15 +++++++++++++++ 2 files changed, 16 insertions(+), 1 deletion(-) diff --git a/lib/rules/new-parens.js b/lib/rules/new-parens.js index b000fc703b1..df33d6e38fb 100644 --- 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 index bbf5ad8e1f8..13df4e3e1fe 100644 --- 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: {} };