aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/test/cases/persistence_test.rb
diff options
context:
space:
mode:
authorLeonel Galan <leonelgalan@gmail.com>2017-09-06 13:31:23 -0400
committerLeonel Galan <leonelgalan@gmail.com>2018-01-22 10:04:47 -0500
commit3562331669443b600f1cf9fd93a0cddaaf94d319 (patch)
treebd19f9c4b9452e4961bbcbe16002cb370d9c76ca /activerecord/test/cases/persistence_test.rb
parent7bb0b397ea51b0e41b5ebdf4fe5edcfc7345b600 (diff)
downloadrails-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/persistence_test.rb')
-rw-r--r--activerecord/test/cases/persistence_test.rb16
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