dm-core 0.9.2 → 0.9.3

data/spec/integration/resource_spec.rb

@@ -1,5 +1,103 @@
 require File.expand_path(File.join(File.dirname(__FILE__), '..', 'spec_helper'))
 
+# ------------------------------------------------------------
+# -----   Read SPECS for information about how to read   -----
+# -----   and contribute to the DataMapper specs.        -----
+# ------------------------------------------------------------
+
+if ADAPTER
+  describe "DataMapper::Resource with #{ADAPTER}" do
+    
+    load_models_for_metaphor :zoo
+    
+    before(:each) do
+      DataMapper.auto_migrate!(ADAPTER)
+      @zoo = Zoo.new(:name => "San Francisco")
+      repository(ADAPTER) { @zoo.save }
+    end
+    
+  # --- Move somewhere ----
+    it "should be able to destroy objects" do
+      lambda { @zoo.destroy.should be_true }.should_not raise_error
+    end
+
+    describe '#attribute_get' do
+      it 'should provide #attribute_get' do
+        Zoo.new.should respond_to(:attribute_get)
+      end
+
+      it 'should delegate to Property#get' do
+        Zoo.properties[:name].should_receive(:get).with(zoo = Zoo.new)
+        zoo.name
+      end
+
+      it "should return Property#get's return value"  do
+        Zoo.properties[:name].should_receive(:get).and_return("San Francisco")
+        Zoo.new.name.should == "San Francisco"
+      end
+    end
+
+    describe '#attribute_set' do
+      it "should provide #attribute_set" do
+        Zoo.new.should respond_to(:attribute_set)
+      end
+
+      it 'should delegate to Property#set' do
+        Zoo.properties[:name].should_receive(:set).with(zoo = Zoo.new, "San Francisco")
+        zoo.name = "San Francisco"
+      end
+    end
+    
+    describe '#eql?' do
+      it "should return true if the objects are the same instances"
+      it "should return false if the other object is not an instance of the same model"
+      it "should return false if the other object is a different class"
+      it "should return true if the repositories are the same and the primary key is the same"
+      it "should return true if all the properties are the same"
+      it "should return false if any of the properties are different"
+    end
+    
+    describe '#id' do
+      it "should be awesome"
+    end
+    
+    describe '#inspect' do
+      it "should return a string representing the object"
+    end
+    
+    describe '#key' do
+      it "should be awesome"
+    end
+    
+    describe '#pretty_print' do
+      it "should display a pretty version of inspect"
+    end
+
+    describe '#save' do
+
+      describe 'with a new resource' do
+        it 'should set defaults before create'
+        it 'should create when dirty'
+        it 'should create when non-dirty, and it has a serial key'
+      end
+
+      describe 'with an existing resource' do
+        it 'should update'
+      end
+
+    end
+    
+    describe '#repository' do
+      it "should return the repository associated with the object if there is one"
+      it "should return the repository associated with the model if the object doesn't have one"
+    end
+  end
+end
+
+
+
+
+# ---------- Old specs... BOOOOOOOOOO ---------------
 if ADAPTER
   class Orange
     include DataMapper::Resource
@@ -133,13 +231,13 @@ if ADAPTER
 
     it "should be able to overwrite Resource#to_s" do
       repository(ADAPTER) do
-        ted = FortunePig.create!(:name => "Ted")
+        ted = FortunePig.create(:name => "Ted")
         FortunePig.get!(ted.id).to_s.should == 'Ted'
       end
     end
 
     it "should be able to destroy objects" do
-      apple = Apple.create!(:color => 'Green')
+      apple = Apple.create(:color => 'Green')
       lambda do
         apple.destroy.should be_true
       end.should_not raise_error
@@ -250,14 +348,14 @@ if ADAPTER
         Geek.auto_migrate!(ADAPTER)
 
         repository(ADAPTER) do
