refactor: Move isConstructor, isGetter, and isSetter to jsdocUt…

…ils to allow use in non-iterateJsdoc files
gajus · Jul 2, 2020 · d7a669b · d7a669b
1 parent ef35f91
commit d7a669b
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 4 deletions.
diff --git a/src/iterateJsdoc.js b/src/iterateJsdoc.js
@@ -189,16 +189,15 @@ const getUtils = (
   };
 
   utils.isConstructor = () => {
-    return node?.type === 'MethodDefinition' && node.kind === 'constructor' ||
-      node?.parent?.kind === 'constructor';
+    return jsdocUtils.isConstructor(node);
   };
 
   utils.isGetter = () => {
-    return node && node.parent.kind === 'get';
+    return jsdocUtils.isGetter(node);
   };
 
   utils.isSetter = () => {
-    return node && node.parent.kind === 'set';
+    return jsdocUtils.isSetter(node);
   };
 
   utils.getJsdocTagsDeep = (tagName) => {

diff --git a/src/jsdocUtils.js b/src/jsdocUtils.js
@@ -764,6 +764,19 @@ const getIndent = (sourceCode) => {
   return indent;
 };
 
+const isConstructor = (node) => {
+  return node?.type === 'MethodDefinition' && node.kind === 'constructor' ||
+    node?.parent?.kind === 'constructor';
+};
+
+const isGetter = (node) => {
+  return node && node.parent.kind === 'get';
+};
+
+const isSetter = (node) => {
+  return node && node.parent.kind === 'set';
+};
+
 export default {
   enforcedContexts,
   filterTags,
@@ -779,7 +792,10 @@ export default {
   hasReturnValue,
   hasTag,
   hasThrowValue,
+  isConstructor,
+  isGetter,
   isNamepathDefiningTag,
+  isSetter,
   isValidTag,
   parseClosureTemplateTag,
   tagMightHaveEitherTypeOrNamePosition,