aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/test
diff options
context:
space:
mode:
authorSean Griffin <sean@thoughtbot.com>2015-03-10 11:35:09 -0600
committerSean Griffin <sean@thoughtbot.com>2015-03-10 11:56:45 -0600
commitafc124c3b417b5327ed4d85dc34393f3d19bfbcf (patch)
tree5aca59a32ca87ef927ffd687c261cf5a110ebda7 /activerecord/test
parent08469012e4db36d4d8dea507691070cd39526268 (diff)
downloadrails-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.rb43
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