padrino-gen 0.16.0.pre4 → 0.16.0

data/test/test_plugin_generator.rb

@@ -1,6 +1,6 @@
-require File.expand_path(File.dirname(__FILE__) + '/helper')
+require_relative 'helper'
 
-describe "PluginGenerator" do
+describe 'PluginGenerator' do
   def setup
     @apptmp = "#{Dir.tmpdir}/padrino-tests/#{SecureRandom.hex}"
     `mkdir -p #{@apptmp}`
@@ -10,7 +10,7 @@ describe "PluginGenerator" do
     `rm -rf #{@apptmp}`
   end
 
-  describe "the plugin generator" do
+  describe 'the plugin generator' do
     it 'should respect --root option' do
       path = File.expand_path('fixtures/plugin_template.rb', __dir__)
       capture_io { generate(:project, 'sample_project', "--root=#{@apptmp}") }
@@ -19,21 +19,21 @@ describe "PluginGenerator" do
     end
   end
 
-  describe "the plugin destroy option" do
+  describe 'the plugin destroy option' do
     it 'should remove the plugin instance' do
       path = File.expand_path('fixtures/plugin_template.rb', __dir__)
       capture_io { generate(:project, 'sample_project', "--root=#{@apptmp}") }
       capture_io { generate(:plugin, path, "--root=#{@apptmp}/sample_project") }
       capture_io { generate(:plugin, path, "--root=#{@apptmp}/sample_project", '-d') }
       assert_no_file_exists("#{@apptmp}/sample_project/lib/hoptoad_initializer.rb")
-      assert_no_match_in_file(/enable :raise_errors/,"#{@apptmp}/sample_project/app/app.rb")
+      assert_no_match_in_file(/enable :raise_errors/, "#{@apptmp}/sample_project/app/app.rb")
       assert_no_match_in_file(/rack_hoptoad/, "#{@apptmp}/sample_project/Gemfile")
     end
   end
 
   describe 'the project generator with template' do
     it 'should invoke Padrino.bin_gen' do
-      expects_generated_project :name => 'sample_project', :test => :shoulda, :orm => :activerecord, :dev => true, :template => 'mongochist', :root => @apptmp
+      expects_generated_project name: 'sample_project', test: :shoulda, orm: :activerecord, dev: true, template: 'mongochist', root: @apptmp
       expects_generated :model, "post title:string body:text -r=#{@apptmp}/sample_project"
       expects_generated :controller, "posts get:index get:new post:new -r=#{@apptmp}/sample_project"
       expects_generated :migration, "AddEmailToUser email:string -r=#{@apptmp}/sample_project"
@@ -41,17 +41,16 @@ describe "PluginGenerator" do
       expects_generated :plugin, "carrierwave -r=#{@apptmp}/sample_project"
       File.stubs(:exist?).returns(true)
       expects_dependencies 'nokogiri'
-      expects_initializer :test, "# Example", :root => "#{@apptmp}/sample_project"
+      expects_initializer :test, '# Example', root: "#{@apptmp}/sample_project"
       expects_generated :app, "testapp -r=#{@apptmp}/sample_project"
       expects_generated :controller, "users get:index -r=#{@apptmp}/sample_project --app=testapp"
-      example_template_path = File.join(File.dirname(__FILE__), 'fixtures', 'example_template.rb')
+      example_template_path = File.join(__dir__, 'fixtures', 'example_template.rb')
       capture_io { generate(:project, 'sample_project', "--root=#{@apptmp}", "-p=#{example_template_path}", '> /dev/null') }
       File.unstub(:exist?)
     end
   end
 
-  describe "with resolving urls" do
-
+  describe 'with resolving urls' do
     it 'should resolve generic url properly' do
       template_file = 'http://www.example.com/test.rb'
       stub_request :get, template_file
@@ -61,7 +60,7 @@ describe "PluginGenerator" do
     end
 
     it 'should resolve gist url properly' do
