refactor(): extract include node modules to a sep function

nestjs · Jul 9, 2020 · d78bff5 · d78bff5
1 parent 480963a
commit d78bff5
Showing 1 changed file with 9 additions and 8 deletions.
diff --git a/lib/graphql-types.loader.ts b/lib/graphql-types.loader.ts
@@ -26,15 +26,9 @@ export class GraphQLTypesLoader {
   }
 
   private async getTypesFromPaths(paths: string | string[]): Promise<string[]> {
-    let includeNodeModules = false;
+    const includeNodeModules = this.includeNodeModules(paths);
 
-    if (util.isArray(paths)) {
-      includeNodeModules = paths.some((path) => path.includes('node_modules'));
-    } else {
-      includeNodeModules = paths.includes('node_modules');
-    }
-
-    paths = util.isArray(paths)
+    paths = Array.isArray(paths)
       ? paths.map((path) => normalize(path))
       : normalize(paths);
 
@@ -52,4 +46,11 @@ export class GraphQLTypesLoader {
 
     return Promise.all(fileContentsPromises);
   }
+
+  private includeNodeModules(pathOrPaths: string | string[]): boolean {
+    if (Array.isArray(pathOrPaths)) {
+      return pathOrPaths.some((path) => path.includes('node_modules'));
+    }
+    return pathOrPaths.includes('node_modules');
+  }
 }