[Refactor] create getAncestors util, since context.getAncestors is deprecated

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

Author: MatiPl01

lib/rules/jsx-no-bind.js

@@ -11,6 +11,7 @@ const propName = require('jsx-ast-utils/propName');
 const docsUrl = require('../util/docsUrl');
 const jsxUtil = require('../util/jsx');
 const report = require('../util/report');
+const getAncestors = require('../util/eslint').getAncestors;
 
 // -----------------------------------------------------------------------------
 // Rule Definition
@@ -123,7 +124,7 @@ module.exports = {
     }
 
     function getBlockStatementAncestors(node) {
-      return context.getAncestors(node).filter(
+      return getAncestors(context, node).filter(
         (ancestor) => ancestor.type === 'BlockStatement'
       ).reverse();
     }

lib/rules/no-is-mounted.js

@@ -6,6 +6,7 @@
 'use strict';
 
 const docsUrl = require('../util/docsUrl');
+const getAncestors = require('../util/eslint').getAncestors;
 const report = require('../util/report');
 
 // ------------------------------------------------------------------------------
@@ -40,7 +41,7 @@ module.exports = {
         if (callee.object.type !== 'ThisExpression' || callee.property.name !== 'isMounted') {
           return;
         }
-        const ancestors = context.getAncestors(callee);
+        const ancestors = getAncestors(context, node);
         for (let i = 0, j = ancestors.length; i < j; i++) {
           if (ancestors[i].type === 'Property' || ancestors[i].type === 'MethodDefinition') {
             report(context, messages.noIsMounted, 'noIsMounted', {

lib/rules/require-render-return.js

@@ -12,6 +12,7 @@ const astUtil = require('../util/ast');
 const componentUtil = require('../util/componentUtil');
 const docsUrl = require('../util/docsUrl');
 const report = require('../util/report');
+const getAncestors = require('../util/eslint').getAncestors;
 
 // ------------------------------------------------------------------------------
 // Rule Definition
@@ -61,7 +62,7 @@ module.exports = {
 
     return {
       ReturnStatement(node) {
-        const ancestors = context.getAncestors(node).reverse();
+        const ancestors = getAncestors(context, node).reverse();
         let depth = 0;
         ancestors.forEach((ancestor) => {
           if (/Function(Expression|Declaration)$/.test(ancestor.type)) {

lib/util/eslint.js

@@ -4,6 +4,12 @@ function getSourceCode(context) {
   return context.getSourceCode ? context.getSourceCode() : context.sourceCode;
 }
 
+function getAncestors(context, node) {
+  const sourceCode = getSourceCode(context);
+  return sourceCode.getAncestors ? sourceCode.getAncestors(node) : context.getAncestors();
+}
+
 module.exports = {
+  getAncestors,
   getSourceCode,
 };

lib/util/makeNoMethodSetStateRule.js

@@ -9,6 +9,7 @@ const findLast = require('array.prototype.findlast');
 
 const docsUrl = require('./docsUrl');
 const report = require('./report');
+const getAncestors = require('./eslint').getAncestors;
 const testReactVersion = require('./version').testReactVersion;
 
 // ------------------------------------------------------------------------------
@@ -93,7 +94,7 @@ module.exports = function makeNoMethodSetStateRule(methodName, shouldCheckUnsafe
           ) {
             return;
           }
-          const ancestors = context.getAncestors(callee);
+          const ancestors = getAncestors(context, node);
           let depth = 0;
           findLast(ancestors, (ancestor) => {
           // ancestors.some((ancestor) => {