Refactor generators

remove inheritance and add composition for a more flexible code
management

Author: AlexAvlonitis

bin/pico_api

@@ -1,12 +1,11 @@
-
 #!/usr/bin/env ruby
 
 require 'optparse'
 require 'pico_api'
 
 options = {}
 OptionParser.new do |opts|
-  opts.banner = "Usage: pico_api [options]"
+  opts.banner = 'Usage: pico_api [options]'
 
   opts.on('-n', '--new project_name') { |o| options[:project_name] = o }
 end.parse!

bin/setup

@@ -0,0 +1,48 @@
+# frozen_string_literal: true
+
+require 'fileutils'
+
+module PicoApi
+  module Generators
+    module Commands
+      class CreateBasicStructure
+        def self.call(project_name)
+          project_name_converter =
+            ::PicoApi::Generators::ProjectNameConverter.new(project_name)
+
+          new(project_name_converter).call
+        end
+
+        def initialize(project_name_converter)
+          @project_name_converter = project_name_converter
+        end
+
+        def call
+          create_bin_folder
+          create_lib_folder
+          create_config_folder
+        end
+
+        private
+
+        attr_reader :project_name_converter
+
+        def create_bin_folder
+          FileUtils.mkdir_p("#{snakecased_name}/bin")
+        end
+
+        def create_lib_folder
+          FileUtils.mkdir_p("#{snakecased_name}/lib/#{snakecased_name}")
+        end
+
+        def create_config_folder
+          FileUtils.mkdir_p("#{snakecased_name}/config")
+        end
+
+        def snakecased_name
+          project_name_converter.snakecased
+        end
+      end
+    end
+  end
+end

lib/pico_api/generators/commands/base.rb

@@ -3,16 +3,32 @@
 module PicoApi
   module Generators
     module Commands
-      class CreateConfigApplication < CreateTemplate
-        private
+      class CreateConfigApplication
+        def self.call(project_name)
+          file_creator = ::PicoApi::Generators::FileCreator.build(project_name)
 
-        def destination_path
-          '/config/application.rb'
+          new(file_creator).call
+        end
+
+        def initialize(file_creator)
+          @file_creator = file_creator
+        end
+
+        def call
+          file_creator.create(template_file_path, destination_path)
         end
 
-        def template_relative_path
+        private
+
+        attr_reader :file_creator
+
+        def template_file_path
           '/generators/templates/config/application.erb'
         end
+
+        def destination_path
+          '/config/application.rb'
+        end
       end
     end
   end

lib/pico_api/generators/commands/copy_template.rb

@@ -3,16 +3,32 @@
 module PicoApi
   module Generators
     module Commands
-      class CreateConfigBoot < CopyTemplate
-        private
+      class CreateConfigBoot
+        def self.call(project_name)
+          file_copier = ::PicoApi::Generators::FileCopier.build(project_name)
 
-        def destination_path
-          '/config/boot.rb'
+          new(file_copier).call
+        end
+
+        def initialize(file_copier)
+          @file_copier = file_copier
+        end
+
+        def call
+          file_copier.copy(template_file_path, destination_path)
         end
 
-        def template_relative_path
+        private
+
+        attr_reader :file_copier
+
+        def template_file_path
           '/generators/templates/config/boot.erb'
         end
+
+        def destination_path
+          '/config/boot.rb'
+        end
       end
     end
   end

lib/pico_api/generators/commands/create_base.rb

@@ -3,16 +3,32 @@
 module PicoApi
   module Generators
     module Commands
-      class CreateConfigConfiguration < CreateTemplate
-        private
+      class CreateConfigConfiguration
+        def self.call(project_name)
+          file_creator = ::PicoApi::Generators::FileCreator.build(project_name)
 
-        def destination_path
-          '/config/configuration.rb'
+          new(file_creator).call
+        end
+
+        def initialize(file_creator)
+          @file_creator = file_creator
+        end
+
+        def call
+          file_creator.create(template_file_path, destination_path)
         end
 
-        def template_relative_path
+        private
+
+        attr_reader :file_creator
+
+        def template_file_path
           '/generators/templates/config/configuration.erb'
         end
+
+        def destination_path
+          '/config/configuration.rb'
+        end
       end
     end
   end

lib/pico_api/generators/commands/create_basic_structure.rb

@@ -3,16 +3,32 @@
 module PicoApi
   module Generators
     module Commands
-      class CreateConfigDatabaseSetup < CopyTemplate
-        private
+      class CreateConfigDatabaseSetup
+        def self.call(project_name)
+          file_copier = ::PicoApi::Generators::FileCopier.build(project_name)
 
-        def destination_path
-          '/config/database_setup.rb'
+          new(file_copier).call
+        end
+
+        def initialize(file_copier)
+          @file_copier = file_copier
+        end
+
+        def call
+          file_copier.copy(template_file_path, destination_path)
         end
 
-        def template_relative_path
+        private
+
+        attr_reader :file_copier
+
+        def template_file_path
           '/generators/templates/config/database_setup.erb'
         end
+
+        def destination_path
+          '/config/database_setup.rb'
+        end
       end
     end
   end