Fix: Keep indentation when fixing padded-blocks "never" (fixes esli…

…nt#6454)
Mardak · Jun 17, 2016 · 8fa380d · 8fa380d
1 parent 414206c
commit 8fa380d
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/lib/rules/padded-blocks.js b/lib/rules/padded-blocks.js
@@ -190,7 +190,7 @@ module.exports = {
                         node: node,
                         loc: { line: openBrace.loc.start.line, column: openBrace.loc.start.column },
                         fix: function(fixer) {
-                            return fixer.replaceTextRange([openBrace.end, nextToken.start], "\n");
+                            return fixer.replaceTextRange([openBrace.end, nextToken.start - nextToken.loc.start.column], "\n");
                         },
                         message: NEVER_MESSAGE
                     });

diff --git a/tests/lib/rules/padded-blocks.js b/tests/lib/rules/padded-blocks.js
@@ -307,7 +307,7 @@ ruleTester.run("padded-blocks", rule, {
         },
         {
             code: "{\n\n\n  a();\n\n\n}",
-            output: "{\na();\n}",
+            output: "{\n  a();\n}",
             options: ["never"],
             errors: [
                 {
@@ -331,6 +331,17 @@ ruleTester.run("padded-blocks", rule, {
                 }
             ]
         },
+        {
+            code: "{\n\n\ta();\n}",
+            output: "{\n\ta();\n}",
+            options: ["never"],
+            errors: [
+                {
+                    message: NEVER_MESSAGE,
+                    line: 1
+                }
+            ]
+        },
         {
             code: "{\na();\n\n}",
             output: "{\na();\n}",