-          Male.create!(:name => 'John Dorian')
-          Bully.create!(:name => 'Bob', :iq => 69)
-          Geek.create!(:name => 'Steve', :awkward => false, :iq => 132)
-          Geek.create!(:name => 'Bill', :iq => 150)
-          Bully.create!(:name => 'Johnson')
-          Mugger.create!(:name => 'Frank')
-          Maniac.create!(:name => 'William')
-          Psycho.create!(:name => 'Norman')
+          Male.create(:name => 'John Dorian')
+          Bully.create(:name => 'Bob', :iq => 69)
+          Geek.create(:name => 'Steve', :awkward => false, :iq => 132)
+          Geek.create(:name => 'Bill', :iq => 150)
+          Bully.create(:name => 'Johnson')
+          Mugger.create(:name => 'Frank')
+          Maniac.create(:name => 'William')
+          Psycho.create(:name => 'Norman')
         end
 
         Flanimal.auto_migrate!(ADAPTER)

data/spec/integration/strategic_eager_loading_spec.rb

@@ -0,0 +1,138 @@
+require File.expand_path(File.join(File.dirname(__FILE__), '..', 'spec_helper'))
+
+describe "Strategic Eager Loading" do
+  include LoggingHelper
+
+  before :all do
+    class Zoo
+      include DataMapper::Resource
+      def self.default_repository_name; ADAPTER end
+
+      property :id, Serial
+      property :name, String
+
+      has n, :exhibits
+    end
+
+    class Exhibit
+      include DataMapper::Resource
+      def self.default_repository_name; ADAPTER end
+
+      property :id, Serial
+      property :name, String
+      property :zoo_id, Integer
+
+      belongs_to :zoo
+      has n, :animals
+    end
+
+    class Animal
+      include DataMapper::Resource
+      def self.default_repository_name; ADAPTER end
+
+      property :id, Serial
+      property :name, String
+      property :exhibit_id, Integer
+
+      belongs_to :exhibit
+    end
+
+    [Zoo, Exhibit, Animal].each { |k| k.auto_migrate!(ADAPTER) }
+
+    repository(ADAPTER) do
+      Zoo.create(:name => "Dallas Zoo")
+      Exhibit.create(:name => "Primates", :zoo_id => 1)
+      Animal.create(:name => "Chimpanzee", :exhibit_id => 1)
+      Animal.create(:name => "Orangutan", :exhibit_id => 1)
+
+      Zoo.create(:name => "San Diego")
+      Exhibit.create(:name => "Aviary", :zoo_id => 2)
+      Exhibit.create(:name => "Insectorium", :zoo_id => 2)
+      Exhibit.create(:name => "Bears", :zoo_id => 2)
+      Animal.create(:name => "Bald Eagle", :exhibit_id => 2)
+      Animal.create(:name => "Parakeet", :exhibit_id => 2)
+      Animal.create(:name => "Roach", :exhibit_id => 3)
+      Animal.create(:name => "Brown Bear", :exhibit_id => 4)
+    end
+  end
+
+  it "should eager load children" do
+    zoo_ids     = Zoo.all.map { |z| z.key }
+    exhibit_ids = Exhibit.all.map { |e| e.key }
+
+    repository(ADAPTER) do
+      zoos = Zoo.all.entries # load all zoos
+      dallas = zoos.find { |z| z.name == 'Dallas Zoo' }
+
+      dallas.exhibits.entries # load all exhibits for zoos in identity_map
+      dallas.exhibits.size.should == 1
+      repository.identity_map(Zoo).keys.sort.should == zoo_ids
+      repository.identity_map(Exhibit).keys.sort.should == exhibit_ids
+
+      logger do |log|
+        zoos.each { |zoo| zoo.exhibits.entries } # issues no queries
+        log.readlines.should be_empty
+      end
+
+      dallas.exhibits << Exhibit.new(:name => "Reptiles")
+      dallas.exhibits.size.should == 2
+      dallas.save
+    end
+    repository(ADAPTER) do
+      Zoo.first.exhibits.size.should == 2
+    end
+  end
+
+  it "should not eager load children when a query is provided" do
+    repository(ADAPTER) do
+      dallas = Zoo.all.entries.find { |z| z.name == 'Dallas Zoo' }
+      exhibits = dallas.exhibits.entries # load all exhibits
+
+      logger do |log|
+        reptiles = dallas.exhibits(:name => 'Reptiles')
+        reptiles.size.should == 1
+
+        primates = dallas.exhibits(:name => 'Primates')
+        primates.size.should == 1
+        primates.should_not == reptiles
+
+        log.readlines.size.should == 2
+      end
+    end
+  end
+
+  it "should eager load parents" do
+    animal_ids  = Animal.all.map { |a| a.key }
+    exhibit_ids = Exhibit.all.map { |e| e.key }.sort
+    exhibit_ids.pop # remove Reptile exhibit, which has no Animals
+
+    repository(ADAPTER) do
+      animals = Animal.all.entries
+      bear = animals.find { |a| a.name == 'Brown Bear' }
+      bear.exhibit
+      repository.identity_map(Animal).keys.sort.should == animal_ids
+      repository.identity_map(Exhibit).keys.sort.should == exhibit_ids
+    end
+  end
+
+  it "should not eager load parents when parent is in IM" do
+    repository(ADAPTER) do
+      animal = Animal.first
+      exhibit = Exhibit.get(1) # load exhibit into IM
+
+      logger do |log|
+        animal.exhibit # load exhibit from IM
+        log.readlines.should be_empty
+      end
+      repository.identity_map(Exhibit).keys.should == [exhibit.key]
+    end
+  end
+
+  it "should return a Collection when no children" do
+    Zoo.create(:name => 'Portland')
+
+    Zoo.all.each do |zoo|
+      zoo.exhibits.should be_kind_of(DataMapper::Collection)
+    end
+  end
+end

