dm-core 0.10.2 → 1.0.0.rc1

data/spec/semipublic/resource/state/dirty_spec.rb

@@ -0,0 +1,133 @@
+require File.expand_path(File.join(File.dirname(__FILE__), '..', '..', '..', 'spec_helper'))
+
+describe DataMapper::Resource::State::Dirty do
+  before :all do
+    class ::Author
+      include DataMapper::Resource
+
+      property :id,     HugeInteger, :key => true, :default => 1
+      property :name,   String
+      property :active, Boolean,     :default => true
+      property :coding, Boolean,     :default => true
+
+      belongs_to :parent, self, :required => false
+      has n, :children, self, :inverse => :parent
+    end
+
+    @model = Author
+  end
+
+  before do
+    @resource = @model.create(:name => 'Dan Kubb')
+    @resource.attributes = { :name => 'John Doe' }
+
+    @state = @resource.persisted_state
+    @state.should be_kind_of(DataMapper::Resource::State::Dirty)
+  end
+
+  after do
+    @resource.destroy
+  end
+
+  after do
+    @model.all.destroy!
+  end
+
+  describe '#commit' do
+    subject { @state.commit }
+
+    supported_by :all do
+      before do
+        @new_id = @resource.id = @resource.id.succ
+      end
+
+      it 'should return a Clean state' do
+        should eql(DataMapper::Resource::State::Clean.new(@resource))
+      end
+
+      it 'should update the resource' do
+        subject
+        @model.get!(*@resource.key).should == @resource
+      end
+
+      it 'should update the resource to the identity map if the key changed' do
+        identity_map = @resource.repository.identity_map(@model)
+        identity_map.should == { @resource.key => @resource }
+        subject
+        identity_map.should == { [ @new_id ] => @resource }
+      end
+    end
+  end
+
+  describe '#delete' do
+    subject { @state.delete }
+
+    supported_by :all do
+      before do
+        @resource.children = [ @resource.parent = @resource ]
+      end
+
+      it_should_behave_like 'It resets resource state'
+
+      it 'should return a Deleted state' do
+        should eql(DataMapper::Resource::State::Deleted.new(@resource))
+      end
+    end
+  end
+
+  describe '#get' do
+    before do
+      @loaded_value = 'John Doe'
+    end
+
+    it_should_behave_like 'Resource::State::Persisted#get'
+  end
+
+  describe '#rollback' do
+    subject { @state.rollback }
+
+    supported_by :all do
+      before do
+        @resource.children = [ @resource.parent = @resource ]
+      end
+
+      it_should_behave_like 'It resets resource state'
+
+      it 'should return a Clean state' do
+        should eql(DataMapper::Resource::State::Clean.new(@resource))
+      end
+    end
+  end
+
+  describe '#set' do
+    subject { @state.set(@key, @value) }
+
+    supported_by :all do
+      describe 'with attributes that keep the resource dirty' do
+        before do
+          @key   = @model.properties[:name]
+          @value = @key.get!(@resource)
+        end
+
+        it_should_behave_like 'A method that delegates to the superclass #set'
+
+        it 'should return a Dirty state' do
+          should equal(@state)
+        end
+      end
+
+      describe 'with attributes that make the resource clean' do
+        before do
+          @key   = @model.properties[:name]
+          @value = 'Dan Kubb'
+        end
+
+        it_should_behave_like 'A method that delegates to the superclass #set'
+
+        it 'should return a Clean state' do
+          should eql(DataMapper::Resource::State::Clean.new(@resource))
+        end
+      end
+    end
+  end
+end

data/spec/semipublic/resource/state/immutable_spec.rb

@@ -0,0 +1,99 @@
+require File.expand_path(File.join(File.dirname(__FILE__), '..', '..', '..', 'spec_helper'))
+
+describe DataMapper::Resource::State::Immutable do
+  before :all do
+    class ::Author
+      include DataMapper::Resource
+
+      property :id,     Serial
+      property :name,   String
+      property :active, Boolean, :default => true
+      property :coding, Boolean, :default => true
+
+      belongs_to :parent, self, :required => false
+    end
+
+    @model = Author
+  end
+
+  before do
+    @parent  = @model.create(:name => 'John Doe')
+
+    @resource = @model.create(:name => 'Dan Kubb', :parent => @parent)
+    @resource = @model.first(@model.key.zip(@resource.key).to_hash.merge(:fields => [ :name, :parent_id ]))
+
+    @state = @resource.persisted_state
+    @state.should be_kind_of(DataMapper::Resource::State::Immutable)
+  end
+
+  describe '#commit' do
+    subject { @state.commit }
+
+    supported_by :all do
+      it 'should be a no-op' do
+        should equal(@state)
+      end
+    end
+  end
+
+  describe '#delete' do
+    subject { @state.delete }
+
+    supported_by :all do
+      it 'should raise an exception' do
+        method(:subject).should raise_error(DataMapper::ImmutableError, 'Immutable resource cannot be deleted')
+      end
+    end
+  end
+
+  describe '#get' do
+    subject { @state.get(@key) }
+
+    supported_by :all do
+      describe 'with an unloaded property' do
+        before do
+          @key = @model.properties[:id]
+        end
+
+        it 'should raise an exception' do
+          method(:subject).should raise_error(DataMapper::ImmutableError, 'Immutable resource cannot be lazy loaded')
+        end
+      end
+
+      describe 'with an unloaded relationship' do
+        before do
+          @key = @model.relationships[:parent]
+        end
+
+        it 'should return value' do
+          should == @parent
+        end
+      end
+
+      describe 'with a loaded subject' do
+        before do
+          @key = @model.properties[:name]
+        end
+
+        it 'should return value' do
+          should == 'Dan Kubb'
+        end
+      end
+    end
+  end
+
+  describe '#set' do
+    before do
+      @key   = @model.properties[:name]
+      @value = @key.get!(@resource)
+    end
+
+    subject { @state.set(@key, @value) }
+
+    supported_by :all do
+      it 'should raise an exception' do
+        method(:subject).should raise_error(DataMapper::ImmutableError, 'Immutable resource cannot be modified')
+      end
+    end
+  end
+end