-      stub_request(:get, "https://gist.github.com/357045").to_return(body: '<a href="/raw/357045/4356/blog_template.rb">raw</a>')
+      stub_request(:get, 'https://gist.github.com/357045').to_return(body: '<a href="/raw/357045/4356/blog_template.rb">raw</a>')
       template_file = 'https://gist.github.com/357045'
       resolved_path = 'https://gist.github.com/raw/357045/4356/blog_template.rb'
       project_gen = Padrino::Generators::Project.new(['sample_project'], ["-p=#{template_file}", "-r=#{@apptmp}"], {})
@@ -71,7 +70,7 @@ describe "PluginGenerator" do
 
     it 'should resolve official template' do
       template_file = 'sampleblog'
-      resolved_path = "https://raw.github.com/padrino/padrino-recipes/master/templates/sampleblog_template.rb"
+      resolved_path = 'https://raw.github.com/padrino/padrino-recipes/master/templates/sampleblog_template.rb'
       stub_request(:get, resolved_path).to_return(body: template_file)
       project_gen = Padrino::Generators::Project.new(['sample_project'], ["-p=#{template_file}", "-r=#{@apptmp}"], {})
       project_gen.expects(:apply).with(resolved_path).returns(true).once
@@ -87,70 +86,70 @@ describe "PluginGenerator" do
 
     it 'should resolve official plugin' do
       template_file = 'hoptoad'
-      resolved_path = "https://raw.github.com/padrino/padrino-recipes/master/plugins/hoptoad_plugin.rb"
+      resolved_path = 'https://raw.github.com/padrino/padrino-recipes/master/plugins/hoptoad_plugin.rb'
       stub_request(:get, resolved_path).to_return(body: template_file)
-      plugin_gen = Padrino::Generators::Plugin.new([ template_file], ["-r=#{@apptmp}/sample_project"],{})
+      plugin_gen = Padrino::Generators::Plugin.new([template_file], ["-r=#{@apptmp}/sample_project"], {})
       plugin_gen.expects(:in_app_root?).returns(true).once
       plugin_gen.expects(:apply).with(resolved_path).returns(true).once
       capture_io { plugin_gen.invoke_all }
     end
 
     it 'should print a warning if template cannot be found' do
-      template_file  = 'hwat'
-      resolved_path = "https://raw.github.com/padrino/padrino-recipes/master/plugins/hwat_plugin.rb"
+      template_file = 'hwat'
+      resolved_path = 'https://raw.github.com/padrino/padrino-recipes/master/plugins/hwat_plugin.rb'
       stub_request(:get, resolved_path).to_return(status: 404)
-      plugin_gen = Padrino::Generators::Plugin.new([ template_file], ["-r=#{@apptmp}/sample_project"],{})
+      plugin_gen = Padrino::Generators::Plugin.new([template_file], ["-r=#{@apptmp}/sample_project"], {})
       plugin_gen.expects(:in_app_root?).returns(true).once
       # Use regex to ignore trailing whitespace in message
