[react/jsx-uses-vars]: False negative on HTML elements used in JSX

[karlhorky]

This PR is to add a failing test for the following situation:

// False Negative:
// The `div` parameter below is not recognized as an unused variable
function App(div) {
  return <div />
}

This also happens with other variables (eg. variables declared separately):

const div = 123;

export function App() {
  return <div />;
}

Incorrectly reported here: typescript-eslint/typescript-eslint#2985

[ljharb]

Thanks; there's some work that should likely be done here with our variableUtil.

[ljharb]

This test is now passing; it was fixed in #3070. Will merge to close.

[codecov-commenter]

Codecov Report

Merging #2916 (783d4cd) into master (a9f0b95) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2916   +/-   ##
=======================================
  Coverage   97.55%   97.55%           
=======================================
  Files         118      118           
  Lines        7855     7855           
  Branches     2801     2801           
=======================================
  Hits         7663     7663           
  Misses        192      192           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a9f0b95...783d4cd. Read the comment docs.

[karlhorky]

Great, thanks! Will look for this in the next patch (7.26.1) or minor (7.27.0)

[ljharb]

It’ll be a minor, since there’s already new rules landed.