From 9dfb3a8a74b4d3d642e97e1014127a4d63dec02c Mon Sep 17 00:00:00 2001 From: st-sloth Date: Wed, 10 Oct 2018 13:13:26 +0500 Subject: [PATCH] Fix: Remove extraneous linefeeds in `one-var` fixer (#10741) --- lib/rules/one-var.js | 8 ++++++-- tests/lib/rules/one-var.js | 8 ++++---- 2 files changed, 10 insertions(+), 6 deletions(-) diff --git a/lib/rules/one-var.js b/lib/rules/one-var.js index 15b664d77a3c..9fe59b2fa6b3 100644 --- a/lib/rules/one-var.js +++ b/lib/rules/one-var.js @@ -334,7 +334,11 @@ module.exports = { * y` * ^ afterComma */ - if (afterComma.loc.start.line > tokenAfterDeclarator.loc.end.line || afterComma.type === "Line" || afterComma.type === "Block") { + if ( + afterComma.loc.start.line > tokenAfterDeclarator.loc.end.line || + afterComma.type === "Line" || + afterComma.type === "Block" + ) { let lastComment = afterComma; while (lastComment.type === "Line" || lastComment.type === "Block") { @@ -343,7 +347,7 @@ module.exports = { return fixer.replaceTextRange( [tokenAfterDeclarator.range[0], lastComment.range[0]], - `;\n${sourceCode.text.slice(tokenAfterDeclarator.range[1], lastComment.range[0])}\n${declaration.kind} ` + `;${sourceCode.text.slice(tokenAfterDeclarator.range[1], lastComment.range[0])}${declaration.kind} ` ); } diff --git a/tests/lib/rules/one-var.js b/tests/lib/rules/one-var.js index b628f512a9fd..100886eb57b6 100644 --- a/tests/lib/rules/one-var.js +++ b/tests/lib/rules/one-var.js @@ -884,7 +884,7 @@ ruleTester.run("one-var", rule, { }, { code: "const foo = 1,\n bar = 2;", - output: "const foo = 1;\n\n \nconst bar = 2;", + output: "const foo = 1;\n const bar = 2;", options: [{ initialized: "never" }], parserOptions: { ecmaVersion: 6 }, errors: [{ @@ -896,7 +896,7 @@ ruleTester.run("one-var", rule, { }, { code: "var foo = 1,\n bar = 2;", - output: "var foo = 1;\n\n \nvar bar = 2;", + output: "var foo = 1;\n var bar = 2;", options: [{ initialized: "never" }], errors: [{ message: "Split initialized 'var' declarations into multiple statements.", @@ -907,7 +907,7 @@ ruleTester.run("one-var", rule, { }, { code: "var foo = 1, // comment\n bar = 2;", - output: "var foo = 1;\n // comment\n \nvar bar = 2;", + output: "var foo = 1; // comment\n var bar = 2;", options: [{ initialized: "never" }], errors: [{ message: "Split initialized 'var' declarations into multiple statements.", @@ -929,7 +929,7 @@ ruleTester.run("one-var", rule, { }, { code: "var f, /* test */ l;", - output: "var f;\n /* test */ \nvar l;", + output: "var f; /* test */ var l;", options: ["never"], errors: [{ message: "Split 'var' declarations into multiple statements.",