ardm-core 1.2.1

data/spec/public/resource_spec.rb

@@ -0,0 +1,284 @@
+require 'spec_helper'
+
+describe DataMapper::Resource do
+  before :all do
+    module ::Blog
+      class User
+        include DataMapper::Resource
+
+        property :name,        String, :key => true
+        property :age,         Integer
+        property :summary,     Text
+        property :description, Text
+        property :admin,       Boolean, :accessor => :private
+
+        belongs_to :parent, self, :required => false
+        has n, :children, self, :inverse => :parent
+
+        belongs_to :referrer, self, :required => false
+        has n, :comments
+
+        # FIXME: figure out a different approach than stubbing things out
+        def comment=(*)
+          # do nothing with comment
+        end
+      end
+
+      class Author < User; end
+
+      class Comment
+        include DataMapper::Resource
+
+        property :id,   Serial
+        property :body, Text
+
+        belongs_to :user
+      end
+
+      class Article
+        include DataMapper::Resource
+
+        property :id,   Serial
+        property :body, Text
+
+        has n, :paragraphs
+      end
+
+      class Paragraph
+        include DataMapper::Resource
+
+        property :id,   Serial
+        property :text, String
+
+        belongs_to :article
+      end
+    end
+
+    class ::Default
+      include DataMapper::Resource
+
+      property :name, String, :key => true, :default => 'a default value'
+    end
+    DataMapper.finalize
+
+    @user_model      = Blog::User
+    @author_model    = Blog::Author
+    @comment_model   = Blog::Comment
+    @article_model   = Blog::Article
+    @paragraph_model = Blog::Paragraph
+  end
+
+  supported_by :all do
+    before :all do
+      user = @user_model.create(:name => 'dbussink', :age => 25, :description => 'Test')
+
+      @user = @user_model.get(*user.key)
+    end
+
+    it_should_behave_like 'A public Resource'
+    it_should_behave_like 'A Resource supporting Strategic Eager Loading'
+
+    it 'A resource should respond to raise_on_save_failure' do
+      @user.should respond_to(:raise_on_save_failure)
+    end
+
+    describe '#raise_on_save_failure' do
+      after do
+        # reset to the default value
+        reset_raise_on_save_failure(@user_model)
+        reset_raise_on_save_failure(@user)
+      end
+
+      subject { @user.raise_on_save_failure }
+
+      describe 'when model.raise_on_save_failure has not been set' do
+        it { should be(false) }
+      end
+
+      describe 'when model.raise_on_save_failure has been set to true' do
+        before do
+          @user_model.raise_on_save_failure = true
+        end
+
+        it { should be(true) }
+      end
+
+      describe 'when resource.raise_on_save_failure has been set to true' do
+        before do
+          @user.raise_on_save_failure = true
+        end
+
+        it { should be(true) }
+      end
+    end
+
+    it 'A model should respond to raise_on_save_failure=' do
+      @user_model.should respond_to(:raise_on_save_failure=)
+    end
+
+    describe '#raise_on_save_failure=' do
+      after do
+        # reset to the default value
+        @user_model.raise_on_save_failure = false
+      end
+
+      subject { @user_model.raise_on_save_failure = @value }
+
+      describe 'with a true value' do
+        before do
+          @value = true
+        end
+
+        it { should be(true) }
+
+        it 'should set raise_on_save_failure' do
+          method(:subject).should change {
+            @user_model.raise_on_save_failure
+          }.from(false).to(true)
+        end
+      end
+
+      describe 'with a false value' do
+        before do
+          @value = false
+        end
+
+        it { should be(false) }
+
+        it 'should set raise_on_save_failure' do
+          method(:subject).should_not change {
+            @user_model.raise_on_save_failure
+          }
+        end
+      end
+    end
+
+    [ :save, :save! ].each do |method|
+      describe "##{method}" do
+        subject { @user.__send__(method) }
+
+        describe 'when raise_on_save_failure is true' do
+          before do
+            @user.raise_on_save_failure = true
+          end
+
+          describe 'and it is a savable resource' do
+            it { should be(true) }
+          end
+
+          describe 'and it is an invalid resource' do
+            before do
+              @user.name = nil  # name is required
+            end
+
+            it 'should raise an exception' do
+              method(:subject).should raise_error(DataMapper::SaveFailureError, "Blog::User##{method} returned false, Blog::User was not saved") { |error|
+                error.resource.should equal(@user)
+              }
+            end
+          end
+        end
+      end
+    end
+
+    [ :update, :update! ].each do |method|
+      describe 'with attributes where one is a foreign key' do
+        before :all do
+          rescue_if @skip do
+            @dkubb = @user.referrer = @user_model.create(:name => 'dkubb', :age => 33)
+            @user.save
+            @user = @user_model.get(*@user.key)
+            @user.referrer.should == @dkubb
+
+            @solnic = @user_model.create(:name => 'solnic', :age => 28)
+
+            @attributes = {}
+
+            relationship = @user_model.relationships[:referrer]
+            relationship.child_key.to_a.each_with_index do |k, i|
+              @attributes[k.name] = relationship.parent_key.to_a[i].get(@solnic)
+            end
+
+            @return = @user.__send__(method, @attributes)
+          end
+        end
+
+        it 'should return true' do
+          @return.should be(true)
+        end
+
+        it 'should update attributes of Resource' do
+          @attributes.each { |key, value| @user.__send__(key).should == value }
+        end
+
+        it 'should persist the changes' do
+          resource = @user_model.get(*@user.key)
+          @attributes.each { |key, value| resource.__send__(key).should == value }
+        end
+
+        it 'should return correct parent' do
+          resource = @user_model.get(*@user.key)
+          resource.referrer.should == @solnic
+        end
+      end
+    end
+
+    describe '#attribute_get' do
+      subject { object.attribute_get(name) }
+
+      let(:object) { @user }
+
+      context 'with a known property' do
+        let(:name) { :name }
+
+        it 'returns the attribute value' do
+          should == 'dbussink'
+        end
+      end
+
+      context 'with an unknown property' do
+        let(:name) { :unknown }
+
+        it 'returns nil' do
+          should be_nil
+        end
+      end
+    end
+
+    describe '#attribute_set' do
+      subject { object.attribute_set(name, value) }
+
+      let(:object) { @user.dup }
+
+      context 'with a known property' do
+        let(:name)  { :name   }
+        let(:value) { 'dkubb' }
+
+        it 'sets the attribute' do
+          expect { subject }.to change { object.name }.
+            from('dbussink').
+            to('dkubb')
+        end
+
+        it 'makes the object dirty' do
+          expect { subject }.to change { object.dirty? }.
+            from(false).
+            to(true)
+        end
+      end
+
+      context 'with an unknown property' do
+        let(:name)  { :unknown              }
+        let(:value) { mock('Unknown Value') }
+
+        it 'does not set the attribute' do
+          expect { subject }.to_not change { object.attributes.dup }
+        end
+
+        it 'does not make the object dirty' do
+          expect { subject }.to_not change { object.dirty? }
+        end
+      end
+    end
+  end
+end

