dm-ambition 0.10.2

data/lib/dm-ambition/query.rb

@@ -0,0 +1,23 @@
+module DataMapper
+  module Ambition
+    module Query
+      @@sexps = {}
+
+      # TODO: spec and document this
+      # @api semipublic
+      def filter(negated = false, &block)
+        # TODO: benchmark Marshal versus just building the sexp on demand
+
+        # deep clone the sexp for multiple re-use
+        sexp = Marshal.load(@@sexps[block.to_s] ||= Marshal.dump(block.to_sexp))
+
+        processor = FilterProcessor.new(block.binding, model, negated)
+        processor.process(sexp)
+
+        merge(:conditions => processor.conditions)
+      end
+    end # module Query
+  end # module Ambition
+end # module DataMapper
+
+require Pathname(__FILE__).dirname.expand_path / 'query' / 'filter_processor'

data/lib/dm-ambition/version.rb

@@ -0,0 +1,5 @@
+module DataMapper
+  module Ambition
+    VERSION = '0.10.2'.freeze
+  end
+end

data/lib/dm-ambition.rb

@@ -0,0 +1,50 @@
+require 'pathname'
+
+dir = Pathname(__FILE__).dirname.expand_path + 'dm-ambition'
+
+require dir / 'collection'
+require dir / 'model'
+require dir / 'query'
+require dir / 'version'
+
+module DataMapper
+  class Collection
+    include Ambition::Collection
+  end
+
+  module Model
+    include Ambition::Model
+  end
+
+  class Query
+    include Ambition::Query
+  end
+
+  if Gem::Version.new(DataMapper::VERSION) < Gem::Version.new('0.10')
+    module Model
+      def self.descendants
+        Resource.descendants
+      end
+    end
+
+    class Collection
+      def new_collection(query, resources = nil, &block)
+        collection = self.class.new(query, &block)
+
+        if resources
+          collection.replace(resources)
+        end
+
+        collection
+      end
+    end
+
+    module Resource
+      def saved?
+        !new_record?
+      end
+
+      alias new? new_record?
+    end
+  end
+end

data/spec/public/collection_spec.rb

@@ -0,0 +1,138 @@
+require 'pathname'
+require Pathname(__FILE__).dirname.expand_path.parent + 'spec_helper'
+
+[ false, true ].each do |loaded|
+  describe DataMapper::Ambition::Collection do
+    cattr_accessor :loaded
+
+    self.loaded = loaded
+
+    before :all do
+      class ::User
+        include DataMapper::Resource
+
+        property :id,    Serial
+        property :name,  String
+        property :admin, Boolean, :default => false
+      end
+
+      if DataMapper.respond_to?(:auto_migrate!)
+        DataMapper.auto_migrate!
+      end
+    end
+
+    before :all do
+      @repository = DataMapper.repository(:default)
+      @model      = User
+      @query      = DataMapper::Query.new(@repository, @model)
+
+      @user  = @model.create(:name => 'Dan Kubb', :admin => true)
+      @other = @model.create(:name => 'Sam Smoot')
+
+      @subject = @model.all(@query)
+      @subject.to_a if loaded
+    end
+
+    it_should_behave_like 'it has public filter methods'
+
+    unless loaded || Gem::Version.new(DataMapper::VERSION) < Gem::Version.new('0.10')
+      [ :select, :find_all ].each do |method|
+        describe "##{method}", '(unloaded)' do
+          describe 'when matching resource prepended' do
+            before :all do
+              @subject.unshift(@other)
+              @return = @subject.send(method) { |u| u.admin == true }
+            end
+
+            it 'should return a Collection' do
+              @return.should be_kind_of(DataMapper::Collection)
+            end
+
+            it 'should not return self' do
+              @return.should_not equal(@subject)
+            end
+
+            it 'should not be a kicker' do
+              @return.should_not be_loaded
+            end
+
+            it 'should return expected values' do
+              @return.should == [ @user ]
+            end
+          end
+
+          describe 'when matching resource appended' do
+            before :all do
+              @subject << @other
+              @return = @subject.send(method) { |u| u.admin == true }
+            end
+
+            it 'should return a Collection' do
+              @return.should be_kind_of(DataMapper::Collection)
+            end
+
+            it 'should not return self' do
+              @return.should_not equal(@subject)
+            end
+
+            it 'should not be a kicker' do
+              @return.should_not be_loaded
+            end
+
+            it 'should return expected values' do
+              @return.should == [ @user ]
+            end
+          end
+        end
+      end
+
+      describe '#reject', '(unloaded)' do
+        describe 'when matching resource prepended' do
+          before :all do
+            @subject.unshift(@other)
+            @return = @subject.reject { |u| u.admin != true }
+          end
+
+          it 'should return a Collection' do
+            @return.should be_kind_of(DataMapper::Collection)
+          end
+
+          it 'should not return self' do
+            @return.should_not equal(@subject)
+          end
+
+          it 'should not be a kicker' do
+            @return.should_not be_loaded
+          end
+
+          it 'should return expected values' do
+            @return.should == [ @user ]
+          end
+        end
+
+        describe 'when matching resource appended' do
+          before :all do
+            @subject << @other
+            @return = @subject.reject { |u| u.admin != true }
+          end
+
+          it 'should return a Collection' do
+            @return.should be_kind_of(DataMapper::Collection)
+          end
+
+          it 'should not return self' do
+            @return.should_not equal(@subject)
+          end
+
+          it 'should not be a kicker' do
+            @return.should_not be_loaded
+          end
+
+          it 'should return expected values' do
+            @return.should == [ @user ]
+          end
+        end
+      end
+    end
+  end
+end

