diff options
author | José Valim <jose.valim@gmail.com> | 2009-07-06 18:31:28 +0200 |
---|---|---|
committer | José Valim <jose.valim@gmail.com> | 2009-07-06 18:31:28 +0200 |
commit | 66d1b968d108787114372144e97f4f817847892b (patch) | |
tree | fde26589fbb2fe4abc79e10e09068b4e5d18972b /railties | |
parent | eeb6a0786a061b640bcd90a5d90bd0d0215fe809 (diff) | |
download | rails-66d1b968d108787114372144e97f4f817847892b.tar.gz rails-66d1b968d108787114372144e97f4f817847892b.tar.bz2 rails-66d1b968d108787114372144e97f4f817847892b.zip |
Make specs pass on Ruby 1.9.
Diffstat (limited to 'railties')
-rw-r--r-- | railties/lib/generators/actions.rb | 10 | ||||
-rw-r--r-- | railties/test/generators/actions_test.rb | 23 | ||||
-rw-r--r-- | railties/test/generators/generators_test_helper.rb | 8 |
3 files changed, 18 insertions, 23 deletions
diff --git a/railties/lib/generators/actions.rb b/railties/lib/generators/actions.rb index b6a3eb95e4..e0f765f201 100644 --- a/railties/lib/generators/actions.rb +++ b/railties/lib/generators/actions.rb @@ -214,7 +214,7 @@ module Rails def rake(command, options={}) log :rake, command env = options[:env] || 'development' - sudo = options[:sudo] && RUBY_PLATFORM !~ /win32|mswin/ ? 'sudo ' : '' + sudo = options[:sudo] && RUBY_PLATFORM !~ /mswin|mingw/ ? 'sudo ' : '' in_root { run("#{sudo}#{extify(:rake)} #{command} RAILS_ENV=#{env}", false) } end @@ -268,10 +268,14 @@ module Rails end end - # Add the ruby command extension to the given name. + # Add an extension to the given name based on the platform. # def extify(name) - "#{name}#{File.extname(Thor::Util.ruby_command)}" + if RUBY_PLATFORM =~ /mswin|mingw/ + "#{name}.bat" + else + name + end end end diff --git a/railties/test/generators/actions_test.rb b/railties/test/generators/actions_test.rb index 57ac3f0950..3313e92c31 100644 --- a/railties/test/generators/actions_test.rb +++ b/railties/test/generators/actions_test.rb @@ -98,7 +98,10 @@ class ActionsTest < GeneratorsTestCase '# This will be added' end - assert_file 'config/environment.rb', /# This will be added/ + assert_file 'config/environment.rb' do |content| + assert_match /# This will be added/, content + assert_no_match /# This wont be added/, content + end end def test_git_with_symbol_should_run_command_using_git_scm @@ -151,34 +154,16 @@ class ActionsTest < GeneratorsTestCase action :rake, 'log:clear', :sudo => true end - def test_rake_uses_ruby_extension - Thor::Util.expects(:ruby_command).returns('ruby.bat') - generator.expects(:run).once.with('rake.bat log:clear RAILS_ENV=development', false) - action :rake, 'log:clear' - end - def test_capify_should_run_the_capify_command generator.expects(:run).once.with('capify .', false) action :capify! end - def test_capify_uses_ruby_extension - Thor::Util.expects(:ruby_command).returns('ruby.bat') - generator.expects(:run).once.with('capify.bat .', false) - action :capify! - end - def test_freeze_should_freeze_rails_edge generator.expects(:run).once.with('rake rails:freeze:edge', false) action :freeze! end - def test_freeze_uses_ruby_extension - Thor::Util.expects(:ruby_command).returns('ruby.bat') - generator.expects(:run).once.with('rake.bat rails:freeze:edge', false) - action :freeze! - end - def test_route_should_add_data_to_the_routes_block_in_config_routes run_generator route_command = "map.route '/login', :controller => 'sessions', :action => 'new'" diff --git a/railties/test/generators/generators_test_helper.rb b/railties/test/generators/generators_test_helper.rb index 2ccedf814d..f6799eb6da 100644 --- a/railties/test/generators/generators_test_helper.rb +++ b/railties/test/generators/generators_test_helper.rb @@ -8,17 +8,23 @@ else RAILS_ROOT = fixtures end +$LOAD_PATH.unshift "#{File.dirname(__FILE__)}/../../../activerecord/lib" $LOAD_PATH.unshift "#{File.dirname(__FILE__)}/../../lib" require 'generators' class GeneratorsTestCase < Test::Unit::TestCase include FileUtils + def self.test_dir + @@test_dir = File.expand_path("#{File.dirname(__FILE__)}/../../") + end + def destination_root - @destination_root ||= File.expand_path("#{File.dirname(__FILE__)}/../fixtures/tmp") + @destination_root ||= File.join(self.class.test_dir, 'fixtures', 'tmp') end def setup + cd self.class.test_dir rm_rf(destination_root) mkdir_p(destination_root) end |