data/spec/public/sel_spec.rb

@@ -0,0 +1,53 @@
+require 'spec_helper'
+
+describe 'SEL', 'with STI subclasses' do
+  before :all do
+    module ::Blog
+      class Author
+        include DataMapper::Resource
+
+        property :id,   Serial
+        property :name, String
+
+        has n, :messages
+      end
+
+      class Message
+        include DataMapper::Resource
+
+        property :id,    Serial
+        property :type,  Discriminator
+        property :title, String, :required => true
+
+        belongs_to :author
+      end
+
+      class Article < Message; end
+      class Comment < Message; end
+    end
+
+    DataMapper.finalize
+
+    @author_model  = Blog::Author
+    @message_model = Blog::Message
+    @article_model = Blog::Article
+    @comment_model = Blog::Comment
+  end
+
+  supported_by :all do
+    before :all do
+      author1 = @author_model.create(:name => 'Dan Kubb')
+      author2 = @author_model.create(:name => 'Sindre Aarsaether')
+
+      @article_model.create(:title => 'SEL',               :author => author1)
+      @article_model.create(:title => 'STI',               :author => author1)
+      @comment_model.create(:title => 'SEL and STI error', :author => author2)
+    end
+
+    it 'should allow STI loading of mixed relationships' do
+      lambda {
+        @message_model.all.each { |message| message.author }
+      }.should_not raise_error(ArgumentError)
+    end
+  end
+end

