[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.