data/spec/semipublic/resource/state/transient_spec.rb

@@ -0,0 +1,128 @@
+require File.expand_path(File.join(File.dirname(__FILE__), '..', '..', '..', 'spec_helper'))
+
+describe DataMapper::Resource::State::Transient do
+  before :all do
+    class ::Author
+      include DataMapper::Resource
+
+      property :id,          Serial
+      property :name,        String
+      property :age,         Integer
+      property :description, Text,    :default => lambda { |resource, property| resource.name }
+      property :active,      Boolean, :default => true
+      property :coding,      Boolean, :default => true
+    end
+
+    @model = Author
+  end
+
+  before do
+    @resource = @model.new(:name => 'Dan Kubb', :coding => false)
+
+    @state = @resource.persisted_state
+    @state.should be_kind_of(DataMapper::Resource::State::Transient)
+  end
+
+  describe '#commit' do
+    subject { @state.commit }
+
+    supported_by :all do
+      it 'should return the expected Clean state' do
+        should eql(DataMapper::Resource::State::Clean.new(@resource))
+      end
+
+      it 'should set the serial property' do
+        method(:subject).should change(@resource, :id).from(nil)
+      end
+
+      it 'should set default values' do
+        method(:subject).should change { @model.properties[:active].get!(@resource) }.from(nil).to(true)
+      end
+
+      it 'should not set default values when they are already set' do
+        method(:subject).should_not change(@resource, :coding)
+      end
+
+      it 'should create the resource' do
+        subject
+        @model.get(*@resource.key).should == @resource
+      end
+
+      it 'should reset original attributes' do
+        expect do
+          @resource.persisted_state = subject
+        end.should change { @resource.original_attributes.dup }.from(@model.properties[:name] => nil, @model.properties[:coding] => nil).to({})
+      end
+
+      it 'should add the resource to the identity map' do
+        DataMapper.repository do |repository|
+          identity_map = repository.identity_map(@model)
+          identity_map.should be_empty
+          subject
+          identity_map.should == { @resource.key => @resource }
+        end
+      end
+    end
+  end
+
+  [ :delete, :rollback ].each do |method|
+    describe "##{method}" do
+      subject { @state.send(method) }
+
+      it 'should be a no-op' do
+        should equal(@state)
+      end
+    end
+  end
+
+  describe '#get' do
+    subject { @state.get(@key) }
+
+    describe 'with a set value' do
+      before do
+        @key = @model.properties[:coding]
+        @key.should be_loaded(@resource)
+      end
+
+      it 'should return value' do
+        should be(false)
+      end
+
+      it 'should be idempotent' do
+        should equal(subject)
+      end
+    end
+
+    describe 'with an unset value and no default value' do
+      before do
+        @key = @model.properties[:age]
+        @key.should_not be_loaded(@resource)
+        @key.should_not be_default
+      end
+
+      it 'should return nil' do
+        should be_nil
+      end
+
+      it 'should be idempotent' do
+        should equal(subject)
+      end
+    end
+
+    describe 'with an unset value and a default value' do
+      before do
+        @key = @model.properties[:description]
+        @key.should_not be_loaded(@resource)
+        @key.should be_default
+      end
+
+      it 'should return the name' do
+        should == 'Dan Kubb'
+      end
+
+      it 'should be idempotent' do
+        should equal(subject)
+      end
+    end
+  end
+end

data/spec/semipublic/resource/state_spec.rb

