padrino-gen 0.16.0.pre4 → 0.16.0

data/padrino-gen.gemspec

@@ -1,29 +1,29 @@
 #!/usr/bin/env gem build
 # encoding: utf-8
 
-require File.expand_path("../../padrino-core/lib/padrino-core/version.rb", __FILE__)
+require File.expand_path('../padrino-core/lib/padrino-core/version.rb', __dir__)
 
 Gem::Specification.new do |s|
-  s.name = "padrino-gen"
-  s.authors = ["Padrino Team", "Nathan Esquenazi", "Davide D'Agostino", "Arthur Chiu"]
-  s.email = "padrinorb@gmail.com"
-  s.summary = "Generators for easily creating and building padrino applications"
-  s.homepage = "http://www.padrinorb.com"
-  s.description = "Generators for easily creating and building padrino applications from the console"
-  s.required_rubygems_version = ">= 1.3.6"
+  s.name = 'padrino-gen'
+  s.authors = ['Padrino Team', 'Nathan Esquenazi', "Davide D'Agostino", 'Arthur Chiu']
+  s.email = 'padrinorb@gmail.com'
+  s.summary = 'Generators for easily creating and building padrino applications'
+  s.homepage = 'http://www.padrinorb.com'
+  s.description = 'Generators for easily creating and building padrino applications from the console'
+  s.required_rubygems_version = '>= 1.3.6'
   s.version = Padrino.version
-  s.date = Time.now.strftime("%Y-%m-%d")
-  s.license = "MIT"
+  s.date = Time.now.strftime('%Y-%m-%d')
+  s.license = 'MIT'
 
-  s.extra_rdoc_files = Dir["*.rdoc"]
-  s.files         = `git ls-files`.split("\n") | Dir.glob("{lib}/**/*")
+  s.extra_rdoc_files = Dir['*.rdoc']
+  s.files         = `git ls-files`.split("\n") | Dir.glob('{lib}/**/*')
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
-  s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
-  s.require_paths = ["lib"]
-  s.rdoc_options  = ["--charset=UTF-8"]
+  s.executables   = `git ls-files -- bin/*`.split("\n").map { |f| File.basename(f) }
+  s.require_paths = ['lib']
+  s.rdoc_options  = ['--charset=UTF-8']
 
-  s.add_dependency("padrino-core", Padrino.version)
-  s.add_dependency("bundler", ">= 1.0", "< 3")
-  s.add_development_dependency("padrino-helpers", Padrino.version)
-  s.add_development_dependency("padrino-mailer", Padrino.version)
+  s.add_dependency('padrino-core', Padrino.version)
+  s.add_dependency('bundler', '>= 1.0', '< 3')
+  s.add_development_dependency('padrino-helpers', Padrino.version)
+  s.add_development_dependency('padrino-mailer', Padrino.version)
 end

data/test/fixtures/admin_template.rb

@@ -1,7 +1,7 @@
-project :test => :shoulda, :orm => :activerecord
+project test: :shoulda, orm: :activerecord
 
-generate :model, "post title:string body:text"
-rake "ar:create"
+generate :model, 'post title:string body:text'
+rake 'ar:create'
 generate :admin
-rake "ar:migrate"
-generate :admin_page, "post"
+rake 'ar:migrate'
+generate :admin_page, 'post'

data/test/fixtures/database_template.rb

@@ -2,34 +2,31 @@
 # You can use other adapters like:
 #
 #   ActiveRecord::Base.configurations = {
