From bd09cd50283760b57949802c55a6e2b5862c6d15 Mon Sep 17 00:00:00 2001 From: Teddy Katz Date: Wed, 9 Aug 2017 15:02:34 -0700 Subject: [PATCH] Update: avoid requiring NaN spaces of indentation (fixes #9083) (#9085) This fixes an issue where the `indent` rule would sometimes expect "NaN spaces" of indentation when an `"off"` option was used. --- lib/rules/indent.js | 2 +- tests/lib/rules/indent.js | 18 ++++++++++++++++++ 2 files changed, 19 insertions(+), 1 deletion(-) diff --git a/lib/rules/indent.js b/lib/rules/indent.js index 45332f4473c..429567ee560 100644 --- a/lib/rules/indent.js +++ b/lib/rules/indent.js @@ -783,7 +783,7 @@ module.exports = { offsets.setDesiredOffsets( [startToken.range[1], endToken.range[0]], startToken, - offset === "first" ? 1 : offset + typeof offset === "number" ? offset : 1 ); offsets.setDesiredOffset(endToken, startToken, 0); diff --git a/tests/lib/rules/indent.js b/tests/lib/rules/indent.js index e2e843bce44..be026fe514a 100644 --- a/tests/lib/rules/indent.js +++ b/tests/lib/rules/indent.js @@ -4915,6 +4915,24 @@ ruleTester.run("indent", rule, { ); } ` + }, + { + code: unIndent` + a(b + , c + ) + `, + options: [2, { CallExpression: { arguments: "off" } }] + }, + { + code: unIndent` + a( + new B({ + c, + }) + ); + `, + options: [2, { CallExpression: { arguments: "off" } }] } ],