data/spec/integration/transaction_spec.rb

@@ -32,7 +32,7 @@ if ADAPTERS.any?
     it "should commit changes to all involved adapters on a two phase commit" do
       DataMapper::Transaction.new(*@repositories) do
         ADAPTERS.each do |name|
-          repository(name) { Sputnik.create!(:name => 'hepp') }
+          repository(name) { Sputnik.create(:name => 'hepp') }
         end
       end
 
@@ -45,7 +45,7 @@ if ADAPTERS.any?
       lambda do
         DataMapper::Transaction.new(*@repositories) do
           ADAPTERS.each do |name|
-            repository(name) { Sputnik.create!(:name => 'hepp') }
+            repository(name) { Sputnik.create(:name => 'hepp') }
           end
           raise "plur"
         end
@@ -60,7 +60,7 @@ if ADAPTERS.any?
       lambda do
         DataMapper::Transaction.new(*@repositories) do |transaction|
           ADAPTERS.each do |name|
-            repository(name) { Sputnik.create!(:name => 'hepp') }
+            repository(name) { Sputnik.create(:name => 'hepp') }
           end
 
           transaction.primitive_for(@repositories.last.adapter).should_receive(:prepare).and_throw(Exception.new("I am the famous test exception"))

data/spec/integration/type_spec.rb

@@ -125,6 +125,37 @@ if ADAPTER
       end
     end
 
+    it "should provide access to paranoid items with DateTime" do
+      Lemon.auto_migrate!(ADAPTER)
+
+      lemon = nil
+
+      repository(ADAPTER) do |repository|
+        %w(red green yellow blue).each do |color|
+          Lemon.create(:color => color)
+        end
+
+        Lemon.all.size.should == 4
+        Lemon.first.destroy
+        Lemon.all.size.should == 3
+        Lemon.with_deleted{Lemon.all.size.should == 1}
+      end
+    end
+
+    it "should set paranoid datetime to a date time" do
+      tmp = (DateTime.now - 0.5)
+      dt = DateTime.now
+      DateTime.stub!(:now).and_return(tmp)
+
+      repository(ADAPTER) do |repository|
+        lemon = Lemon.new
+        lemon.color = 'green'
+        lemon.save
+        lemon.destroy
+        lemon.deleted_at.should == tmp
+      end
+    end
+
     it "should respect paranoia with a boolean" do
       Lime.auto_migrate!(ADAPTER)
 