-#     :development => {
-#       :adapter   => 'mysql2',
-#       :encoding  => 'utf8',
-#       :reconnect => true,
-#       :database  => 'your_database',
-#       :pool      => 5,
-#       :username  => 'root',
-#       :password  => '',
-#       :host      => 'localhost',
-#       :socket    => '/tmp/mysql.sock'
+#     development: {
+#       adapter:   'mysql2',
+#       encoding:  'utf8',
+#       reconnect: true,
+#       database:  'your_database',
+#       pool:      5,
+#       username:  'root',
+#       password:  '',
+#       host:      'localhost',
+#       socket:    '/tmp/mysql.sock'
 #     }
 #   }
 #
 ActiveRecord::Base.configurations = {
-  :development => {
-    :adapter => 'sqlite3',
-    :database => Padrino.root('db', 'sample_project_development.db')
-
+  development: {
+    adapter: 'sqlite3',
+    database: Padrino.root('db', 'sample_project_development.db')
   },
-  :production => {
-    :adapter => 'sqlite3',
-    :database => Padrino.root('db', 'sample_project_production.db')
-
+  production: {
+    adapter: 'sqlite3',
+    database: Padrino.root('db', 'sample_project_production.db')
   },
-  :test => {
-    :adapter => 'sqlite3',
-    :database => Padrino.root('db', 'sample_project_test.db')
-
+  test: {
+    adapter: 'sqlite3',
+    database: Padrino.root('db', 'sample_project_test.db')
   }
 }
 

data/test/fixtures/example_template.rb

@@ -1,15 +1,15 @@
-project :test => :shoulda, :orm => :activerecord, :template => 'mongochist', :dev => true
+project test: :shoulda, orm: :activerecord, template: 'mongochist', dev: true
 
-generate :model, "post title:string body:text"
-generate :controller, "posts get:index get:new post:new"
-generate :migration, "AddEmailToUser email:string"
-generate :fake, "foo bar"
-generate :plugin, "carrierwave"
+generate :model, 'post title:string body:text'
+generate :controller, 'posts get:index get:new post:new'
+generate :migration, 'AddEmailToUser email:string'
+generate :fake, 'foo bar'
+generate :plugin, 'carrierwave'
 
 require_dependencies 'nokogiri'
 
-initializer :test, "# Example"
+initializer :test, '# Example'
 
 app :testapp do
-  generate :controller, "users get:index"
+  generate :controller, 'users get:index'
 end

data/test/fixtures/git_template.rb

@@ -1,4 +1,4 @@
-project :test => :rspec, :orm => :activerecord
+project test: :rspec, orm: :activerecord
 git :init
-git :add, "."
+git :add, '.'
 git :commit, "-m 'hello'"

data/test/fixtures/plugin_template.rb

@@ -8,6 +8,6 @@ HOPTOAD = <<-HOPTOAD
       #notifier.environment_filters << %w(MY_SECRET_KEY MY_SECRET_TOKEN)
     end
 HOPTOAD
-require_dependencies 'rack_hoptoad', :require => 'rack/hoptoad'
-initializer :hoptoad,HOPTOAD
-inject_into_file destination_root('/app/app.rb'),"    enable :raise_errors\n", :after => "configure do\n"
+require_dependencies 'rack_hoptoad', require: 'rack/hoptoad'
+initializer :hoptoad, HOPTOAD
+inject_into_file destination_root('/app/app.rb'), "    enable :raise_errors\n", after: "configure do\n"

data/test/fixtures/rake_template.rb

@@ -1,9 +1,9 @@
-project :test => :shoulda, :orm => :activerecord
-# create_file "lib/tasks/test.rake", <<-RAKE
+project test: :shoulda, orm: :activerecord
+# create_file 'lib/tasks/test.rake', <<-RAKE
 # task :custom do
-#   File.open('#{destination_root("/tmp/custom.txt")}', 'w') { |f|
+#   File.open('#{destination_root("/tmp/custom.txt")}', 'w') do |f|
 #     f.puts('Completed custom rake test')
-#   }
+#   end
 # end
 # RAKE
-rake "custom"
+rake 'custom'

data/test/helper.rb

@@ -22,7 +22,7 @@ class Minitest::Spec
 
   def stub_static_files
     # register fake URL to avoid downloading static files every time tests run
-    fake_uri_base = "https://raw.github.com/padrino/padrino-static/master/"
+    fake_uri_base = 'https://raw.github.com/padrino/padrino-static/master/'
     %w[
       js/dojo.js
       js/ext.js
@@ -47,37 +47,39 @@ class Minitest::Spec
     "Padrino::Generators::#{name.to_s.camelize}".constantize.start(params)
   end
 
-  # generate_with_parts(:app, "demo", "--root=/tmp/sample_project", :apps => "subapp")
+  # generate_with_parts(:app, 'demo', "--root=/tmp/sample_project", apps: 'subapp')
   # This method is intended to reproduce the real environment.
   def generate_with_parts(name, *params)
-    features, constants = [$", Object.constants].map{|x| Marshal.load(Marshal.dump(x)) }
+    features, constants = [$LOADED_FEATURES, Object.constants].map { |x| Marshal.load(Marshal.dump(x)) }
 
-    if root = params.find{|x| x.index(/-r=|--root=/) }
+    if (root = params.find { |x| x.index(/-r=|--root=/) })
       root = root.split(/=/)[1]
-      options, model_path = {}, File.expand_path(File.join(root, "/models/**/*.rb"))
+      options, model_path = {}, File.expand_path(File.join(root, '/models/**/*.rb'))
       options = params.pop if params.last.is_a?(Hash)
-      Dir[model_path].each{|path| require path }
+      Dir[model_path].sort.each { |path| require path }
+
       Array(options[:apps]).each do |app_name|
         path = File.expand_path(File.join(root, "/#{app_name}/app.rb"))
         require path if File.exist?(path)
-      end if options[:apps]
+      end
     end
+
     "Padrino::Generators::#{name.to_s.camelize}".constantize.start(params)
-    ($" - features).each{|x| $".delete(x) }
-    (Object.constants - constants).each{|constant| Object.instance_eval{ remove_const(constant) }}
+    ($LOADED_FEATURES - features).each { |x| $LOADED_FEATURES.delete(x) }
+    (Object.constants - constants).each { |constant| Object.instance_eval { remove_const(constant) } }
   end
 
   # expects_generated :model, "post title:string body:text"
-  def expects_generated(generator, params="")
+  def expects_generated(generator, params = '')
     Padrino.expects(:bin_gen).with(generator, *params.split(' ')).returns(true)
   end
 
-  # expects_generated_project :test => :shoulda, :orm => :activerecord, :dev => true
-  def expects_generated_project(options={})
+  # expects_generated_project test: :shoulda, orm: :activerecord, dev: true
+  def expects_generated_project(options = {})
     options = options.dup
     project_root = options.delete(:root)
     project_name = options.delete(:name)
-    components = options.sort_by{ |k, _v| k.to_s }.map{ |component, value| "--#{component}=#{value}" }
+    components = options.sort_by { |k, _v| k.to_s }.map { |component, value| "--#{component}=#{value}" }
     params = [project_name, *components].push("-r=#{project_root}")
     Padrino.expects(:bin_gen).with(*params.unshift('project')).returns(true)
   end
@@ -87,23 +89,23 @@ class Minitest::Spec
     instance = mock
     instance.expects(:invoke!).once
     include_text = "gem '#{name}'\n"
-    Thor::Actions::InjectIntoFile.expects(:new).with(anything,'Gemfile', include_text, anything).returns(instance)
+    Thor::Actions::InjectIntoFile.expects(:new).with(anything, 'Gemfile', include_text, anything).returns(instance)
   end
 
   # expects_initializer :test, "# Example"
-  def expects_initializer(name, body,options={})
-    #options.reverse_merge!(:root => "/tmp/sample_project")
-    path = File.join(options[:root],'config/initializers',"#{name}.rb")
+  def expects_initializer(name, body, options = {})
+    # options.reverse_merge!(root: '/tmp/sample_project')
+    path = File.join(options[:root], 'config/initializers', "#{name}.rb")
     instance = mock
     instance.expects(:invoke!).at_least_once
     include_text = "    register #{name.to_s.camelize}Initializer\n"
-    Thor::Actions::InjectIntoFile.expects(:new).with(anything,anything, include_text, anything).returns(instance)
+    Thor::Actions::InjectIntoFile.expects(:new).with(anything, anything, include_text, anything).returns(instance)
     Thor::Actions::CreateFile.expects(:new).with(anything, path, kind_of(Proc), anything).returns(instance)
   end
 
   # expects_rake "custom"
-  def expects_rake(command,options={})
-    #options.reverse_merge!(:root => '/tmp')
-    Padrino.expects(:bin).with("rake", command, "-c=#{options[:root]}").returns(true)
+  def expects_rake(command, options = {})
+    # options.reverse_merge!(root: '/tmp')
+    Padrino.expects(:bin).with('rake', command, "-c=#{options[:root]}").returns(true)
   end
 end

data/test/test_app_generator.rb

@@ -1,6 +1,6 @@
-require File.expand_path(File.dirname(__FILE__) + '/helper')
+require_relative 'helper'
 
-describe "AppGenerator" do
+describe 'AppGenerator' do
   def setup
     @apptmp = "#{Dir.tmpdir}/padrino-tests/#{SecureRandom.hex}"
     `mkdir -p #{@apptmp}`
@@ -12,7 +12,7 @@ describe "AppGenerator" do
 
   describe 'the app generator' do
     it 'should fail outside app root' do
-      out, err = capture_io { generate(:app, 'demo_root', "-r=#{@apptmp}") }
+      out, = capture_io { generate(:app, 'demo_root', "-r=#{@apptmp}") }
       assert_match(/not at the root/, out)
       assert_no_file_exists("#{@apptmp}/demo_root")
     end
@@ -28,7 +28,7 @@ describe "AppGenerator" do
       assert_dir_exists("#{@apptmp}/sample_project/demo/views")
       assert_dir_exists("#{@apptmp}/sample_project/demo/views/layouts")
       assert_dir_exists("#{@apptmp}/sample_project/public/demo")
-      assert_match_in_file("Padrino.mount('SampleProject::Demo', :app_file => Padrino.root('demo/app.rb')).to('/demo')", "#{@apptmp}/sample_project/config/apps.rb")
+      assert_match_in_file("Padrino.mount('SampleProject::Demo', app_file: Padrino.root('demo/app.rb')).to('/demo')", "#{@apptmp}/sample_project/config/apps.rb")
       assert_match_in_file('module SampleProject', "#{@apptmp}/sample_project/demo/app.rb")
       assert_match_in_file('class Demo < Padrino::Application', "#{@apptmp}/sample_project/demo/app.rb")
       assert_match_in_file(/Padrino.configure_apps do/, "#{@apptmp}/sample_project/config/apps.rb")
@@ -46,7 +46,7 @@ describe "AppGenerator" do
       assert_dir_exists("#{@apptmp}/sample_project/demo_app/views")
       assert_dir_exists("#{@apptmp}/sample_project/demo_app/views/layouts")
       assert_dir_exists("#{@apptmp}/sample_project/public/demo_app")
-      assert_match_in_file("Padrino.mount('SampleProject::DemoApp', :app_file => Padrino.root('demo_app/app.rb')).to('/demo_app')", "#{@apptmp}/sample_project/config/apps.rb")
+      assert_match_in_file("Padrino.mount('SampleProject::DemoApp', app_file: Padrino.root('demo_app/app.rb')).to('/demo_app')", "#{@apptmp}/sample_project/config/apps.rb")
       assert_match_in_file('module SampleProject', "#{@apptmp}/sample_project/demo_app/app.rb")
       assert_match_in_file('class DemoApp < Padrino::Application', "#{@apptmp}/sample_project/demo_app/app.rb")
       assert_match_in_file(/Padrino.configure_apps do/, "#{@apptmp}/sample_project/config/apps.rb")
@@ -55,14 +55,14 @@ describe "AppGenerator" do
 
     it 'should generate tiny app skeleton' do
       capture_io { generate(:project, 'sample_project', "--root=#{@apptmp}") }
-      capture_io { generate(:app, 'demo','--tiny',"--root=#{@apptmp}/sample_project") }
+      capture_io { generate(:app, 'demo', '--tiny', "--root=#{@apptmp}/sample_project") }
       assert_dir_exists("#{@apptmp}/sample_project")
       assert_dir_exists("#{@apptmp}/sample_project/demo")
       assert_file_exists("#{@apptmp}/sample_project/demo/helpers.rb")
       assert_file_exists("#{@apptmp}/sample_project/demo/controllers.rb")
       assert_file_exists("#{@apptmp}/sample_project/demo/mailers.rb")
       assert_dir_exists("#{@apptmp}/sample_project/public/demo")
-      assert_match_in_file(/:notifier/,"#{@apptmp}/sample_project/demo/mailers.rb")
+      assert_match_in_file(/:notifier/, "#{@apptmp}/sample_project/demo/mailers.rb")
       assert_match_in_file(/module Helper/, "#{@apptmp}/sample_project/demo/helpers.rb")
       assert_match_in_file(/helpers Helper/, "#{@apptmp}/sample_project/demo/helpers.rb")
       assert_no_file_exists("#{@apptmp}/sample_project/demo/helpers")
@@ -89,19 +89,19 @@ describe "AppGenerator" do
     it 'should destroys itself' do
       capture_io { generate(:project, 'sample_project', "--root=#{@apptmp}") }
       capture_io { generate(:app, 'demo', "--root=#{@apptmp}/sample_project") }
-      out, err = capture_io { generate_with_parts(:app, 'demo', "--root=#{@apptmp}/sample_project", '-d', :apps => "demo") }
+      out, = capture_io { generate_with_parts(:app, 'demo', "--root=#{@apptmp}/sample_project", '-d', apps: 'demo') }
       refute_match(/has been mounted/, out)
       assert_no_dir_exists("#{@apptmp}/sample_project/public/demo")
       assert_no_file_exists("#{@apptmp}/sample_project/demo/app.rb")
       assert_no_dir_exists("#{@apptmp}/sample_project/demo/controllers")
       assert_no_dir_exists("#{@apptmp}/sample_project/demo/helpers")
       assert_no_dir_exists("#{@apptmp}/sample_project/demo/views")
-      assert_no_match_in_file(/Padrino\.mount\("Demo"\).to\("\/demo"\)/,"#{@apptmp}/sample_project/config/apps.rb")
+      assert_no_match_in_file(%r{Padrino\.mount\("Demo"\).to\("/demo"\)}, "#{@apptmp}/sample_project/config/apps.rb")
     end
 
     it 'should abort if app name already exists' do
       capture_io { generate(:project, 'sample_project', "--root=#{@apptmp}") }
-      out, err = capture_io { generate(:app, 'kernel', "--root=#{@apptmp}/sample_project") }
+      out, = capture_io { generate(:app, 'kernel', "--root=#{@apptmp}/sample_project") }
       assert_match(/Kernel already exists/, out)
       assert_no_dir_exists("#{@apptmp}/sample_project/public/kernel")
       assert_no_dir_exists("#{@apptmp}/sample_project/kernel/controllers")
@@ -112,7 +112,7 @@ describe "AppGenerator" do
     it 'should abort if app name already exists in root' do
       capture_io { generate(:project, 'sample_project', "--root=#{@apptmp}") }
       capture_io { generate(:app, 'subapp', "--root=#{@apptmp}/sample_project") }
-      out, err = capture_io { generate_with_parts(:app, 'subapp', "--root=#{@apptmp}/sample_project", :apps => "subapp") }
+      out, = capture_io { generate_with_parts(:app, 'subapp', "--root=#{@apptmp}/sample_project", apps: 'subapp') }
       assert_dir_exists("#{@apptmp}/sample_project/public/subapp")
       assert_dir_exists("#{@apptmp}/sample_project/subapp/controllers")
       assert_dir_exists("#{@apptmp}/sample_project/subapp/helpers")
@@ -122,7 +122,7 @@ describe "AppGenerator" do
 
     it 'should generate app files if :force option is specified' do
       capture_io { generate(:project, 'sample_project', "--root=#{@apptmp}") }
-      out, err = capture_io { generate(:app, 'kernel', "--root=#{@apptmp}/sample_project", "--force") }
+      capture_io { generate(:app, 'kernel', "--root=#{@apptmp}/sample_project", '--force') }
       assert_dir_exists("#{@apptmp}/sample_project/public/kernel")
       assert_dir_exists("#{@apptmp}/sample_project/kernel/controllers")
       assert_dir_exists("#{@apptmp}/sample_project/kernel/helpers")
@@ -131,7 +131,7 @@ describe "AppGenerator" do
 
     it 'should allow to pass upcased name as the app name' do
       capture_io { generate(:project, 'sample_project', "--root=#{@apptmp}") }
-      out, err = capture_io { generate(:app, 'API', "--root=#{@apptmp}/sample_project", "--force") }
+      capture_io { generate(:app, 'API', "--root=#{@apptmp}/sample_project", '--force') }
       assert_dir_exists("#{@apptmp}/sample_project/public/api")
       assert_dir_exists("#{@apptmp}/sample_project/api/controllers")
       assert_dir_exists("#{@apptmp}/sample_project/api/helpers")

data/test/test_cli.rb

@@ -1,7 +1,7 @@
-require File.expand_path(File.dirname(__FILE__) + '/helper')
+require_relative 'helper'
 require 'padrino-gen/generators/cli'
 
-describe "Cli" do
+describe 'Cli' do
   def setup
     @apptmp = "#{Dir.tmpdir}/padrino-tests/#{SecureRandom.hex}"
     `mkdir -p #{@apptmp}`
@@ -12,16 +12,15 @@ describe "Cli" do
   end
 
   describe 'the cli' do
-
     it 'should fail without arguments' do
-      out, err = capture_io { generate(:cli) }
+      out, = capture_io { generate(:cli) }
       assert_match(/Please specify generator to use/, out)
     end
 
     it 'should work correctly if we have a project' do
       capture_io { generate(:project, 'sample_project', "--root=#{@apptmp}") }
       capture_io { generate(:cli, "--root=#{@apptmp}/sample_project") }
-      skip "Make a great asserition"
+      skip 'Make a great asserition'
     end
   end
 end

data/test/test_component_generator.rb

@@ -1,7 +1,7 @@
-require File.expand_path(File.dirname(__FILE__) + '/helper')
-require "rb-readline"
+require_relative 'helper'
+require 'rb-readline'
 
-describe "ComponentGenerator" do
+describe 'ComponentGenerator' do
   def setup
     @apptmp = "#{Dir.tmpdir}/padrino-tests/#{SecureRandom.hex}"
     `mkdir -p #{@apptmp}`
@@ -14,20 +14,20 @@ describe "ComponentGenerator" do
 
   describe 'the controller generator' do
     it 'should fail outside app root' do
-      out, err = capture_io { generate(:component, 'demo', "-r=#{@apptmp}") }
+      out, = capture_io { generate(:component, 'demo', "-r=#{@apptmp}") }
       assert_match(/not at the root/, out)
     end
   end
 
-  describe "add components" do
+  describe 'add components' do
     it 'should properly generate default' do
       capture_io { generate(:project, 'sample_project', "--root=#{@apptmp}") }
-      out, err = capture_io { generate(:component, '--orm=activerecord', "-r=#{@apptmp}/sample_project") }
+      out, = capture_io { generate(:component, '--orm=activerecord', "-r=#{@apptmp}/sample_project") }
       assert_match(/applying.*?activerecord.*?orm/, out)
-      assert_match_in_file(/gem 'activerecord', '>= 3.1', :require => 'active_record'/, "#{@apptmp}/sample_project/Gemfile")
+      assert_match_in_file(/gem 'activerecord', '>= 3.1', require: 'active_record'/, "#{@apptmp}/sample_project/Gemfile")
       assert_match_in_file(/gem 'sqlite3'/, "#{@apptmp}/sample_project/Gemfile")
       refute_match(/Switch renderer to/, out)
-      database_template_path = File.join(File.dirname(__FILE__), 'fixtures', 'database_template.rb')
+      database_template_path = File.join(__dir__, 'fixtures', 'database_template.rb')
       assert FileUtils.compare_file("#{@apptmp}/sample_project/config/database.rb", database_template_path)
       components_chosen = YAML.load_file("#{@apptmp}/sample_project/.components")
       assert_equal 'activerecord', components_chosen[:orm]
@@ -35,7 +35,7 @@ describe "ComponentGenerator" do
 
     it 'should properly generate with adapter' do
       capture_io { generate(:project, 'sample_project', "--root=#{@apptmp}") }
-      out, err = capture_io { generate(:component, '--orm=sequel', '--adapter=postgres', "-r=#{@apptmp}/sample_project") }
+      out, = capture_io { generate(:component, '--orm=sequel', '--adapter=postgres', "-r=#{@apptmp}/sample_project") }
       assert_match_in_file(/gem 'pg'/, "#{@apptmp}/sample_project/Gemfile")
       refute_match(/Switch renderer to/, out)
       components_chosen = YAML.load_file("#{@apptmp}/sample_project/.components")
@@ -44,31 +44,31 @@ describe "ComponentGenerator" do
 
     it 'should enable @app_name value' do
       capture_io { generate(:project, 'sample_project', "--root=#{@apptmp}") }
-      out, err = capture_io { generate_with_parts(:component, '--test=cucumber', "-r=#{@apptmp}/sample_project", :apps => "app") }
+      capture_io { generate_with_parts(:component, '--test=cucumber', "-r=#{@apptmp}/sample_project", apps: 'app') }
       assert_match_in_file(/SampleProject::App\.tap \{ \|app\|  \}/, "#{@apptmp}/sample_project/features/support/env.rb")
     end
 
     it 'should generate component in specified app' do
       capture_io { generate(:project, 'sample_project', "--root=#{@apptmp}") }
       capture_io { generate(:app, 'compo', "-r=#{@apptmp}/sample_project") }
-      out, err = capture_io { generate_with_parts(:component, '--test=cucumber', "--app=compo", "-r=#{@apptmp}/sample_project", :apps => "compo") }
+      capture_io { generate_with_parts(:component, '--test=cucumber', '--app=compo', "-r=#{@apptmp}/sample_project", apps: 'compo') }
       assert_match_in_file(/SampleProject::Compo\.tap \{ \|app\|  \}/, "#{@apptmp}/sample_project/features/support/env.rb")
     end
 
     it 'should not generate component in specified app if the app does not exist' do
       capture_io { generate(:project, 'sample_project', "--root=#{@apptmp}") }
-      out, err = capture_io { generate_with_parts(:component, '--test=cucumber', "--app=compo", "-r=#{@apptmp}/sample_project", :apps => "compo") }
+      out, = capture_io { generate_with_parts(:component, '--test=cucumber', '--app=compo', "-r=#{@apptmp}/sample_project", apps: 'compo') }
       assert_match(/SampleProject::Compo does not exist./, out)
       assert_no_file_exists("#{@apptmp}/sample_project/features")
     end
   end
 
-  describe "specified of same the component" do
+  describe 'specified of same the component' do
     it 'should does not change' do
       capture_io { generate(:project, 'sample_project', '--script=jquery', "--root=#{@apptmp}") }
       components_chosen = YAML.load_file("#{@apptmp}/sample_project/.components")
       assert_equal 'jquery', components_chosen[:script]
-      out, err = capture_io { generate(:component, '--script=jquery', "-r=#{@apptmp}/sample_project") }
+      out, = capture_io { generate(:component, '--script=jquery', "-r=#{@apptmp}/sample_project") }
       assert_match(/applying.*?jquery.*?script/, out)
       refute_match(/Switch renderer to/, out)
       components_chosen = YAML.load_file("#{@apptmp}/sample_project/.components")
@@ -76,13 +76,13 @@ describe "ComponentGenerator" do
     end
   end
 
-  describe "component changes" do
+  describe 'component changes' do
     it 'should when allow changes, will be applied' do
       capture_io { generate(:project, 'sample_project', '--renderer=slim', "--root=#{@apptmp}") }
       components_chosen = YAML.load_file("#{@apptmp}/sample_project/.components")
       assert_equal 'slim', components_chosen[:renderer]
       Readline.stubs(:readline).returns('yes').once
-      out, err = capture_io { generate(:component, '--renderer=haml', "-r=#{@apptmp}/sample_project") }
+      out, = capture_io { generate(:component, '--renderer=haml', "-r=#{@apptmp}/sample_project") }
       assert_match(/applying.*?haml.*?renderer/, out)
       components_chosen = YAML.load_file("#{@apptmp}/sample_project/.components")
       assert_equal 'haml', components_chosen[:renderer]
@@ -93,7 +93,7 @@ describe "ComponentGenerator" do
       components_chosen = YAML.load_file("#{@apptmp}/sample_project/.components")
       assert_equal 'slim', components_chosen[:renderer]
       Readline.stubs(:readline).returns('no').once
-      out, err = capture_io { generate(:component, '--renderer=haml', "-r=#{@apptmp}/sample_project") }
+      out, = capture_io { generate(:component, '--renderer=haml', "-r=#{@apptmp}/sample_project") }
       refute_match(/applying.*?haml.*?renderer/, out)
       components_chosen = YAML.load_file("#{@apptmp}/sample_project/.components")
       assert_equal 'slim', components_chosen[:renderer]