[Fix] jsx-indent: don't check literals not within JSX

Fixes #2563

Author: toshi-toma

lib/rules/jsx-indent.js

@@ -374,6 +374,9 @@ module.exports = {
       if (!node.parent) {
         return;
       }
+      if (node.parent.type !== 'JSXElement' && node.parent.type !== 'JSXFragment') {
+        return;
+      }
       const parentNodeIndent = getNodeIndent(node.parent);
       checkLiteralNodeIndent(node, parentNodeIndent + indentSize);
     }

tests/lib/rules/jsx-indent.js

@@ -989,6 +989,14 @@ const Component = () => (
       '</App>'
     ].join('\n'),
     options: ['tab']
+  }, {
+    // don't check literals not within JSX. See #2563
+    code: [
+      'function foo() {',
+      'const a = `aa`;',
+      'const b = `b\nb`;',
+      '}'
+    ].join('\n')
   }],
 
   invalid: [{
@@ -2003,5 +2011,20 @@ const Component = () => (
     errors: [
       {message: 'Expected indentation of 1 tab character but found 2.'}
     ]
+  }, {
+    code: [
+      '<>',
+      'aaa',
+      '</>'
+    ].join('\n'),
+    parser: parsers.BABEL_ESLINT,
+    output: [
+      '<>',
+      '    aaa',
+      '</>'
+    ].join('\n'),
+    errors: [
+      {message: 'Expected indentation of 4 space characters but found 0.'}
+    ]
   }]
 });