@@ -145,5 +176,95 @@ if ADAPTER
         Lime.get(lime.id).should be_nil
       end
     end
+
+    it "should provide access to paranoid items with Boolean" do
+      Lime.auto_migrate!(ADAPTER)
+
+      lemon = nil
+
+      repository(ADAPTER) do |repository|
+        %w(red green yellow blue).each do |color|
+          Lime.create(:color => color)
+        end
+
+        Lime.all.size.should == 4
+        Lime.first.destroy
+        Lime.all.size.should == 3
+        Lime.with_deleted{Lime.all.size.should == 1}
+      end
+    end
+
+    describe "paranoid types across repositories" do
+      before(:all) do
+        DataMapper::Repository.adapters[:alternate_paranoid] = repository(ADAPTER).adapter.dup
+
+        class Orange
+          include DataMapper::Resource
+
+          def self.default_repository_name
+            ADAPTER
+          end
+
+          property :id, Serial
+          property :color, String
+
+          repository(:alternate_paranoid) do
+            property :deleted,    DataMapper::Types::ParanoidBoolean
+            property :deleted_at, DataMapper::Types::ParanoidDateTime
+          end
+        end
+
+        repository(:alternate_paranoid){Orange.auto_migrate!}
+      end
+
+      before(:each) do
+        %w(red orange blue green).each{|color| o = Orange.create(:color => color)}
+      end
+
+      after(:each) do
+        Orange.repository.adapter.execute("DELETE FROM oranges")
+      end
+
+      it "should setup the correct objects for the spec" do
+        repository(:alternate_paranoid){Orange.all.should have(4).items}
+      end
+
+      it "should allow access the the default repository" do
+        Orange.all.should have(4).items
+      end
+
+      it "should mark the objects as deleted in the alternate_paranoid repository" do
+        repository(:alternate_paranoid) do
+          Orange.first.destroy
+          Orange.all.should have(3).items
+          Orange.find_by_sql("SELECT * FROM oranges").should have(4).items
+        end
+      end
+
+      it "should mark the objects as deleted in the alternate_paranoid repository but ignore it in the #{ADAPTER} repository" do
+        repository(:alternate_paranoid) do
+          Orange.first.destroy
+        end
+        Orange.all.should have(4).items
+      end
+
+      it "should raise an error when trying to destroy from a repository that is not paranoid" do
+        lambda do
+          Orange.first.destroy
+        end.should raise_error(ArgumentError)
+      end
+
+      it "should set all paranoid attributes on delete" do
+        repository(:alternate_paranoid) do
+          orange = Orange.first
+          orange.deleted.should be_false
+          orange.deleted_at.should be_nil
+          orange.destroy
+
+          orange.deleted.should be_true
+          orange.deleted_at.should be_a_kind_of(DateTime)
+        end
+      end
+    end
   end
 end

data/spec/lib/logging_helper.rb

@@ -0,0 +1,18 @@
+module LoggingHelper
+  def logger(adapter = ADAPTER, &block)
+    current_adapter = DataObjects.const_get(repository(adapter).adapter.uri.scheme.capitalize)
+    old_logger = current_adapter.logger
+
+    log_path = File.join(SPEC_ROOT, "tmp.log")
+    handle = File.open(log_path, "a+")
+    current_adapter.logger = DataObjects::Logger.new(log_path, 0)
+    begin
+      yield(handle)
+    ensure
+      handle.truncate(0)
+      handle.close
+      current_adapter.logger = old_logger
+      File.delete(log_path)
+    end
+  end
+end