From f43d97a860de396d4f087c122d8e4f714aaed42c Mon Sep 17 00:00:00 2001 From: Marcel Molina Date: Wed, 28 Sep 2005 17:50:28 +0000 Subject: Add failing test that shows that AR::Migration.remove_column fails silently with SQLite when passed arguments as symbols. git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@2396 5ecf4fe2-1ee6-0310-87b1-e25e094e27de --- activerecord/test/migration_test.rb | 20 +++++++++++++++++--- 1 file changed, 17 insertions(+), 3 deletions(-) diff --git a/activerecord/test/migration_test.rb b/activerecord/test/migration_test.rb index 5a86a678cd..cbcaff8cc4 100644 --- a/activerecord/test/migration_test.rb +++ b/activerecord/test/migration_test.rb @@ -132,15 +132,29 @@ if ActiveRecord::Base.connection.supports_migrations? assert_equal TrueClass, bob.male?.class end - def test_add_remove_single_field + def test_add_remove_single_field_using_string_arguments assert !Person.column_methods_hash.include?(:last_name) - PeopleHaveLastNames.up + ActiveRecord::Migration.add_column 'people', 'last_name', :string Person.reset_column_information assert Person.column_methods_hash.include?(:last_name) - PeopleHaveLastNames.down + ActiveRecord::Migration.remove_column 'people', 'last_name' + + Person.reset_column_information + assert !Person.column_methods_hash.include?(:last_name) + end + + def test_add_remove_single_field_using_symbol_arguments + assert !Person.column_methods_hash.include?(:last_name) + + ActiveRecord::Migration.add_column :people, :last_name, :string + + Person.reset_column_information + assert Person.column_methods_hash.include?(:last_name) + + ActiveRecord::Migration.remove_column :people, :last_name Person.reset_column_information assert !Person.column_methods_hash.include?(:last_name) -- cgit v1.2.3