refactor: move function isFragment to utils/jsx.js

Author: golopot

lib/rules/jsx-no-useless-fragment.js

@@ -5,6 +5,7 @@
 'use strict';
 
 const pragmaUtil = require('../util/pragma');
+const jsxUtil = require('../util/jsx');
 const docsUrl = require('../util/docsUrl');
 
 
@@ -28,36 +29,6 @@ module.exports = {
     const reactPragma = pragmaUtil.getFromContext(context);
     const fragmentPragma = pragmaUtil.getFragmentFromContext(context);
 
-    /**
-     * Test whether a JSXElement is a fragment
-     * @param {JSXElement} node
-     * @returns {boolean}
-     */
-    function isFragment(node) {
-      const name = node.openingElement.name;
-
-      // <Fragment>
-      if (
-        name.type === 'JSXIdentifier' &&
-        name.name === fragmentPragma
-      ) {
-        return true;
-      }
-
-      // <React.Fragment>
-      if (
-        name.type === 'JSXMemberExpression' &&
-        name.object.type === 'JSXIdentifier' &&
-        name.object.name === reactPragma &&
-        name.property.type === 'JSXIdentifier' &&
-        name.property.name === fragmentPragma
-      ) {
-        return true;
-      }
-
-      return false;
-    }
-
     /**
      * Test whether a node is an padding spaces trimmed by react runtime.
      * @param {ASTNode} node
@@ -158,7 +129,7 @@ module.exports = {
 
     return {
       JSXElement(node) {
-        if (isFragment(node)) {
+        if (jsxUtil.isFragment(node, reactPragma, fragmentPragma)) {
           checkNode(node);
         }
       },

lib/util/jsx.js

@@ -26,6 +26,35 @@ function isDOMComponent(node) {
   return COMPAT_TAG_REGEX.test(name);
 }
 
+/**
+ * Test whether a JSXElement is a fragment
+ * @param {JSXElement} node
+ * @param {string} reactPragma
+ * @param {string} fragmentPragma
+ * @returns {boolean}
+ */
+function isFragment(node, reactPragma, fragmentPragma) {
+  const name = node.openingElement.name;
+
+  // <Fragment>
+  if (name.type === 'JSXIdentifier' && name.name === fragmentPragma) {
+    return true;
+  }
+
+  // <React.Fragment>
+  if (
+    name.type === 'JSXMemberExpression' &&
+    name.object.type === 'JSXIdentifier' &&
+    name.object.name === reactPragma &&
+    name.property.type === 'JSXIdentifier' &&
+    name.property.name === fragmentPragma
+  ) {
+    return true;
+  }
+
+  return false;
+}
+
 /**
  * Checks if a node represents a JSX element or fragment.
  * @param {object} node - node to check.
@@ -37,5 +66,6 @@ function isJSX(node) {
 
 module.exports = {
   isDOMComponent,
+  isFragment,
   isJSX
 };