mirror of
https://github.com/rails/rails.git
synced 2022-11-09 12:12:34 -05:00
Merge pull request #9597 from senny/9110_serialized_not_null_default
test case for `serialize` default values.
This commit is contained in:
commit
c6839277a9
4 changed files with 13 additions and 0 deletions
|
@ -1,6 +1,7 @@
|
||||||
require 'cases/helper'
|
require 'cases/helper'
|
||||||
require 'models/topic'
|
require 'models/topic'
|
||||||
require 'models/person'
|
require 'models/person'
|
||||||
|
require 'models/traffic_light'
|
||||||
require 'bcrypt'
|
require 'bcrypt'
|
||||||
|
|
||||||
class SerializedAttributeTest < ActiveRecord::TestCase
|
class SerializedAttributeTest < ActiveRecord::TestCase
|
||||||
|
@ -234,4 +235,10 @@ class SerializedAttributeTest < ActiveRecord::TestCase
|
||||||
person = person.reload
|
person = person.reload
|
||||||
assert_equal(insures, person.insures)
|
assert_equal(insures, person.insures)
|
||||||
end
|
end
|
||||||
|
|
||||||
|
def test_regression_serialized_default_on_text_column_with_null_false
|
||||||
|
light = TrafficLight.new
|
||||||
|
assert_equal [], light.state
|
||||||
|
assert_equal [], light.long_state
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
|
@ -4,3 +4,7 @@ uk:
|
||||||
- Green
|
- Green
|
||||||
- Red
|
- Red
|
||||||
- Orange
|
- Orange
|
||||||
|
long_state:
|
||||||
|
- "Green, go ahead"
|
||||||
|
- "Red, wait"
|
||||||
|
- "Orange, caution light is about to switch"
|
|
@ -1,3 +1,4 @@
|
||||||
class TrafficLight < ActiveRecord::Base
|
class TrafficLight < ActiveRecord::Base
|
||||||
serialize :state, Array
|
serialize :state, Array
|
||||||
|
serialize :long_state, Array
|
||||||
end
|
end
|
||||||
|
|
|
@ -687,6 +687,7 @@ ActiveRecord::Schema.define do
|
||||||
create_table :traffic_lights, :force => true do |t|
|
create_table :traffic_lights, :force => true do |t|
|
||||||
t.string :location
|
t.string :location
|
||||||
t.string :state
|
t.string :state
|
||||||
|
t.text :long_state, :null => false
|
||||||
t.datetime :created_at
|
t.datetime :created_at
|
||||||
t.datetime :updated_at
|
t.datetime :updated_at
|
||||||
end
|
end
|
||||||
|
|
Loading…
Reference in a new issue