Permalink
Browse files

Cleanup tests for unused variables

  • Loading branch information...
1 parent 51af1b1 commit 647a9abb02ceffc7002b94d46a290621280e8d67 @vipulnsward vipulnsward committed Mar 11, 2013
View
2 activesupport/test/dependencies_test.rb
@@ -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'
View
2 activesupport/test/inflector_test.rb
@@ -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))
View
2 railties/test/railties/engine_test.rb
@@ -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 647a9ab

Please sign in to comment.