Merge pull request #1077 from iancmyers/fix-jsx-indent-template-conditional

Fix error caused by templates in ConditionalExpressions (jsx-indent)

Author: ljharb

lib/rules/jsx-indent.js

@@ -275,7 +275,8 @@ module.exports = {
         do {
           prevToken = sourceCode.getTokenBefore(prevToken);
         } while (prevToken.type === 'Punctuator');
-        prevToken = sourceCode.getNodeByRangeIndex(prevToken.start);
+        prevToken = sourceCode.getNodeByRangeIndex(prevToken.range[0]);
+
         while (prevToken.parent && prevToken.parent.type !== 'ConditionalExpression') {
           prevToken = prevToken.parent;
         }

tests/lib/rules/jsx-indent.js

@@ -410,6 +410,47 @@ ruleTester.run('jsx-indent', rule, {
     ].join('\n'),
     options: [4, {indentLogicalExpressions: true}],
     parserOptions: parserOptions
+  }, {
+    code: [
+      '<span>',
+      '  {condition ?',
+      '    <Thing',
+      '      foo={`bar`}',
+      '    /> :',
+      '    <Thing/>',
+      '  }',
+      '</span>'
+    ].join('\n'),
+    options: [2],
+    parserOptions: parserOptions
+  }, {
+    code: [
+      '<span>',
+      '  {condition ?',
+      '    <Thing',
+      '      foo={"bar"}',
+      '    /> :',
+      '    <Thing/>',
+      '  }',
+      '</span>'
+    ].join('\n'),
+    options: [2],
+    parserOptions: parserOptions
+  }, {
+    code: [
+      'function foo() {',
+      '  <span>',
+      '    {condition ?',
+      '      <Thing',
+      '        foo={super}',
+      '      /> :',
+      '      <Thing/>',
+      '    }',
+      '  </span>',
+      '}'
+    ].join('\n'),
+    options: [2],
+    parserOptions: parserOptions
   }],
 
   invalid: [{