diff options
author | Leonel Galan <leonelgalan@gmail.com> | 2017-09-06 13:31:23 -0400 |
---|---|---|
committer | Leonel Galan <leonelgalan@gmail.com> | 2018-01-22 10:04:47 -0500 |
commit | 3562331669443b600f1cf9fd93a0cddaaf94d319 (patch) | |
tree | bd19f9c4b9452e4961bbcbe16002cb370d9c76ca /activerecord/test/cases | |
parent | 7bb0b397ea51b0e41b5ebdf4fe5edcfc7345b600 (diff) | |
download | rails-3562331669443b600f1cf9fd93a0cddaaf94d319.tar.gz rails-3562331669443b600f1cf9fd93a0cddaaf94d319.tar.bz2 rails-3562331669443b600f1cf9fd93a0cddaaf94d319.zip |
Ignores a default subclass when `becomes(Parent)`
Fixes issue described in #30399: A default value on the
inheritance column prevented `child.becomes(Parent)` to return
an instance of `Parent` as expected, instead it returns an instance
of the default subclass.
The change was introduced by #17169 and it was meant to affect
initialization, alone. Where `Parent.new` is expected to return
an instance of the default subclass.
Diffstat (limited to 'activerecord/test/cases')
-rw-r--r-- | activerecord/test/cases/persistence_test.rb | 16 |
1 files changed, 16 insertions, 0 deletions
diff --git a/activerecord/test/cases/persistence_test.rb b/activerecord/test/cases/persistence_test.rb index 0edca96cf5..d242fff442 100644 --- a/activerecord/test/cases/persistence_test.rb +++ b/activerecord/test/cases/persistence_test.rb @@ -473,6 +473,22 @@ class PersistenceTest < ActiveRecord::TestCase assert_instance_of Reply, Reply.find(reply.id) end + def test_becomes_default_sti_subclass + original_type = Topic.columns_hash["type"].default + ActiveRecord::Base.connection.change_column_default :topics, :type, "Reply" + Topic.reset_column_information + + reply = topics(:second) + assert_instance_of Reply, reply + + topic = reply.becomes(Topic) + assert_instance_of Topic, topic + + ensure + ActiveRecord::Base.connection.change_column_default :topics, :type, original_type + Topic.reset_column_information + end + def test_update_after_create klass = Class.new(Topic) do def self.name; "Topic"; end |