Add auto fix for jsx-boolean-value

Author: yannickcr

README.md

@@ -117,7 +117,7 @@ Finally, enable all of the rules that you would like to use.
 
 * [display-name](docs/rules/display-name.md): Prevent missing `displayName` in a React component definition
 * [forbid-prop-types](docs/rules/forbid-prop-types.md): Forbid certain propTypes
-* [jsx-boolean-value](docs/rules/jsx-boolean-value.md): Enforce boolean attributes notation in JSX
+* [jsx-boolean-value](docs/rules/jsx-boolean-value.md): Enforce boolean attributes notation in JSX (fixable)
 * [jsx-closing-bracket-location](docs/rules/jsx-closing-bracket-location.md): Validate closing bracket location in JSX
 * [jsx-curly-spacing](docs/rules/jsx-curly-spacing.md): Enforce or disallow spaces inside of curly braces in JSX attributes
 * [jsx-handler-names](docs/rules/jsx-handler-names.md): Enforce event handler naming conventions in JSX

docs/rules/jsx-boolean-value.md

@@ -2,6 +2,8 @@
 
 In JSX when using a boolean attribute you can set the attribute value to `true` or omit the value. This rule will enforce one or the other to keep consistency in your code.
 
+**Fixable:** This rule is automatically fixable using the `--fix` flag on the command line.
+
 ## Rule Details
 
 This rule takes one argument. If it is `"always"` then it warns whenever an attribute is missing it's value. If `"never"` then it warns if an attribute has a `true` value. The default value of this option is `"never"`.

lib/rules/jsx-boolean-value.js

@@ -20,12 +20,24 @@ module.exports = function(context) {
       switch (configuration) {
         case 'always':
           if (node.value === null) {
-            context.report(node, ALWAYS_MESSAGE);
+            context.report({
+              node: node,
+              message: ALWAYS_MESSAGE,
+              fix: function(fixer) {
+                return fixer.insertTextAfter(node, '={true}');
+              }
+            });
           }
           break;
         case 'never':
           if (node.value && node.value.type === 'JSXExpressionContainer' && node.value.expression.value === true) {
-            context.report(node, NEVER_MESSAGE);
+            context.report({
+              node: node,
+              message: NEVER_MESSAGE,
+              fix: function(fixer) {
+                return fixer.removeRange([node.name.range[1], node.value.range[1]]);
+              }
+            });
           }
           break;
         default:

tests/lib/rules/jsx-boolean-value.js

@@ -30,11 +30,13 @@ ruleTester.run('jsx-boolean-value', rule, {
     {code: '<App foo={true} />;', options: ['always'], parserOptions: parserOptions}
   ],
   invalid: [
-    {code: '<App foo={true} />;', options: ['never'],
+    {code: '<App foo={true} />;', output: '<App foo />;', options: ['never'],
      errors: [{message: 'Value must be omitted for boolean attributes'}], parserOptions: parserOptions},
-    {code: '<App foo={true} />;',
+    {code: '<App foo={true} />;', output: '<App foo />;',
      errors: [{message: 'Value must be omitted for boolean attributes'}], parserOptions: parserOptions},
-    {code: '<App foo />;', options: ['always'],
+    {code: '<App foo = {true} />;', output: '<App foo />;',
+     errors: [{message: 'Value must be omitted for boolean attributes'}], parserOptions: parserOptions},
+    {code: '<App foo />;', output: '<App foo={true} />;', options: ['always'],
      errors: [{message: 'Value must be set for boolean attributes'}], parserOptions: parserOptions}
   ]
 });