mirror of
https://github.com/rails/rails.git
synced 2022-11-09 12:12:34 -05:00
Cleanup tests for unused variables
This commit is contained in:
parent
51af1b1126
commit
647a9abb02
3 changed files with 2 additions and 4 deletions
|
@ -878,7 +878,7 @@ class DependenciesTest < ActiveSupport::TestCase
|
|||
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 @@ class InflectorTest < ActiveSupport::TestCase
|
|||
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 @@ module RailtiesTest
|
|||
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
|
||||
|
|
Loading…
Reference in a new issue