From 8ffb22056c2896a67917b813064b2247233d3a08 Mon Sep 17 00:00:00 2001 From: Jeremy Kemper Date: Fri, 1 Dec 2006 21:24:47 +0000 Subject: Subclasses of an abstract class work with single-table inheritance. Closes #5704. git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@5660 5ecf4fe2-1ee6-0310-87b1-e25e094e27de --- activerecord/test/base_test.rb | 57 +++++++++++++++++++++- .../test/fixtures/db_definitions/mysql.sql | 6 +-- .../test/fixtures/db_definitions/sqlite.sql | 4 +- activerecord/test/fixtures/post.rb | 3 +- 4 files changed, 62 insertions(+), 8 deletions(-) (limited to 'activerecord/test') diff --git a/activerecord/test/base_test.rb b/activerecord/test/base_test.rb index 8cd17d7034..3eb5a676a4 100755 --- a/activerecord/test/base_test.rb +++ b/activerecord/test/base_test.rb @@ -10,6 +10,7 @@ require 'fixtures/auto_id' require 'fixtures/column_name' require 'fixtures/subscriber' require 'fixtures/keyboard' +require 'fixtures/post' class Category < ActiveRecord::Base; end class Smarts < ActiveRecord::Base; end @@ -28,8 +29,9 @@ class NonExistentTable < ActiveRecord::Base; end class TestOracleDefault < ActiveRecord::Base; end class LoosePerson < ActiveRecord::Base - attr_protected :credit_rating, :administrator + self.table_name = 'people' self.abstract_class = true + attr_protected :credit_rating, :administrator end class LooseDescendant < LoosePerson @@ -37,6 +39,7 @@ class LooseDescendant < LoosePerson end class TightPerson < ActiveRecord::Base + self.table_name = 'people' attr_accessible :name, :address end @@ -1361,13 +1364,63 @@ class BasicsTest < Test::Unit::TestCase end end - def test_base_class + def test_abstract_class + assert ActiveRecord::Base.abstract_class? assert LoosePerson.abstract_class? assert !LooseDescendant.abstract_class? + end + + def test_base_class assert_equal LoosePerson, LoosePerson.base_class assert_equal LooseDescendant, LooseDescendant.base_class assert_equal TightPerson, TightPerson.base_class assert_equal TightPerson, TightDescendant.base_class + + assert_equal Post, Post.base_class + assert_equal Post, SpecialPost.base_class + assert_equal Post, StiPost.base_class + assert_equal SubStiPost, SubStiPost.base_class + end + + def test_descends_from_active_record + # Tries to call Object.abstract_class? + assert_raise(NoMethodError) do + ActiveRecord::Base.descends_from_active_record? + end + + # Abstract subclass of AR::Base. + assert LoosePerson.descends_from_active_record? + + # Concrete subclass of an abstract class. + assert LooseDescendant.descends_from_active_record? + + # Concrete subclass of AR::Base. + assert TightPerson.descends_from_active_record? + + # Concrete subclass of a concrete class but has no type column. + assert TightDescendant.descends_from_active_record? + + # Concrete subclass of AR::Base. + assert Post.descends_from_active_record? + + # Abstract subclass of a concrete class which has a type column. + # This is pathological, as you'll never have Sub < Abstract < Concrete. + assert !StiPost.descends_from_active_record? + + # Concrete subclasses an abstract class which has a type column. + assert SubStiPost.descends_from_active_record? + end + + def test_find_on_abstract_base_class_doesnt_use_type_condition + old_class = LooseDescendant + Object.send :remove_const, :LooseDescendant + + descendant = old_class.create! + assert_not_nil LoosePerson.find(descendant.id), "Should have found instance of LooseDescendant when finding abstract LoosePerson: #{descendant.inspect}" + ensure + unless Object.const_defined?(:LooseDescendant) + Object.const_set :LooseDescendant, old_class + end end def test_assert_queries diff --git a/activerecord/test/fixtures/db_definitions/mysql.sql b/activerecord/test/fixtures/db_definitions/mysql.sql index c6a78962c6..9e5cf72aba 100755 --- a/activerecord/test/fixtures/db_definitions/mysql.sql +++ b/activerecord/test/fixtures/db_definitions/mysql.sql @@ -159,14 +159,14 @@ CREATE TABLE `posts` ( `author_id` INTEGER, `title` VARCHAR(255) NOT NULL, `body` TEXT NOT NULL, - `type` VARCHAR(255) NOT NULL + `type` VARCHAR(255) default NULL ) TYPE=InnoDB; CREATE TABLE `comments` ( `id` INTEGER NOT NULL auto_increment PRIMARY KEY, `post_id` INTEGER NOT NULL, `body` TEXT NOT NULL, - `type` VARCHAR(255) NOT NULL + `type` VARCHAR(255) default NULL ) TYPE=InnoDB; CREATE TABLE `authors` ( @@ -184,7 +184,7 @@ CREATE TABLE `tasks` ( CREATE TABLE `categories` ( `id` int(11) NOT NULL auto_increment, `name` VARCHAR(255) NOT NULL, - `type` VARCHAR(255) NOT NULL, + `type` VARCHAR(255) default NULL, PRIMARY KEY (`id`) ) TYPE=InnoDB; diff --git a/activerecord/test/fixtures/db_definitions/sqlite.sql b/activerecord/test/fixtures/db_definitions/sqlite.sql index 8f89c419a1..edf1e815da 100644 --- a/activerecord/test/fixtures/db_definitions/sqlite.sql +++ b/activerecord/test/fixtures/db_definitions/sqlite.sql @@ -144,14 +144,14 @@ CREATE TABLE 'posts' ( 'id' INTEGER NOT NULL PRIMARY KEY, 'author_id' INTEGER, 'title' VARCHAR(255) NOT NULL, - 'type' VARCHAR(255) NOT NULL, + 'type' VARCHAR(255) DEFAULT NULL, 'body' TEXT NOT NULL ); CREATE TABLE 'comments' ( 'id' INTEGER NOT NULL PRIMARY KEY, 'post_id' INTEGER NOT NULL, - 'type' VARCHAR(255) NOT NULL, + 'type' VARCHAR(255) DEFAULT NULL, 'body' TEXT NOT NULL ); diff --git a/activerecord/test/fixtures/post.rb b/activerecord/test/fixtures/post.rb index 9dc18441d0..44ac3c1053 100644 --- a/activerecord/test/fixtures/post.rb +++ b/activerecord/test/fixtures/post.rb @@ -46,7 +46,7 @@ class Post < ActiveRecord::Base end end -class SpecialPost < Post; end; +class SpecialPost < Post; end class StiPost < Post self.abstract_class = true @@ -54,4 +54,5 @@ class StiPost < Post end class SubStiPost < StiPost + self.table_name = Post.table_name end -- cgit v1.2.3