Refactored migrator by adding a better separation of concerns between removeEmptyClassicComponentDirectories and execute

Author: ijlee2

lib/migrator.js

@@ -101,23 +101,24 @@ module.exports = class Migrator {
     });
   }
 
-  async removeEmptyClassicComponentDirectories(removeOnlyEmptyDirectories) {
+  async removeEmptyClassicComponentDirectories() {
     const templateFolderPath = path.join(this.projectRoot, 'app/templates/components');
+
+    const classFilePaths = this.findClassicComponentClasses();
+    const templatesWithLayoutName = getLayoutNameTemplates(classFilePaths);
+    const removeOnlyEmptyDirectories = Boolean(templatesWithLayoutName.length);
 
     await removeDirs(templateFolderPath, removeOnlyEmptyDirectories);
   }
 
   async execute() {
     let templateFilePaths = this.findClassicComponentTemplates();
-    let classFilePaths = this.findClassicComponentClasses();
-
     templateFilePaths = this.skipTemplatesUsedAsLayoutName(templateFilePaths);
     templateFilePaths = this.skipTemplatesUsedAsPartial(templateFilePaths);
 
     this.changeComponentStructureToFlat(templateFilePaths);
 
-    const templatesWithLayoutName = getLayoutNameTemplates(classFilePaths);
-    const removeOnlyEmptyDirectories = Boolean(templatesWithLayoutName.length);
-    await this.removeEmptyClassicComponentDirectories(removeOnlyEmptyDirectories);
+    // Clean up
+    await this.removeEmptyClassicComponentDirectories();
   }
 }