Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

Merge pull request #6593 from yahonda/skip_test_remove_column_with_ar…

…ray_as_an_argument_is_deprecated

Skip test_remove_column_with_array_as_an_argument_is_deprecated with Oracle
  • Loading branch information...
commit 43f0af4d39a5d8c1191fef589d947fc92a34f7d4 2 parents f550d4d + 7544c7a
@rafaelfranca rafaelfranca authored
Showing with 3 additions and 1 deletion.
  1. +3 −1 activerecord/test/cases/migration_test.rb
View
4 activerecord/test/cases/migration_test.rb
@@ -874,6 +874,8 @@ def test_remove_column_no_second_parameter_raises_exception
end
def test_remove_column_with_array_as_an_argument_is_deprecated
+ return skip "remove_column with array as argument is not supported with OracleAdapter" if current_adapter? :OracleAdapter
+
ActiveRecord::Base.connection.create_table(:hats) do |table|
table.column :hat_name, :string, :limit => 100
table.column :hat_size, :integer
@@ -884,7 +886,7 @@ def test_remove_column_with_array_as_an_argument_is_deprecated
Person.connection.remove_column("hats", ["hat_name", "hat_size"])
end
ensure
- ActiveRecord::Base.connection.drop_table(:hats)
+ ActiveRecord::Base.connection.drop_table(:hats) rescue nil
end
def test_change_type_of_not_null_column
Please sign in to comment.
Something went wrong with that request. Please try again.