fix(eslint-plugin): [no-use-before-define] false positive with jsx pragma reference (typescript-eslint#2503)

bradzacher · web-flow · commit 5afeeab24ad0 · 2020-09-06T11:08:16.000-07:00
Fixes typescript-eslint#2502
diff --git a/packages/eslint-plugin/src/rules/no-use-before-define.ts b/packages/eslint-plugin/src/rules/no-use-before-define.ts
@@ -258,7 +258,7 @@ export default util.createRule<Options, MessageIds>({
           reference.init ||
           !variable ||
           variable.identifiers.length === 0 ||
-          (variable.identifiers[0].range[1] < reference.identifier.range[1] &&
+          (variable.identifiers[0].range[1] <= reference.identifier.range[1] &&
             !isInInitializer(variable, reference)) ||
           !isForbidden(variable, reference)
         ) {
diff --git a/packages/eslint-plugin/tests/rules/no-use-before-define.test.ts b/packages/eslint-plugin/tests/rules/no-use-before-define.test.ts
@@ -290,6 +290,59 @@ enum Foo {
       `,
       options: [{ enums: false }],
     },
+    // https://github.com/typescript-eslint/typescript-eslint/issues/2502
+    {
+      code: `
+import * as React from 'react';
+
+<div />;
+      `,
+      parserOptions: {
+        sourceType: 'module',
+        ecmaFeatures: {
+          jsx: true,
+        },
+      },
+    },
+    {
+      code: `
+import React from 'react';
+
+<div />;
+      `,
+      parserOptions: {
+        sourceType: 'module',
+        ecmaFeatures: {
+          jsx: true,
+        },
+      },
+    },
+    {
+      code: `
+import { h } from 'preact';
+
+<div />;
+      `,
+      parserOptions: {
+        sourceType: 'module',
+        jsxPragma: 'h',
+        ecmaFeatures: {
+          jsx: true,
+        },
+      },
+    },
+    {
+      code: `
+const React = require('react');
+
+<div />;
+      `,
+      parserOptions: {
+        ecmaFeatures: {
+          jsx: true,
+        },
+      },
+    },
   ],
   invalid: [
     {
