Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Fix build - 3.2 #6219

Merged
merged 1 commit into from

4 participants

@frodsan

No description provided.

Francesco Rodriguez Fix build ba48e23
@carlosantoniodasilva

Tests ok :+1:

@jeremy
Owner

Is this a backport from master?

@carlosantoniodasilva

@jeremy it was changed in master: ae8f497, and deprecated in 3.2: 02ca915, so this test only exists in 3-2-stable branch, but seems the test is failing due to not existing column.

@frodsan
@frodsan
@drogus drogus merged commit 8f5079f into rails:3-2-stable
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on May 9, 2012
  1. Fix build

    Francesco Rodriguez authored
This page is out of date. Refresh to see the latest.
Showing with 9 additions and 1 deletion.
  1. +9 −1 activerecord/test/cases/migration_test.rb
View
10 activerecord/test/cases/migration_test.rb
@@ -874,9 +874,17 @@ def test_remove_column_no_second_parameter_raises_exception
end
def test_remove_column_with_array_as_an_argument_is_deprecated
+ ActiveRecord::Base.connection.create_table(:hats) do |table|
+ table.column :hat_name, :string, :limit => 100
+ table.column :hat_size, :integer
+ table.column :hat_style, :string, :limit => 100
+ end
+
assert_deprecated /Passing array to remove_columns is deprecated/ do
- Person.connection.remove_column("people", ["last_name", "description"])
+ Person.connection.remove_column("hats", ["hat_name", "hat_size"])
end
+ ensure
+ ActiveRecord::Base.connection.drop_table(:hats)
end
def test_change_type_of_not_null_column
Something went wrong with that request. Please try again.