-      plugin_gen.expects(:say).with{ |message, color| (message =~ /The template at #{resolved_path} could not be loaded:.*404.*/) && (color == :red) }.returns(true).once
+      plugin_gen.expects(:say).with { |message, color| (message =~ /The template at #{resolved_path} could not be loaded:.*404.*/) && (color == :red) }.returns(true).once
       capture_io { plugin_gen.invoke_all }
     end
   end
 
-  describe "with list option" do
+  describe 'with list option' do
     it 'should return a list of available plugins with no parameter' do
-      plugin_gen = Padrino::Generators::Plugin.new([], [],{})
+      plugin_gen = Padrino::Generators::Plugin.new([], [], {})
       plugin_gen.expects(:list_plugins).returns(true).once
       capture_io { plugin_gen.invoke_all }
     end
 
     it 'should return a list of available plugins with list option' do
-      plugin_gen = Padrino::Generators::Plugin.new(['some_plugin'], ["-l", "-r=#{@apptmp}/sample_project"],{})
+      plugin_gen = Padrino::Generators::Plugin.new(['some_plugin'], ['-l', "-r=#{@apptmp}/sample_project"], {})
       plugin_gen.expects(:list_plugins).returns(true).once
       capture_io { plugin_gen.invoke_all }
     end
   end
 
-  describe "with git commands" do
+  describe 'with git commands' do
     it 'should generate a repository correctly' do
       skip 'Change stubs here'
-      expects_generated_project :test => :rspec, :orm => :activerecord, :name => 'sample_git', :root => @apptmp.to_s
-      expects_git :init, :root => "#{@apptmp}/sample_git"
-      expects_git :add, :arguments => '.', :root => "#{@apptmp}/sample_git"
-      expects_git :commit, :arguments => 'hello', :root => "#{@apptmp}/sample_git"
-      git_template_path = File.join(File.dirname(__FILE__), 'fixtures', 'git_template.rb')
+      expects_generated_project test: :rspec, orm: :activerecord, name: 'sample_git', root: @apptmp.to_s
+      expects_git :init, root: "#{@apptmp}/sample_git"
+      expects_git :add, arguments: '.', root: "#{@apptmp}/sample_git"
+      expects_git :commit, arguments: 'hello', root: "#{@apptmp}/sample_git"
+      git_template_path = File.join(__dir__, 'fixtures', 'git_template.rb')
       capture_io { generate(:project, 'sample_git', "-p=#{git_template_path}", "-r=#{@apptmp}", '2>&1 /dev/null') }
     end
   end
 
-  describe "with rake invocations" do
+  describe 'with rake invocations' do
     it 'should Run rake task and list tasks' do
-      expects_generated_project :test => :shoulda, :orm => :activerecord, :name => 'sample_rake', :root => @apptmp.to_s
-      expects_rake "custom", :root => "#{@apptmp}/sample_rake"
-      rake_template_path = File.join(File.dirname(__FILE__), 'fixtures', 'rake_template.rb')
+      expects_generated_project test: :shoulda, orm: :activerecord, name: 'sample_rake', root: @apptmp.to_s
+      expects_rake 'custom', root: "#{@apptmp}/sample_rake"
+      rake_template_path = File.join(__dir__, 'fixtures', 'rake_template.rb')
       capture_io { generate(:project, 'sample_rake', "-p=#{rake_template_path}", "-r=#{@apptmp}", '> /dev/null') }
     end
   end
 
-  describe "with admin commands" do
+  describe 'with admin commands' do
     it 'should generate correctly an admin' do
-      expects_generated_project :test => :shoulda, :orm => :activerecord, :name => 'sample_admin', :root => @apptmp.to_s
+      expects_generated_project test: :shoulda, orm: :activerecord, name: 'sample_admin', root: @apptmp.to_s
       expects_generated :model, "post title:string body:text -r=#{@apptmp}/sample_admin"
-      expects_rake "ar:create", :root => "#{@apptmp}/sample_admin"
+      expects_rake 'ar:create', root: "#{@apptmp}/sample_admin"
       expects_generated :admin, "-r=#{@apptmp}/sample_admin"
-      expects_rake "ar:migrate", :root => "#{@apptmp}/sample_admin"
+      expects_rake 'ar:migrate', root: "#{@apptmp}/sample_admin"
       expects_generated :admin_page, "post -r=#{@apptmp}/sample_admin"
-      admin_template_path = File.join(File.dirname(__FILE__), 'fixtures', 'admin_template.rb')
+      admin_template_path = File.join(__dir__, 'fixtures', 'admin_template.rb')
       capture_io { generate(:project, 'sample_admin', "-p=#{admin_template_path}", "-r=#{@apptmp}", '> /dev/null') }
     end
   end