mirror of
https://github.com/rails/rails.git
synced 2022-11-09 12:12:34 -05:00
Merge pull request #12008 from wangjohn/skipping_deprecated_migration_tests
Skip tests using deprecated +proper_table_name+ method.
This commit is contained in:
commit
539180cf8e
1 changed files with 21 additions and 7 deletions
|
@ -322,11 +322,19 @@ class MigrationTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_proper_table_name_on_migrator
|
||||
assert_deprecated do
|
||||
assert_equal "table", ActiveRecord::Migrator.proper_table_name('table')
|
||||
end
|
||||
assert_deprecated do
|
||||
assert_equal "table", ActiveRecord::Migrator.proper_table_name(:table)
|
||||
end
|
||||
assert_deprecated do
|
||||
assert_equal "reminders", ActiveRecord::Migrator.proper_table_name(Reminder)
|
||||
end
|
||||
Reminder.reset_table_name
|
||||
assert_deprecated do
|
||||
assert_equal Reminder.table_name, ActiveRecord::Migrator.proper_table_name(Reminder)
|
||||
end
|
||||
|
||||
# Use the model's own prefix/suffix if a model is given
|
||||
ActiveRecord::Base.table_name_prefix = "ARprefix_"
|
||||
|
@ -334,7 +342,9 @@ class MigrationTest < ActiveRecord::TestCase
|
|||
Reminder.table_name_prefix = 'prefix_'
|
||||
Reminder.table_name_suffix = '_suffix'
|
||||
Reminder.reset_table_name
|
||||
assert_deprecated do
|
||||
assert_equal "prefix_reminders_suffix", ActiveRecord::Migrator.proper_table_name(Reminder)
|
||||
end
|
||||
Reminder.table_name_prefix = ''
|
||||
Reminder.table_name_suffix = ''
|
||||
Reminder.reset_table_name
|
||||
|
@ -343,9 +353,13 @@ class MigrationTest < ActiveRecord::TestCase
|
|||
ActiveRecord::Base.table_name_prefix = "prefix_"
|
||||
ActiveRecord::Base.table_name_suffix = "_suffix"
|
||||
Reminder.reset_table_name
|
||||
assert_deprecated do
|
||||
assert_equal "prefix_table_suffix", ActiveRecord::Migrator.proper_table_name('table')
|
||||
end
|
||||
assert_deprecated do
|
||||
assert_equal "prefix_table_suffix", ActiveRecord::Migrator.proper_table_name(:table)
|
||||
end
|
||||
end
|
||||
|
||||
def test_proper_table_name_on_migration
|
||||
migration = ActiveRecord::Migration.new
|
||||
|
|
Loading…
Reference in a new issue