[Refactor] jsx-key: use isJSX util function instead of shouldCheckNode

Author: yalperg

lib/rules/jsx-key.js

@@ -13,6 +13,7 @@ const pragmaUtil = require('../util/pragma');
 const report = require('../util/report');
 const astUtil = require('../util/ast');
 const getText = require('../util/eslint').getText;
+const isJSX = require('../util/jsx').isJSX;
 
 // ------------------------------------------------------------------------------
 // Rule Definition
@@ -33,8 +34,6 @@ const messages = {
   nonUniqueKeys: '`key` prop must be unique',
 };
 
-const shouldCheckNode = (n) => n && (n.type === 'JSXElement' || n.type === 'JSXFragment');
-
 /** @type {import('eslint').Rule.RuleModule} */
 module.exports = {
   meta: {
@@ -151,10 +150,10 @@ module.exports = {
               const argument = returnStatement.argument;
 
               if (argument.type === 'ConditionalExpression') {
-                if (shouldCheckNode(argument.consequent)) {
+                if (isJSX(argument.consequent)) {
                   checkIteratorElement(argument.consequent);
                 }
-                if (shouldCheckNode(argument.alternate)) {
+                if (isJSX(argument.alternate)) {
                   checkIteratorElement(argument.alternate);
                 }
               } else {
@@ -172,17 +171,17 @@ module.exports = {
      */
     function checkArrowFunctionWithJSX(node) {
       const isArrFn = node && node.type === 'ArrowFunctionExpression';
-      if (isArrFn && shouldCheckNode(node.body)) {
+      if (isArrFn && isJSX(node.body)) {
         checkIteratorElement(node.body);
       }
       if (node.body.type === 'ConditionalExpression') {
-        if (shouldCheckNode(node.body.consequent)) {
+        if (isJSX(node.body.consequent)) {
           checkIteratorElement(node.body.consequent);
         }
-        if (shouldCheckNode(node.body.alternate)) {
+        if (isJSX(node.body.alternate)) {
           checkIteratorElement(node.body.alternate);
         }
-      } else if (node.body.type === 'LogicalExpression' && shouldCheckNode(node.body.right)) {
+      } else if (node.body.type === 'LogicalExpression' && isJSX(node.body.right)) {
         checkIteratorElement(node.body.right);
       }
     }