Permalink
Browse files

Merge pull request #9657 from vipulnsward/cleanup_tests

Cleanup tests for unused variables
  • Loading branch information...
rafaelfranca committed Mar 11, 2013
2 parents 46c3a57 + 647a9ab commit 8fc391572c6e8342088bc0b8ee78c1dff5abb04d
@@ -878,7 +878,7 @@ def test_autoload_doesnt_shadow_error_when_mechanism_not_set_to_load
def test_autoload_doesnt_shadow_name_error
with_autoloading_fixtures do
Object.send(:remove_const, :RaisesNameError) if defined?(::RaisesNameError)
- 2.times do |i|
+ 2.times do
begin
::RaisesNameError::FooBarBaz.object_id
flunk 'should have raised NameError when autoloaded file referenced FooBarBaz'
@@ -61,9 +61,7 @@ def test_uncountable_word_is_not_greedy
assert_equal(plural, ActiveSupport::Inflector.pluralize(plural))
assert_equal(plural.capitalize, ActiveSupport::Inflector.pluralize(plural.capitalize))
end
- end
- SingularToPlural.each do |singular, plural|
define_method "test_singularize_singular_#{singular}" do
assert_equal(singular, ActiveSupport::Inflector.singularize(singular))
assert_equal(singular.capitalize, ActiveSupport::Inflector.singularize(singular.capitalize))
@@ -105,7 +105,7 @@ def up
assert_not_nil bukkits_migration_order, "Expected migration to be skipped"
migrations_count = Dir["#{app_path}/db/migrate/*.rb"].length
- output = `bundle exec rake railties:install:migrations`
+ `bundle exec rake railties:install:migrations`
assert_equal migrations_count, Dir["#{app_path}/db/migrate/*.rb"].length
end

0 comments on commit 8fc3915

Please sign in to comment.