[Fix] jsx-boolean-value: assumeUndefinedIsFalse with never must not allow explicit true value

Author: 6uliver

lib/rules/jsx-boolean-value.js

@@ -132,7 +132,6 @@ module.exports = {
         }
         if (
           isNever(configuration, exceptions, propName)
-          && !configObject.assumeUndefinedIsFalse
           && value
           && value.type === 'JSXExpressionContainer'
           && value.expression.value === true

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

@@ -52,6 +52,10 @@ ruleTester.run('jsx-boolean-value', rule, {
       code: '<App />;',
       options: ['never', { assumeUndefinedIsFalse: true }],
     },
+    {
+      code: '<App foo={false} />;',
+      options: ['never', { assumeUndefinedIsFalse: false }],
+    },
     {
       code: '<App foo={false} />;',
       options: ['never', { assumeUndefinedIsFalse: true, always: ['foo'] }],
@@ -145,6 +149,21 @@ ruleTester.run('jsx-boolean-value', rule, {
         },
       ],
     },
+    {
+      code: '<App foo={true} bak={false} />;',
+      output: '<App foo  />;',
+      options: ['never', { assumeUndefinedIsFalse: true }],
+      errors: [
+        {
+          messageId: 'omitBoolean',
+          data: { propName: 'foo' },
+        },
+        {
+          messageId: 'omitPropAndBoolean',
+          data: { propName: 'bak' },
+        },
+      ],
+    },
     {
       code: '<App foo={true} bar={false} baz={false} bak={false} />;',
       output: '<App foo={true} bar={false}   />;',