jsx-indent rule autofixer

Author: eelyafi

lib/rules/jsx-indent.js

@@ -39,7 +39,7 @@ module.exports = {
       category: 'Stylistic Issues',
       recommended: false
     },
-
+    fixable: 'whitespace',
     schema: [{
       oneOf: [{
         enum: ['tab']
@@ -69,6 +69,30 @@ module.exports = {
       }
     }
 
+    var indentChar = indentType === 'space' ? ' ' : '\u0009';
+
+    /**
+     * Responsible for fixing the indentation issue fix
+     * @param {ASTNode} node Node violating the indent rule
+     * @param {Number} needed Expected indentation character count
+     * @param {Number} gotten Indentation character count in the actual node/code
+     * @returns {Function} function to be executed by the fixer
+     * @private
+     */
+    function getFixerFunction(node, needed, gotten) {
+      if (needed > gotten) {
+        var spaces = indentChar.repeat(needed - gotten);
+
+        return function(fixer) {
+          return fixer.insertTextBeforeRange([node.range[0], node.range[0]], spaces);
+        };
+      }
+
+      return function(fixer) {
+        return fixer.removeRange([node.range[0] - (gotten - needed), node.range[0]]);
+      };
+    }
+
     /**
      * Reports a given indent violation and properly pluralizes the message
      * @param {ASTNode} node Node violating the indent rule
@@ -89,13 +113,15 @@ module.exports = {
           node: node,
           loc: loc,
           message: MESSAGE,
-          data: msgContext
+          data: msgContext,
+          fix: getFixerFunction(node, needed, gotten)
         });
       } else {
         context.report({
           node: node,
           message: MESSAGE,
-          data: msgContext
+          data: msgContext,
+          fix: getFixerFunction(node, needed, gotten)
         });
       }
     }

tests/lib/rules/jsx-indent.js

@@ -107,6 +107,11 @@ ruleTester.run('jsx-indent', rule, {
       '  <Foo />',
       '</App>'
     ].join('\n'),
+    output: [
+      '<App>',
+      '    <Foo />',
+      '</App>'
+    ].join('\n'),
     parserOptions: parserOptions,
     errors: [{message: 'Expected indentation of 4 space characters but found 2.'}]
   }, {
@@ -115,6 +120,11 @@ ruleTester.run('jsx-indent', rule, {
       '    <Foo />',
       '</App>'
     ].join('\n'),
+    output: [
+      '<App>',
+      '  <Foo />',
+      '</App>'
+    ].join('\n'),
     options: [2],
     parserOptions: parserOptions,
     errors: [{message: 'Expected indentation of 2 space characters but found 4.'}]
@@ -135,6 +145,13 @@ ruleTester.run('jsx-indent', rule, {
       '         </App>;',
       '}'
     ].join('\n'),
+    output: [
+      'function App() {',
+      '  return <App>',
+      '    <Foo />',
+      '  </App>;',
+      '}'
+    ].join('\n'),
     options: [2],
     parserOptions: parserOptions,
     errors: [{message: 'Expected indentation of 2 space characters but found 9.'}]
@@ -146,6 +163,13 @@ ruleTester.run('jsx-indent', rule, {
       '    </App>);',
       '}'
     ].join('\n'),
+    output: [
+      'function App() {',
+      '  return (<App>',
+      '    <Foo />',
+      '  </App>);',
+      '}'
+    ].join('\n'),
     options: [2],
     parserOptions: parserOptions,
     errors: [{message: 'Expected indentation of 2 space characters but found 4.'}]