Validate parenthesis in JSXExpressionContainers

Author: Joachim Seminck

lib/rules/jsx-wrap-multilines.js

@@ -14,7 +14,8 @@ const DEFAULTS = {
   declaration: true,
   assignment: true,
   return: true,
-  arrow: true
+  arrow: true,
+  conditional: false
 };
 
 // ------------------------------------------------------------------------------
@@ -44,6 +45,9 @@ module.exports = {
         },
         arrow: {
           type: 'boolean'
+        },
+        conditional: {
+          type: 'boolean'
         }
       },
       additionalProperties: false
@@ -126,6 +130,14 @@ module.exports = {
         }
       },
 
+      JSXExpressionContainer: function(node) {
+        if (!isEnabled('conditional')) {
+          return;
+        }
+
+        check(node.expression.right);
+      },
+
       'ArrowFunctionExpression:exit': function (node) {
         const arrowBody = node.body;
 

tests/lib/rules/jsx-wrap-multilines.js

@@ -134,6 +134,26 @@ const ARROW_NO_PAREN = `
   </div>;
 `;
 
+const CONDITIONAL_PAREN = `
+  <div>
+    {unreadMessages.length > 0 &&
+      (<h2>
+        You have {unreadMessages.length} unread messages.
+      </h2>)
+    }
+  </div>
+`;
+
+const CONDITIONAL_NO_PAREN = `
+  <div>
+    {unreadMessages.length > 0 &&
+      <h2>
+        You have {unreadMessages.length} unread messages.
+      </h2>
+    }
+  </div>
+`;
+
 // ------------------------------------------------------------------------------
 // Tests
 // ------------------------------------------------------------------------------
@@ -187,6 +207,13 @@ ruleTester.run('jsx-wrap-multilines', rule, {
     }, {
       code: ARROW_NO_PAREN,
       options: [{arrow: false}]
+    }, {
+      code: CONDITIONAL_PAREN
+    }, {
+      code: CONDITIONAL_PAREN,
+      options: [{conditional: true}]
+    }, {
+      code: CONDITIONAL_NO_PAREN
     }
   ],
 
@@ -237,6 +264,11 @@ ruleTester.run('jsx-wrap-multilines', rule, {
       output: ARROW_PAREN,
       options: [{arrow: true}],
       errors: [{message: 'Missing parentheses around multilines JSX'}]
+    }, {
+      code: CONDITIONAL_NO_PAREN,
+      output: CONDITIONAL_PAREN,
+      options: [{conditional: true}],
+      errors: [{message: 'Missing parentheses around multilines JSX'}]
     }
   ]
 });