data/spec/public/model_spec.rb

@@ -0,0 +1,34 @@
+require 'pathname'
+require Pathname(__FILE__).dirname.expand_path.parent + 'spec_helper'
+
+describe DataMapper::Ambition::Model do
+  def self.loaded
+    false
+  end
+
+  before :all do
+    class ::User
+      include DataMapper::Resource
+
+      property :id,    Serial
+      property :name,  String
+      property :admin, Boolean, :default => false
+    end
+
+    if DataMapper.respond_to?(:auto_migrate!)
+      DataMapper.auto_migrate!
+    end
+  end
+
+  before :all do
+    @repository = DataMapper.repository(:default)
+    @model      = User
+
+    @user  = @model.create(:name => 'Dan Kubb', :admin => true)
+    @other = @model.create(:name => 'Sam Smoot')
+
+    @subject = @model
+  end
+
+  it_should_behave_like 'it has public filter methods'
+end

data/spec/public/shared/filter_shared_spec.rb

@@ -0,0 +1,221 @@
+share_examples_for 'it has public filter methods' do
+  [ :select, :find_all ].each do |method|
+    it { @subject.should respond_to(method) }
+
+    describe "##{method}", ('(loaded)' if loaded) do
+      describe 'with simple conditions' do
+        before :all do
+          @return = @subject.send(method) { |u| u.name == 'Dan Kubb' }
+        end
+
+        it 'should return a Collection' do
+          @return.should be_kind_of(DataMapper::Collection)
+        end
+
+        it 'should not return self' do
+          @return.should_not equal(@subject)
+        end
+
+        unless loaded
+          it 'should not be a kicker' do
+            @subject.respond_to?(:loaded) and @subject.should_not be_loaded
+          end
+        end
+
+        it 'should return expected values' do
+          @return.should == [ @user ]
+        end
+      end
+
+      describe 'with OR + AND conditions' do
+        before :all do
+          @return = @subject.send(method) { |u| (u.name == 'Dan Kubb' || u.name == 'Sam Smoot') && u.admin == true }
+        end
+
+        it 'should return a Collection' do
+          @return.should be_kind_of(DataMapper::Collection)
+        end
+
+        it 'should not return self' do
+          @return.should_not equal(@subject)
+        end
+
+        unless loaded
+          it 'should not be a kicker' do
+            @subject.respond_to?(:loaded) and @subject.should_not be_loaded
+          end
+        end
+
+        it 'should return expected values' do
+          @return.should == [ @user ]
+        end
+      end
+
+      describe 'with AND + OR conditions' do
+        before :all do
+          @return = @subject.send(method) { |u| (u.admin == true && u.name == 'Dan Kubb') || u.name == 'Sam Smoot' }
+        end
+
+        it 'should return a Collection' do
+          @return.should be_kind_of(DataMapper::Collection)
+        end
+
+        it 'should not return self' do
+          @return.should_not equal(@subject)
+        end
+
+        unless loaded
+          it 'should not be a kicker' do
+            @subject.respond_to?(:loaded) and @subject.should_not be_loaded
+          end
+        end
+
+        it 'should return expected values' do
+          @return.should == [ @user, @other ]
+        end
+      end
+    end
+  end
+
+  [ :detect, :find ].each do |method|
+    it { @subject.should respond_to(method) }
+
+    describe "##{method}", ('(loaded)' if loaded) do
+      describe 'with simple conditions' do
+        before :all do
+          @return = @subject.send(method) { |u| u.name == 'Dan Kubb' }
+        end
+
+        it 'should return a Resource' do
+          @return.should be_kind_of(DataMapper::Resource)
+        end
+
+        unless loaded
+          it 'should not be a kicker' do
+            @subject.respond_to?(:loaded) and @subject.should_not be_loaded
+          end
+        end
+
+        it 'should return expected value' do
+          @return.should == @user
+        end
+      end
+
+      describe 'with OR + AND conditions' do
+        before :all do
+          @return = @subject.send(method) { |u| (u.name == 'Dan Kubb' || u.name == 'Sam Smoot') && u.admin == true }
+        end
+
+        it 'should return a Resource' do
+          @return.should be_kind_of(DataMapper::Resource)
+        end
+
+        unless loaded
+          it 'should not be a kicker' do
+            @subject.respond_to?(:loaded) and @subject.should_not be_loaded
+          end
+        end
+
+        it 'should return expected value' do
+          @return.should == @user
+        end
+      end
+
+      describe 'with AND + OR conditions' do
+        before :all do
+          @return = @subject.send(method) { |u| (u.admin == true && u.name == 'Dan Kubb') || u.name == 'Sam Smoot' }
+        end
+
+        it 'should return a Resource' do
+          @return.should be_kind_of(DataMapper::Resource)
+        end
+
+        unless loaded
+          it 'should not be a kicker' do
+            @subject.respond_to?(:loaded) and @subject.should_not be_loaded
+          end
+        end
+
+        it 'should return expected value' do
+          @return.should == @user
+        end
+      end
+    end
+  end
+
+  it { @subject.should respond_to(:reject) }
+
+  describe '#reject', ('(loaded)' if loaded) do
+    describe 'with simple conditions' do
+      before :all do
+        @return = @subject.reject { |u| u.name != 'Dan Kubb' }
+      end
+
+      it 'should return a Collection' do
+        @return.should be_kind_of(DataMapper::Collection)
+      end
+
+      it 'should not return self' do
+        @return.should_not equal(@subject)
+      end
+
+      unless loaded
+        it 'should not be a kicker' do
+          @subject.respond_to?(:loaded) and @subject.should_not be_loaded
+        end
+      end
+
+      it 'should return expected values' do
+        @return.should == [ @user ]
+      end
+    end
+
+    describe 'with OR + AND conditions' do
+      before :all do
+        @return = @subject.reject { |u| (u.name == 'Dan Kubb' || u.name == 'Sam Smoot') && u.admin == true }
+      end
+
+      it 'should return a Collection' do
+        @return.should be_kind_of(DataMapper::Collection)
+      end
+
+      it 'should not return self' do
+        @return.should_not equal(@subject)
+      end
+
+      unless loaded
+        it 'should not be a kicker' do
+          @subject.respond_to?(:loaded) and @subject.should_not be_loaded
+        end
+      end
+
+      it 'should return expected values' do
+        @return.should == [ @other ]
+      end
+    end
+
+    describe 'with AND + OR conditions' do
+      before :all do
+        @return = @subject.reject { |u| (u.admin == true && u.name == 'Dan Kubb') || u.name == 'Sam Smoot' }
+      end
+
+      it 'should return a Collection' do
+        @return.should be_kind_of(DataMapper::Collection)
+      end
+
+      it 'should not return self' do
+        @return.should_not equal(@subject)
+      end
+
+      unless loaded
+        it 'should not be a kicker' do
+          @subject.respond_to?(:loaded) and @subject.should_not be_loaded
+        end
+      end
+
+      it 'should return expected values' do
+        @return.should == []
+      end
+    end
+  end
+end