aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord
diff options
context:
space:
mode:
authorRyuta Kamizono <kamipo@gmail.com>2016-06-25 20:43:09 +0900
committerRyuta Kamizono <kamipo@gmail.com>2016-06-25 21:17:27 +0900
commit59c218ea9b41e3ef5cef571f31fb83a7ec72e9a4 (patch)
treefca44d86e251af04aec2ea779bb59efbd27bc3ee /activerecord
parent6da5f6b451e27d59c2492e504350d0fc70c37e3a (diff)
downloadrails-59c218ea9b41e3ef5cef571f31fb83a7ec72e9a4.tar.gz
rails-59c218ea9b41e3ef5cef571f31fb83a7ec72e9a4.tar.bz2
rails-59c218ea9b41e3ef5cef571f31fb83a7ec72e9a4.zip
Remove unnecessary `assert_valid_default`
This was added at c7c3f73 but it never raised because MySQL cannot create text/blob columns with a default value.
Diffstat (limited to 'activerecord')
-rw-r--r--activerecord/lib/active_record/connection_adapters/mysql/column.rb7
-rw-r--r--activerecord/test/cases/column_definition_test.rb7
2 files changed, 0 insertions, 14 deletions
diff --git a/activerecord/lib/active_record/connection_adapters/mysql/column.rb b/activerecord/lib/active_record/connection_adapters/mysql/column.rb
index 9c45fdd44a..ea554b188c 100644
--- a/activerecord/lib/active_record/connection_adapters/mysql/column.rb
+++ b/activerecord/lib/active_record/connection_adapters/mysql/column.rb
@@ -6,7 +6,6 @@ module ActiveRecord
def initialize(*)
super
- assert_valid_default
extract_default
end
@@ -38,12 +37,6 @@ module ActiveRecord
@default = null || strict ? nil : ''
end
end
-
- def assert_valid_default
- if blob_or_text_column? && default.present?
- raise ArgumentError, "#{type} columns cannot have a default value: #{default.inspect}"
- end
- end
end
end
end
diff --git a/activerecord/test/cases/column_definition_test.rb b/activerecord/test/cases/column_definition_test.rb
index 81162b7e98..78c0853992 100644
--- a/activerecord/test/cases/column_definition_test.rb
+++ b/activerecord/test/cases/column_definition_test.rb
@@ -60,15 +60,8 @@ module ActiveRecord
end
def test_should_not_set_default_for_blob_and_text_data_types
- assert_raise ArgumentError do
- MySQL::Column.new("title", "a", SqlTypeMetadata.new(sql_type: "blob"))
- end
-
text_type = MySQL::TypeMetadata.new(
SqlTypeMetadata.new(type: :text))
- assert_raise ArgumentError do
- MySQL::Column.new("title", "Hello", text_type)
- end
text_column = MySQL::Column.new("title", nil, text_type)
assert_equal nil, text_column.default