[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
@@ -159,18 +160,17 @@ module.exports = {
      */
     function checkArrowFunctionWithJSX(node) {
       const isArrFn = node && node.type === 'ArrowFunctionExpression';
-      const shouldCheckNode = (n) => n && (n.type === 'JSXElement' || n.type === 'JSXFragment');
-      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);
       }
     }