[Fix] jsx-one-expression-per-line: fix when using tabs

Fixes #2152.

Author: Pavel Lysenko

lib/rules/jsx-one-expression-per-line.js

@@ -94,8 +94,8 @@ module.exports = {
             return;
           }
 
-          countNewLinesBeforeContent = (child.raw.match(/^ *\n/g) || []).length;
-          countNewLinesAfterContent = (child.raw.match(/\n *$/g) || []).length;
+          countNewLinesBeforeContent = (child.raw.match(/^\s*\n/g) || []).length;
+          countNewLinesAfterContent = (child.raw.match(/\n\s*$/g) || []).length;
         }
 
         const startLine = child.loc.start.line + countNewLinesBeforeContent;

tests/lib/rules/jsx-one-expression-per-line.js

@@ -28,6 +28,42 @@ const ruleTester = new RuleTester({parserOptions});
 ruleTester.run('jsx-one-expression-per-line', rule, {
   valid: [{
     code: '<App />'
+  }, {
+    code: `
+\t\t<AllTabs>
+\t\t\tFail
+\t\t</AllTabs>
+    `
+  }, {
+    code: `
+\t\t<TagsWithTabs>
+      Fail
+\t\t</TagsWithTabs>
+    `
+  }, {
+    code: `
+    <ClosedTagWithTabs>
+      Fail
+\t\t</ClosedTagWithTabs>
+    `
+  }, {
+    code: `
+\t\t<OpenTagWithTabs>
+      OK
+    </OpenTagWithTabs>
+    `
+  }, {
+    code: `
+    <TextWithTabs>
+\t\t\tOK
+    </TextWithTabs>
+    `
+  }, {
+    code: `
+    <AllSpaces>
+      OK
+    </AllSpaces>
+    `
   }, {
     code: '<App></App>'
   }, {