diff options
author | Sean Griffin <sean@thoughtbot.com> | 2015-03-10 11:35:09 -0600 |
---|---|---|
committer | Sean Griffin <sean@thoughtbot.com> | 2015-03-10 11:56:45 -0600 |
commit | afc124c3b417b5327ed4d85dc34393f3d19bfbcf (patch) | |
tree | 5aca59a32ca87ef927ffd687c261cf5a110ebda7 /activerecord/test | |
parent | 08469012e4db36d4d8dea507691070cd39526268 (diff) | |
download | rails-afc124c3b417b5327ed4d85dc34393f3d19bfbcf.tar.gz rails-afc124c3b417b5327ed4d85dc34393f3d19bfbcf.tar.bz2 rails-afc124c3b417b5327ed4d85dc34393f3d19bfbcf.zip |
Attempt to provide backwards compatible YAML deserialization
I should have done this in the first place. We are now serializing an
explicit version so we can make more careful changes in the future. This
will load Active Record objects which were serialized in Rails 4.1.
There will be bugs, as YAML serialization was at least partially broken
back then. There will also be edge cases that we might not be able to
handle, especially if the type of a column has changed.
In addition, we're passing this as `from_database`, since that is
required for serialized columns at minimum. All other types were
serializing the cast value. At a glance, there should be no types for
which this is a problem.
Finally, dirty checking information will be lost on records serialized
in 4.1, so no columns will be marked as changed.
Diffstat (limited to 'activerecord/test')
-rw-r--r-- | activerecord/test/cases/yaml_serialization_test.rb | 43 |
1 files changed, 43 insertions, 0 deletions
diff --git a/activerecord/test/cases/yaml_serialization_test.rb b/activerecord/test/cases/yaml_serialization_test.rb index bce59b4fcd..65cd54e380 100644 --- a/activerecord/test/cases/yaml_serialization_test.rb +++ b/activerecord/test/cases/yaml_serialization_test.rb @@ -83,4 +83,47 @@ class YamlSerializationTest < ActiveRecord::TestCase assert_equal 5, author.posts_count assert_equal 5, dumped.posts_count end + + def test_a_yaml_version_is_provided_for_future_backwards_compat + coder = {} + Topic.first.encode_with(coder) + + assert coder['active_record_yaml_version'] + end + + def test_deserializing_rails_41_yaml + yaml = <<-YAML.strip_heredoc + --- !ruby/object:Topic + attributes: + id: + title: The First Topic + author_name: David + author_email_address: david@loudthinking.com + written_on: 2003-07-16 14:28:11.223300000 Z + bonus_time: 2000-01-01 14:28:00.000000000 Z + last_read: 2004-04-15 + content: | + --- + :omg: :lol + important: + approved: false + replies_count: 1 + unique_replies_count: 0 + parent_id: + parent_title: + type: + group: + created_at: 2015-03-10 17:05:42.000000000 Z + updated_at: 2015-03-10 17:05:42.000000000 Z + YAML + topic = YAML.load(yaml) + + assert topic.new_record? + assert_equal nil, topic.id + assert_equal "The First Topic", topic.title + assert_equal({ omg: :lol }, topic.content) + end + + def test_deserializing_rails_42_yaml + end end |