Fix default configuration for jsx-boolean-value (fixes jsx-eslint#210)

Author: yannickcr

CHANGELOG.md

@@ -10,11 +10,13 @@ This change log adheres to standards from [Keep a CHANGELOG](http://keepachangel
 
 ### Fixed
 * Fix allowed methods on arrayOf in `prop-types` ([#146][])
+* Fix default configuration for `jsx-boolean-value` ([#210][])
 
 [Unreleased]: https://github.com/yannickcr/eslint-plugin-react/compare/v3.3.1...HEAD
 [#146]: https://github.com/yannickcr/eslint-plugin-react/issues/146
 [#197]: https://github.com/yannickcr/eslint-plugin-react/pull/197
 [#206]: https://github.com/yannickcr/eslint-plugin-react/pull/206
+[#210]: https://github.com/yannickcr/eslint-plugin-react/issues/210
 
 ## [3.3.1] - 2015-09-01
 ### Changed

lib/rules/jsx-boolean-value.js

@@ -10,7 +10,7 @@
 
 module.exports = function(context) {
 
-  var configuration = context.options[0] || {};
+  var configuration = context.options[0] || 'never';
 
   var NEVER_MESSAGE = 'Value must be omitted for boolean attributes';
   var ALWAYS_MESSAGE = 'Value must be set for boolean attributes';

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

@@ -19,11 +19,14 @@ var ruleTester = new RuleTester();
 ruleTester.run('jsx-boolean-value', rule, {
   valid: [
     {code: '<App foo />;', options: ['never'], ecmaFeatures: {jsx: true}},
+    {code: '<App foo />;', ecmaFeatures: {jsx: true}},
     {code: '<App foo={true} />;', options: ['always'], ecmaFeatures: {jsx: true}}
   ],
   invalid: [
     {code: '<App foo={true} />;', options: ['never'],
      errors: [{message: 'Value must be omitted for boolean attributes'}], ecmaFeatures: {jsx: true}},
+    {code: '<App foo={true} />;',
+     errors: [{message: 'Value must be omitted for boolean attributes'}], ecmaFeatures: {jsx: true}},
     {code: '<App foo />;', options: ['always'],
      errors: [{message: 'Value must be set for boolean attributes'}], ecmaFeatures: {jsx: true}}
   ]