data/spec/public/setup_spec.rb

@@ -0,0 +1,145 @@
+require 'spec_helper'
+
+describe DataMapper do
+  describe '.setup' do
+    describe 'using connection string' do
+      before :all do
+        @return = DataMapper.setup(:setup_test, 'in_memory://user:pass@hostname:1234/path?foo=bar&baz=foo#fragment')
+
+        @options = @return.options
+      end
+
+      after :all do
+        DataMapper::Repository.adapters.delete(@return.name)
+      end
+
+      it 'should return an Adapter' do
+        @return.should be_kind_of(DataMapper::Adapters::AbstractAdapter)
+      end
+
+      it 'should set up the repository' do
+        DataMapper.repository(:setup_test).adapter.should equal(@return)
+      end
+
+      {
+        :adapter  => 'in_memory',
+        :user     => 'user',
+        :password => 'pass',
+        :host     => 'hostname',
+        :port     => 1234,
+        :path     => '/path',
+        :fragment => 'fragment'
+      }.each do |key, val|
+        it "should extract the #{key.inspect} option from the uri" do
+          @options[key].should == val
+        end
+      end
+
+      it 'should alias the scheme of the uri as the adapter' do
+        @options[:scheme].should == @options[:adapter]
+      end
+
+      it 'should leave the query param intact' do
+        @options[:query].should == 'foo=bar&baz=foo'
+      end
+
+      it 'should extract the query param as top-level options' do
+        @options[:foo].should == 'bar'
+        @options[:baz].should == 'foo'
+      end
+    end
+
+    describe 'using options' do
+      before :all do
+        @return = DataMapper.setup(:setup_test, :adapter => :in_memory, :foo => 'bar')
+
+        @options = @return.options
+      end
+
+      after :all do
+        DataMapper::Repository.adapters.delete(@return.name)
+      end
+
+      it 'should return an Adapter' do
+        @return.should be_kind_of(DataMapper::Adapters::AbstractAdapter)
+      end
+
+      it 'should set up the repository' do
+        DataMapper.repository(:setup_test).adapter.should equal(@return)
+      end
+
+      {
+        :adapter => :in_memory,
+        :foo     => 'bar'
+      }.each do |key, val|
+        it "should set the #{key.inspect} option" do
+          @options[key].should == val
+        end
+      end
+    end
+
+    describe 'using invalid options' do
+      it 'should raise an exception' do
+        lambda {
+          DataMapper.setup(:setup_test, :invalid)
+        }.should raise_error(ArgumentError, '+options+ should be Hash or Addressable::URI or String, but was Symbol')
+      end
+    end
+
+    describe 'using an instance of an adapter' do
+      before :all do
+        @adapter = DataMapper::Adapters::InMemoryAdapter.new(:setup_test)
+
+        @return = DataMapper.setup(@adapter)
+      end
+
+      after :all do
+        DataMapper::Repository.adapters.delete(@return.name)
+      end
+
+      it 'should return an Adapter' do
+        @return.should be_kind_of(DataMapper::Adapters::AbstractAdapter)
+      end
+
+      it 'should set up the repository' do
+        DataMapper.repository(:setup_test).adapter.should equal(@return)
+      end
+
+      it 'should use the adapter given' do
+        @return.should == @adapter
+      end
+
+      it 'should use the name given to the adapter' do
+        @return.name.should == @adapter.name
+      end
+    end
+
+    supported_by :postgres, :mysql, :sqlite3, :sqlserver do
+      { :path => :database, :user => :username }.each do |original_key, new_key|
+        describe "using #{new_key.inspect} option" do
+          before :all do
+            @return = DataMapper.setup(:setup_test, :adapter => @adapter.options[:adapter], new_key => @adapter.options[original_key])
+
+            @options = @return.options
+          end
+
+          after :all do
+            DataMapper::Repository.adapters.delete(@return.name)
+          end
+
+          it 'should return an Adapter' do
+            @return.should be_kind_of(DataMapper::Adapters::AbstractAdapter)
+          end
+
+          it 'should set up the repository' do
+            DataMapper.repository(:setup_test).adapter.should equal(@return)
+          end
+
+          it "should set the #{new_key.inspect} option" do
+            @options[new_key].should == @adapter.options[original_key]
+          end
+        end
+      end
+    end
+  end
+end