@@ -0,0 +1,226 @@
+require File.expand_path(File.join(File.dirname(__FILE__), '..', '..', 'spec_helper'))
+
+describe DataMapper::Resource::State do
+  before :all do
+    class ::Author
+      include DataMapper::Resource
+
+      property :id,      Serial
+      property :name,    String
+      property :private, Boolean, :accessor => :private
+
+      belongs_to :parent, self, :required => false
+    end
+
+    @model = Author
+  end
+
+  before do
+    @resource = @model.new(:name => 'Dan Kubb')
+
+    @state = DataMapper::Resource::State.new(@resource)
+  end
+
+  describe '.new' do
+    subject { DataMapper::Resource::State.new(@resource) }
+
+    it { should be_kind_of(DataMapper::Resource::State) }
+  end
+
+  describe '#==' do
+    subject { @state == @other }
+
+    describe 'with the same class and resource' do
+      before do
+        @other = DataMapper::Resource::State.new(@resource)
+      end
+
+      it { should be(true) }
+
+      it 'should be symmetric' do
+        should == (@other == @state)
+      end
+    end
+
+    describe 'with the same class and different resource' do
+      before do
+        @other = DataMapper::Resource::State.new(@model.new)
+      end
+
+      it { should be(false) }
+
+      it 'should be symmetric' do
+        should == (@other == @state)
+      end
+    end
+
+    describe 'with a different class and the same resource' do
+      before do
+        @other = DataMapper::Resource::State::Clean.new(@resource)
+      end
+
+      it 'should be true for a subclass' do
+        should be(true)
+      end
+
+      it 'should be symmetric' do
+        should == (@other == @state)
+      end
+    end
+
+    describe 'with a different class and different resource' do
+      before do
+        @other = DataMapper::Resource::State::Clean.new(@model.new)
+      end
+
+      it { should be(false) }
+
+      it 'should be symmetric' do
+        should == (@other == @state)
+      end
+    end
+  end
+
+  [ :commit, :delete, :rollback ].each do |method|
+    describe "##{method}" do
+      subject { @state.send(method) }
+
+      it 'should raise an exception' do
+        method(:subject).should raise_error(NotImplementedError, "DataMapper::Resource::State##{method} should be implemented")
+      end
+    end
+  end
+
+  describe '#eql?' do
+    subject { @state.eql?(@other) }
+
+    describe 'with the same class and resource' do
+      before do
+        @other = DataMapper::Resource::State.new(@resource)
+      end
+
+      it { should be(true) }
+
+      it 'should be symmetric' do
+        should == @other.eql?(@state)
+      end
+    end
+
+    describe 'with the same class and different resource' do
+      before do
+        @other = DataMapper::Resource::State.new(@model.new)
+      end
+
+      it { should be(false) }
+
+      it 'should be symmetric' do
+        should == @other.eql?(@state)
+      end
+    end
+
+    describe 'with a different class and the same resource' do
+      before do
+        @other = DataMapper::Resource::State::Clean.new(@resource)
+      end
+
+      it { should be(false) }
+
+      it 'should be symmetric' do
+        should == @other.eql?(@state)
+      end
+    end
+
+    describe 'with a different class and different resource' do
+      before do
+        @other = DataMapper::Resource::State::Clean.new(@model.new)
+      end
+
+      it { should be(false) }
+
+      it 'should be symmetric' do
+        should == @other.eql?(@state)
+      end
+    end
+  end
+
+  describe '#get' do
+    subject { @state.get(@key) }
+
+    describe 'with a Property subject' do
+      before do
+        @key = @model.properties[:name]
+      end
+
+      it 'should return the value' do
+        should == 'Dan Kubb'
+      end
+    end
+
+    describe 'with a Relationship subject' do
+      supported_by :all do
+        before do
+          # set the association
+          @resource.parent = @resource
+
+          @key = @model.relationships[:parent]
+        end
+
+        it 'should return the association' do
+          should == @resource
+        end
+      end
+    end
+  end
+
+  describe '#hash' do
+    subject { @state.hash }
+
+    it 'should be the object_id hash of the resource' do
+      should == @resource.object_id.hash
+    end
+  end
+
+  describe '#resource' do
+    subject { @state.resource }
+
+    it 'should return the resource' do
+      should equal(@resource)
+    end
+  end
+
+  describe '#set' do
+    subject { @state.set(@key, @value) }
+
+    describe 'with a Property subject' do
+      before do
+        @key   = @model.properties[:name]
+        @value = 'John Doe'
+      end
+
+      it 'should return a state object' do
+        should be_kind_of(DataMapper::Resource::State)
+      end
+
+      it 'should change the object attributes' do
+        method(:subject).should change(@resource, :attributes).from(:name => 'Dan Kubb').to(:name => 'John Doe')
+      end
+    end
+
+    describe 'with a Relationship subject' do
+      supported_by :all do
+        before do
+          @key   = @model.relationships[:parent]
+          @value = @resource
+        end
+
+        it 'should return a state object' do
+          should be_kind_of(DataMapper::Resource::State)
+        end
+
+        it 'should change the object relationship' do
+          method(:subject).should change(@resource, :parent).from(nil).to(@resource)
+        end
+      end
+    end
+  end
+end