refactor: avoid duplicate implementation for isJSXText

jsx-eslint · Jul 12, 2019 · 7425244 · 7425244
1 parent 01eb97e
commit 7425244
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/lib/rules/jsx-no-useless-fragment.js b/lib/rules/jsx-no-useless-fragment.js
@@ -7,6 +7,9 @@ const pragmaUtil = require('../util/pragma');
 const jsxUtil = require('../util/jsx');
 const docsUrl = require('../util/docsUrl');
 
+function isJSXText(node) {
+  return !!node && (node.type === 'JSXText' || node.type === 'Literal');
+}
 
 module.exports = {
   meta: {
@@ -34,7 +37,7 @@ module.exports = {
      * @returns {boolean}
      */
     function isPaddingSpaces(node) {
-      return (node.type === 'JSXText' || node.type === 'Literal')
+      return isJSXText(node)
         && /^\s*$/.test(node.raw)
         && node.raw.includes('\n');
     }
@@ -65,10 +68,6 @@ module.exports = {
         && /^[a-z]+$/.test(node.parent.openingElement.name.name);
     }
 
-    function isJSXText(node) {
-      return !!node && (node.type === 'JSXText' || node.type === 'Literal');
-    }
-
     /**
      * Avoid fixing case like:
      * ```jsx