active_record-mti 0.3.2 → 0.4.0.pre.1

data/lib/active_record/mti/querying.rb

@@ -1,7 +0,0 @@
-module ActiveRecord
-  module MTI
-    module Querying
-      delegate :count_estimate, to: :all
-    end
-  end
-end

data/lib/active_record/mti/registry.rb

@@ -1,23 +0,0 @@
-module ActiveRecord
-  module MTI
-    module Registry
-
-      mattr_accessor :tableoids
-      self.tableoids = { ActiveRecord::Base => false }
-
-      def self.[]=(klass, tableoid)
-        ActiveRecord::MTI.logger.debug "Adding #{klass} to MTI list with #{tableoid}"
-        tableoids[klass] = tableoid
-      end
-
-      def self.find_mti_class(tableoid)
-        tableoids.key(tableoid)
-      end
-
-      def self.tableoid?(klass)
-        tableoids[klass]
-      end
-
-    end
-  end
-end

data/spec/active_record/mti/calculations_spec.rb

@@ -1,56 +0,0 @@
-require 'spec_helper'
-
-describe ActiveRecord::MTI::Calculations do
-
-  context 'when an exception occurs' do
-    it 'does not continue to adversely affect additional queries' do
-      Admin.create(email: 'bob')
-      expect{ Admin.joins(Transportation::Vehicle).count }.to raise_error(RuntimeError)
-      expect(Thread.current['skip_tableoid_cast']).to_not eq(true)
-      expect(Admin.count).to eq(1)
-    end
-  end
-
-  context "don't project tableoid on" do
-    it "grouping" do
-
-      Admin.create(email: 'foo@bar.baz', god_powers: 3)
-      Admin.create(email: 'foo@bar.baz', god_powers: 3)
-      Admin.create(email: 'foo24@bar.baz', god_powers: 3)
-
-      grouped_count = Admin.group(:email).count
-
-      expect(grouped_count['foo24@bar.baz']).to eq(1)
-      expect(grouped_count['foo@bar.baz']).to eq(2)
-
-    end
-
-    it "count calculations" do
-
-      Admin.create(email: 'foo@bar.baz', god_powers: 3)
-      Admin.create(email: 'foo@bar.baz', god_powers: 3)
-      Admin.create(email: 'foo24@bar.baz', god_powers: 3)
-
-      expect(Admin.count(:email)).to eq(3)
-
-    end
-  end
-
-  context "projects tableoid" do
-    it "and groups tableoid when selecting :tableoid" do
-      sql = Admin.select(:email, :tableoid).group(:email).to_sql
-
-      expect(sql).to match(/SELECT .*, \"admins\".\"tableoid\" AS tableoid FROM \"admins\"/)
-
-      expect(sql).to match(/GROUP BY .*, \"admins\".\"tableoid\"/)
-    end
-
-    it "when grouping :tableoid" do
-      sql = Admin.select(:email).group(:email, :tableoid).to_sql
-
-      expect(sql).to match(/SELECT .*, \"admins\".\"tableoid\" AS tableoid FROM \"admins\"/)
-
-      expect(sql).to match(/GROUP BY .*, \"admins\".\"tableoid\"/)
-    end
-  end
-end

data/spec/active_record/mti/inheritance_spec.rb

@@ -1,184 +0,0 @@
-require 'spec_helper'
-
-describe ActiveRecord::MTI::Inheritance do
-
-  it "creates a column even if class doesn't respond to :attribute" do
-    allow(Admin).to receive(:respond_to?).with(:attribute).and_return(false)
-
-    ActiveRecord::MTI::Registry.tableoids[Admin] = nil
-
-    expect(Admin.using_multi_table_inheritance?).to eq(true)
-  end
-
-  it "warns of deprication when using old `uses_mti`" do
-    expect { Admin.uses_mti }.to output("DEPRECATED - `uses_mti` is no longer needed (nor has any effect)\n").to_stderr
-  end
-
-  context 'class definition' do
-
-    describe 'for classes that use MTI' do
-
-      it 'has non-nil mti_type_column' do
-        expect(Admin.mti_type_column).to_not be_nil
-      end
-
-      it 'has true tableoid_column' do
-        expect(Admin.tableoid_column).to eq(true)
-      end
-
-      it "doesn't check inheritance multiple times" do
-        # Due to the anonymous class ("god = Class.new(Admin)") rspec can't properly distinquish
-        # between the two classes. So at most 2 times!
-        expect(Admin).to receive(:check_inheritance_of).and_call_original.at_most(2).times
-
-        Admin.create(email: 'foo@bar.baz', god_powers: 3)
-        Admin.create(email: 'foo2@bar.baz', god_powers: 3)
-        Admin.create(email: 'foo24@bar.baz', god_powers: 3)
-        Admin.create(email: 'foo246@bar.baz', god_powers: 3)
-
-      end
-    end
-
-    describe "for classes that don't use MTI" do
-
-      it 'has nil tableoid_column' do
-        expect(Post.tableoid_column).to be_nil
-      end
-
-      it 'has nil mti_type_column' do
-        expect(Post.mti_type_column).to be_nil
-      end
-
-      it "doesn't check inheritance multiple times" do
-        # ActiveRecord::MTI::Inheritance.register(Post, false)
-        expect(Post).to receive(:check_inheritance_of).and_call_original.exactly(1).times
-
-        Post.create(title: 'foo@bar.baz')
-        Post.create(title: 'foo2@bar.baz')
-        Post.create(title: 'foo24@bar.baz')
-
-      end
-    end
-
-  end
-
-  context 'default inheritance_column model' do
-    let!(:user) { User.create(email: 'foo@bar.baz') }
-    let!(:admin) { Admin.create(email: 'foo@bar.baz', god_powers: 3) }
-
-    it 'casts properly' do
-      user = User.first
-      expect(user.class).to eq(User)
-    end
-
-    describe 'base class querying' do
-      it 'casts children properly' do
-        users = User.all
-        expect(users.select{ |u| u.is_a?(Admin) }.count).to eql(1)
-      end
-
-      xit 'deserializes children with child specific data' do
-        my_admin = User.find(admin.id)
-        expect(my_admin.god_powers).to eql(3)
-      end
-    end
-
-    describe 'has the correct count for' do
-      it 'parents' do
-        users = User.all
-        expect(users.count).to eq(2)
-      end
-
-      it 'children' do
-        admins = Admin.all
-        expect(admins.count).to eq(1)
-      end
-    end
-
-    describe 'dynamic class creation' do
-      it 'infers the table_name from superclass not base_class' do
-        god = Class.new(Admin)
-        expect(god.table_name).to eql(Admin.table_name)
-      end
-
-      it 'infers the table_name when defined dynamically' do
-
-        class Scrub < ActiveRecord::Base
-          const_set(:All, Class.new(Scrub) do |klass|
-            class_eval <<-AAA
-              self.table_name = 'scrubs/all'
-            AAA
-          end)
-        end
-
-        expect(Scrub::All.table_name).to eq('scrubs/all')
-      end
-    end
-  end
-
-  describe 'views' do
-    before(:each) do
-
-      User.connection.execute <<-SQL
-        CREATE OR REPLACE VIEW "users_all"
-        AS SELECT * FROM "users"
-      SQL
-
-      class UserView < User
-        self.table_name = "users_all"
-      end
-
-    end
-
-    if ActiveRecord::Base.connection.version >= Gem::Version.new('9.4')
-      it 'allows creation pass-through' do
-
-        UserView.create(email: 'dale@twilightcoders.net')
-      end
-    end
-  end
-
-  describe 'dynamic class creation' do
-    it 'infers the table_name from superclass not base_class' do
-      God = Class.new(Admin)
-      Hacker = Class.new(Admin)
-
-      expect(God.table_name).to eql(Admin.table_name)
-      expect(Hacker.table_name).to eql('admin/hackers')
-    end
-  end
-
-  describe 'custom inheritance_column model' do
-    let!(:vehicle) { Transportation::Vehicle.create(color: :red) }
-    let!(:truck) { Transportation::Truck.create(color: :blue, bed_size: 10) }
-
-    describe 'inheritance_column' do
-      xit 'should set the custom column correctly' do
-        expect(vehicle.type).to eql('vehicles')
-        expect(truck.type).to eql('trucks')
-      end
-    end
-
-    describe 'base class querying' do
-      it 'casts children properly' do
-        expect(Transportation::Vehicle.all.select{ |v| v.is_a?(Transportation::Truck) }.count).to eql(1)
-      end
-
-      xit 'deserializes children with child specific data' do
-        my_truck = Transportation::Vehicle.find(truck.id)
-        expect(my_truck.bed_size).to eql(10)
-      end
-    end
-
-    describe 'has the correct count for' do
-      it 'parents' do
-        expect(Transportation::Vehicle.count).to eql(2)
-      end
-
-      it 'children' do
-        expect(Transportation::Truck.count).to eql(1)
-      end
-    end
-  end
-
-end

data/spec/active_record/mti/model_schema_spec.rb

@@ -1,11 +0,0 @@
-require 'spec_helper'
-
-describe ActiveRecord::MTI::ModelSchema do
-
-  it 'rescues suffix' do
-    f = ActiveRecord::ModelSchema::ClassMethods
-    allow_any_instance_of(f).to receive(:full_table_name_suffix).and_raise(NoMethodError)
-    expect( Admin.full_table_name_suffix ).to eq("")
-  end
-
-end

data/spec/active_record/mti/query_methods_spec.rb

@@ -1,12 +0,0 @@
-require 'spec_helper'
-
-describe ActiveRecord::MTI::QueryMethods do
-
-  context "queries" do
-    it "select tableoid" do
-      sql = Admin.all.to_sql
-      expect(sql).to match(/SELECT .*, \"admins\".\"tableoid\" AS tableoid FROM \"admins\"/)
-    end
-  end
-
-end

data/spec/active_record/mti/schema_dumper_spec.rb

@@ -1,22 +0,0 @@
-require 'spec_helper'
-
-describe ActiveRecord::MTI::SchemaDumper do
-
-  before(:each) do
-    ActiveRecord::SchemaMigration.create_table
-  end
-
-  let(:hacker_sql) {
-<<-FOO
-  create_table "admin/hackers", inherits: 'admins' do |t|
-  end
-FOO
-  }
-
-  it 'does not dump indexes for child table' do
-    stream = StringIO.new
-    ActiveRecord::SchemaDumper.dump(ActiveRecord::Base.connection, stream)
-
-    expect(stream.string).to include(hacker_sql)
-  end
-end

data/spec/active_record/mti_spec.rb

@@ -1,24 +0,0 @@
-require 'spec_helper'
-
-describe ActiveRecord::MTI do
-
-  context 'helper' do
-
-    describe '#testify' do
-      it "returns true for truthy values" do
-        expect(ActiveRecord::MTI.testify('f')).to eq(false)
-      end
-    end
-
-    it "root has the right value" do
-      expect(ActiveRecord::MTI.root).not_to be_nil
-    end
-
-    it "recovers if oid class candidate is not constantizable" do
-      ActiveRecord::MTI.oid_class_candidates.unshift("IDontExist")
-      expect(ActiveRecord::MTI.find_oid_class).not_to be_nil
-    end
-
-  end
-
-end

data/spec/active_record/sti/inheritance_spec.rb

@@ -1,24 +0,0 @@
-require 'spec_helper'
-
-describe ActiveRecord::Inheritance do
-
-  context 'class definition' do
-
-    describe 'for classes that use STI' do
-      it "doesn't check inheritance multiple times" do
-
-        Transportation::Military::Vehicle.create(color: :red)
-        Transportation::Military::Vehicle.create(color: :blue)
-        Transportation::Military::Vehicle.create(color: :green)
-        Transportation::Military::Vehicle.create(color: :gold)
-
-        vehicle = Transportation::Military::Vehicle.first
-        expect(vehicle.class.name).to eq('Transportation::Military::Vehicle')
-        expect(vehicle.color).to eq('red')
-
-      end
-    end
-
-  end
-
-end

data/spec/spec_helper.rb

@@ -1,28 +0,0 @@
-ENV['RAILS_ENV'] = 'test'
-
-require 'database_cleaner'
-require 'combustion'
-
-require 'simplecov'
-SimpleCov.start do
-  add_filter 'spec'
-end
-
-Combustion.path = 'spec/support/rails'
-Combustion.initialize! :active_record
-
-RSpec.configure do |config|
-  config.order = 'random'
-
-  # Configure the DatabaseCleaner
-  config.before(:suite) do
-    DatabaseCleaner.strategy = :transaction
-    DatabaseCleaner.clean_with(:truncation)
-  end
-
-  config.around(:each) do |example|
-    DatabaseCleaner.cleaning do
-      example.run
-    end
-  end
-end

data/spec/support/rails/app/models/admin.rb

@@ -1,3 +0,0 @@
-class Admin < User
-  self.table_name = 'admins'
-end

data/spec/support/rails/app/models/comment.rb

@@ -1,4 +0,0 @@
-class Comment < ::ActiveRecord::Base
-  belongs_to :user
-  belongs_to :post
-end

data/spec/support/rails/app/models/post.rb

@@ -1,4 +0,0 @@
-class Post < ::ActiveRecord::Base
-  belongs_to :user
-  has_many :comments
-end

data/spec/support/rails/app/models/transportation/military/vehicle.rb

@@ -1,7 +0,0 @@
-module Transportation
-  module Military
-    class Vehicle < ::Transportation::Vehicle
-      self.inheritance_column = 'type'
-    end
-  end
-end

data/spec/support/rails/app/models/transportation/truck.rb

@@ -1,5 +0,0 @@
-module Transportation
-  class Truck < Vehicle
-    self.table_name = 'vehicles/trucks'
-  end
-end

data/spec/support/rails/app/models/transportation/vehicle.rb

@@ -1,4 +0,0 @@
-module Transportation
-  class Vehicle < ::ActiveRecord::Base
-  end
-end