[Refactor] create markVariableAsUsed util, context.markVariableAsUsed is deprecated

Co-authored-by: Mateusz Łopaciński <[email protected]>
Co-authored-by: Jordan Harband <[email protected]>

Author: MatiPl01

lib/rules/jsx-uses-react.js

@@ -7,6 +7,7 @@
 
 const pragmaUtil = require('../util/pragma');
 const docsUrl = require('../util/docsUrl');
+const markVariableAsUsed = require('../util/eslint').markVariableAsUsed;
 
 // ------------------------------------------------------------------------------
 // Rule Definition
@@ -29,8 +30,12 @@ module.exports = {
     const pragma = pragmaUtil.getFromContext(context);
     const fragment = pragmaUtil.getFragmentFromContext(context);
 
-    function handleOpeningElement() {
-      context.markVariableAsUsed(pragma);
+    /**
+     * @param {ASTNode} node
+     * @returns {void}
+     */
+    function handleOpeningElement(node) {
+      markVariableAsUsed(pragma, node, context);
     }
     // --------------------------------------------------------------------------
     // Public
@@ -39,8 +44,8 @@ module.exports = {
     return {
       JSXOpeningElement: handleOpeningElement,
       JSXOpeningFragment: handleOpeningElement,
-      JSXFragment() {
-        context.markVariableAsUsed(fragment);
+      JSXFragment(node) {
+        markVariableAsUsed(fragment, node, context);
       },
     };
   },

lib/rules/jsx-uses-vars.js

@@ -6,6 +6,7 @@
 'use strict';
 
 const docsUrl = require('../util/docsUrl');
+const markVariableAsUsed = require('../util/eslint').markVariableAsUsed;
 
 // ------------------------------------------------------------------------------
 // Rule Definition
@@ -53,7 +54,7 @@ module.exports = {
           return;
         }
 
-        context.markVariableAsUsed(name);
+        markVariableAsUsed(name, node, context);
       },
 
     };

lib/util/eslint.js

@@ -9,7 +9,15 @@ function getAncestors(context, node) {
   return sourceCode.getAncestors ? sourceCode.getAncestors(node) : context.getAncestors();
 }
 
+function markVariableAsUsed(name, node, context) {
+  const sourceCode = getSourceCode(context);
+  return sourceCode.markVariableAsUsed
+    ? sourceCode.markVariableAsUsed(name, node)
+    : context.markVariableAsUsed(name);
+}
+
 module.exports = {
   getAncestors,
   getSourceCode,
+  markVariableAsUsed,
 };