refactor and improve page generator

- change name: matestack_page => matestack:page
- simplify the genration of a page
- change structure of templates
- add tests for the options --namespace and --controller_action

Author: marcoroth

lib/generators/matestack/page/USAGE

@@ -0,0 +1,28 @@
+Description:
+  Stubs out a new matestack page and its route. Pass the page name and --app-name, both under_scored.
+
+  To place the page within a namespace, specify the namespace name under_scored and it
+  will get generated into 'app/matestack/pages/namespace/*'.
+
+  To create a custom controller#action to use in the created route, specify it as
+  under_scored_controller#under_scored_action
+
+  This generates a matestack page class in app/matestack/pages/ and adds a route to config/routes.rb
+
+Example 1:
+  `rails generate matestack:page example_page --app_name example_app`
+
+  This will create:
+    app/matestack/pages/example_app/example_page.rb
+
+  This will add to routes:
+    get 'example_app/example_page', to: 'example_app#example_page'
+
+Example 2:
+  `rails generate matestack:page example_page --app_name example_app --controller_action=static#second`
+
+  This will create:
+    app/matestack/pages/example_app/example_page.rb
+
+  This will add to routes:
+    get 'example_app/example_page', to: 'static#second'

lib/generators/matestack/page/page_generator.rb

@@ -0,0 +1,54 @@
+module Matestack
+  module Generators
+    class PageGenerator < Rails::Generators::NamedBase
+      source_root File.expand_path('templates', __dir__)
+
+      class_option :app_name, type: :string, required: true
+      class_option :namespace, type: :string
+      class_option :controller_action, type: :string
+      class_option :called_by_app_generator, type: :boolean, default: false
+
+      def app_name
+        options[:app_name]
+      end
+
+      def namespace
+        options[:namespace]
+      end
+
+      def controller_action
+        if options[:controller_action].nil?
+          "#{app_name}\##{file_name}"
+        else
+          options[:controller_action]
+        end
+      end
+
+      def create_page
+        template "app/matestack/pages/%app_name%/%namespace%/%file_name%.rb"
+
+        unless options[:called_by_app_generator]
+          if namespace
+            route %{get '#{app_name}/#{namespace}/#{file_name}', to: '#{controller_action}'}
+          else
+            route %{get '#{app_name}/#{file_name}', to: '#{controller_action}'}
+          end
+
+          puts "Page created! Make sure to add"
+          puts ""
+          puts "def #{file_name}"
+
+          if namespace
+            puts "  responder_for(Pages::#{app_name.camelize}::#{namespace.camelize}::#{file_name.camelize})"
+          else
+            puts "  responder_for(Pages::#{app_name.camelize}::#{file_name.camelize})"
+          end
+
+          puts "end"
+          puts ""
+          puts "to the desired controller!"
+        end
+      end
+    end
+  end
+end

lib/generators/matestack_page/templates/matestack_page.erb renamed to lib/generators/matestack/page/templates/app/matestack/pages/%app_name%/%namespace%/%file_name%.rb.tt

@@ -1,5 +1,4 @@
-class Pages::<% unless @namespace.nil? %><%= @namespace.camelize %>::<% end %><% unless @app_name.nil? %><%= @app_name.camelize %>::<% end %><%= class_name %> < Matestack::Ui::Page
-
+class Pages::<% unless namespace.nil? %><%= namespace.camelize %>::<% end %><% unless app_name.nil? %><%= app_name.camelize %>::<% end %><%= class_name %> < Matestack::Ui::Page
   # OPTIONAL: Prepare data for the response method
   # def prepare
   #   API calls and db queries go here
@@ -26,5 +25,4 @@ class Pages::<% unless @namespace.nil? %><%= @namespace.camelize %>::<% end %><%
   #     # your components go here
   #   }
   # end
-
 end

lib/generators/matestack_page/USAGE

@@ -0,0 +1,50 @@
+require 'generator_spec'
+require 'generators/matestack/page/page_generator'
+
+describe Matestack::Generators::PageGenerator, type: :generator do
+  let(:dummy) { File.expand_path(File.join(__FILE__, '..', '..', '..', '..', '..', 'dummy')) }
+  let(:dummy_copy) { File.expand_path(File.join(__FILE__, '..', '..', '..', '..', '..', 'dummy_copy')) }
+
+  before :each do
+    FileUtils.cp_r dummy, dummy_copy
+  end
+
+  after :each do
+    FileUtils.rm_rf dummy
+    FileUtils.cp_r dummy_copy, dummy
+    FileUtils.rm_rf dummy_copy
+  end
+
+  destination Rails.root
+
+  it "creates example page" do
+    run_generator %w(my_example_page --app_name my_app)
+
+    assert_file "app/matestack/pages/my_app/my_example_page.rb", /class Pages::MyApp::MyExamplePage < Matestack::Ui::Page\b/
+    assert_file "config/routes.rb", /my_app#my_example_page/
+  end
+
+  it 'create example page with namespace' do
+    run_generator %w(my_example_page --app_name my_app --namespace sample_namespace)
+
+    assert_file "app/matestack/pages/my_app/sample_namespace/my_example_page.rb", /class Pages::SampleNamespace::MyApp::MyExamplePage < Matestack::Ui::Page\b/
+    assert_file "config/routes.rb", %r{my_app/sample_namespace/my_example_page}
+    assert_file "config/routes.rb", /my_app#my_example_page/
+  end
+
+  it 'create example page with controller_action' do
+    run_generator %w(my_example_page --app_name my_app --controller_action custom#action)
+
+    assert_file "app/matestack/pages/my_app/my_example_page.rb", /class Pages::MyApp::MyExamplePage < Matestack::Ui::Page\b/
+    assert_file "config/routes.rb", /custom#action/
+  end
+
+  it 'create example page with namespace and controller_action' do
+    run_generator %w(my_example_page --app_name my_app --namespace sample_namespace --controller_action custom#action)
+
+    assert_file "app/matestack/pages/my_app/sample_namespace/my_example_page.rb", /class Pages::SampleNamespace::MyApp::MyExamplePage < Matestack::Ui::Page\b/
+    assert_file "config/routes.rb", %r{my_app/sample_namespace/my_example_page}
+    assert_file "config/routes.rb", /custom#action/
+  end
+
+end