kaminari 0.12.4 → 0.13.0

data/spec/helpers/tags_spec.rb

@@ -1,4 +1,4 @@
-require File.expand_path('../spec_helper', File.dirname(__FILE__))
+require 'spec_helper'
 include Kaminari::Helpers
 
 describe 'Kaminari::Helpers' do

data/spec/models/active_record_relation_methods_spec.rb

@@ -1,4 +1,4 @@
-require File.expand_path('../spec_helper', File.dirname(__FILE__))
+require 'spec_helper'
 
 describe Kaminari::ActiveRecordRelationMethods do
   describe '#total_count' do

data/spec/models/array_spec.rb

@@ -1,10 +1,18 @@
-require File.expand_path('../spec_helper', File.dirname(__FILE__))
+require 'spec_helper'
 
 describe Kaminari::PaginatableArray do
+  it { should have(0).items }
+
+  context 'specifying limit and offset when initializing' do
+    subject { Kaminari::PaginatableArray.new((1..100).to_a, :limit => 10, :offset => 20) }
+    its(:current_page) { should == 3 }
+  end
+
   let(:array) { Kaminari::PaginatableArray.new((1..100).to_a) }
   describe '#page' do
     shared_examples_for 'the first page of array' do
       it { should have(25).users }
+      its(:current_page) { should == 1 }
       its(:first) { should == 1 }
     end
 
@@ -20,6 +28,7 @@ describe Kaminari::PaginatableArray do
     context 'page 2' do
       subject { array.page 2 }
       it { should have(25).users }
+      its(:current_page) { should == 2 }
       its(:first) { should == 26 }
     end
 
@@ -102,4 +111,11 @@ describe Kaminari::PaginatableArray do
       its(:count) { should == 25 }
     end
   end
+
+  context 'when setting total count explicitly' do
+    subject { Kaminari::PaginatableArray.new((1..10).to_a, :total_count => 9999).page(5).per(10) }
+    it { should have(10).items }
+    its(:first) { should == 1 }
+    its(:total_count) { should == 9999 }
+  end
 end

data/spec/models/data_mapper_spec.rb

@@ -0,0 +1,181 @@
+require 'spec_helper'
+require 'dm-core'
+require 'dm-migrations'
+require 'dm-aggregates'
+require 'kaminari/models/data_mapper_extension'
+
+describe Kaminari::DataMapperExtension do
+  before :all do
+    DataMapper.setup(:default, 'sqlite::memory:')
+
+    class Worker
+      include ::DataMapper::Resource
+
+      property :id, Serial
+      property :name, String, :required => true
+      property :age, Integer, :required => true
+
+      has n, :projects, :through => Resource
+    end
+
+    class Project
+      include ::DataMapper::Resource
+
+      property :id, Serial
+      property :name, String, :required => true
+
+      has n, :workers, :through => Resource
+    end
+
+    DataMapper.finalize
+    DataMapper.auto_migrate!
+
+    300.times do |i|
+      Worker.create(:name => "Worker#{i}", :age => i)
+    end
+
+    worker0 = Worker[0]
+    50.times do |i|
+      worker0.projects << Project.create(:name => "Project#{i}")
+    end
+    worker0.projects.save
+  end
+
+  describe 'Collection' do
+    subject{ Worker.all }
+    it { should respond_to(:page) }
+    it { should_not respond_to(:per) }
+  end
+
+  describe 'Model' do
+    subject{ Worker }
+    it { should respond_to(:page) }
+    it { should respond_to(:default_per_page) }
+    it { should_not respond_to(:per) }
+  end
+
+  describe '#page' do
+    context 'page 0' do
+      subject { Worker.all(:age.gte => 200).page 0 }
+      it { should be_a DataMapper::Collection }
+      its(:current_page) { should == 1 }
+      its('query.limit') { should == 25 }
+      its('query.offset') { should == 0 }
+      its(:total_count) { should == Worker.count(:age.gte => 200) }
+      its(:num_pages) { should == 4 }
+    end
+
+    context 'page 1' do
+      subject { Worker.all(:age.gte => 0).page 1 }
+      it { should be_a DataMapper::Collection }
+      its(:current_page) { should == 1 }
+      its('query.limit') { should == 25 }
+      its('query.offset') { should == 0 }
+      its(:total_count) { should == 300 }
+      its(:num_pages) { should == 12 }
+    end
+
+    context 'page 2' do
+      subject { Worker.page 2 }
+      it { should be_a DataMapper::Collection }
+      its(:current_page) { should == 2 }
+      its(:limit_value) { should == 25 }
+      its('query.limit') { should == 25 }
+      its('query.offset') { should == 25 }
+      its(:total_count) { should == 300 }
+      its(:num_pages) { should == 12 }
+    end
+
+    context 'page "foobar"' do
+      subject { Worker.page 'foobar' }
+      it { should be_a DataMapper::Collection }
+      its(:current_page) { should == 1 }
+      its('query.limit') { should == 25 }
+      its('query.offset') { should == 0 }
+      its(:total_count) { should == 300 }
+      its(:num_pages) { should == 12 }
+    end
+
+    context 'with criteria before' do
+      subject { Worker.all(:age.gt => 100).page 2 }
+      it { should be_a DataMapper::Collection }
+      its(:current_page) { should == 2 }
+      its('query.limit') { should == 25 }
+      its('query.offset') { should == 25 }
+      its(:total_count) { should == Worker.count(:age.gt => 100) }
+      its(:num_pages) { should == 8 }
+    end
+
+    context 'with criteria after' do
+      subject { Worker.page(2).all(:age.gt => 100) }
+      it { should be_a DataMapper::Collection }
+      its(:current_page) { should == 2 }
+      its('query.limit') { should == 25 }
+      its('query.offset') { should == 25 }
+      its(:total_count) { should == Worker.count(:age.gt => 100) }
+      its(:num_pages) { should == 8 }
+    end
+  end
+
+  describe '#per' do
+    context 'on simple query' do
+		subject { Worker.page(2).per(10) }
+    it { should be_a DataMapper::Collection }
+		its(:current_page) { should == 2 }
+		its('query.limit') { should == 10 }
+    its(:limit_value) { should == 10 }
+		its('query.offset') { should == 10 }
+		its(:total_count) { should == 300 }
+		its(:num_pages) { should == 30 }
+    end
+
+    context 'on query with condition' do
+      subject { Worker.page(5).all(:age.lte => 100).per(13) }
+      its(:current_page) { should == 5 }
+      its('query.limit') { should == 13 }
+      its('query.offset') { should == 52 }
+      its(:total_count) { should == 101 }
+      its(:num_pages) { should == 8 }
+    end
+
+    context 'on query with order' do
+      subject { Worker.page(5).all(:age.lte => 100, :order => [:age.asc]).per(13) }
+      it('includes worker with age 52') { should include(Worker.first(:age => 52)) }
+      it('does not include worker with age 51') { should_not include(Worker.first(:age => 51)) }
+      it('includes worker with age 52') { should include(Worker.first(:age => 64)) }
+      it('does not include worker with age 51') { should_not include(Worker.first(:age => 65)) }
+      its(:current_page) { should == 5 }
+      its('query.limit') { should == 13 }
+      its('query.offset') { should == 52 }
+      its(:total_count) { should == 101 }
+      its(:num_pages) { should == 8 }
+    end
+
+    context 'on chained queries' do
+      subject { Worker.all(:age.gte => 50).page(3).all(:age.lte => 100).per(13) }
+      its(:current_page) { should == 3 }
+      its('query.limit') { should == 13 }
+      its('query.offset') { should == 26 }
+      its(:total_count) { should == 51 }
+      its(:num_pages) { should == 4 }
+    end
+
+    context 'on query on association' do
+      subject { Worker[0].projects.page(3).all(:name.like => 'Project%').per(5) }
+      its(:current_page) { should == 3 }
+      its('query.limit') { should == 5 }
+      its('query.offset') { should == 10 }
+      its(:total_count) { should == 50 }
+      its(:num_pages) { should == 10 }
+    end
+
+    context 'on query with association conditions' do
+      subject { Worker.page(3).all(:projects => Project.all).per(5) }
+      its(:current_page) { should == 3 }
+      its('query.limit') { should == 5 }
+      its('query.offset') { should == 10 }
+      its(:total_count) { should == 50 }
+      its(:num_pages) { should == 10 }
+    end
+  end
+end

data/spec/models/default_per_page_spec.rb

@@ -1,4 +1,4 @@
-require File.expand_path('../spec_helper', File.dirname(__FILE__))
+require 'spec_helper'
 
 describe 'default per_page' do
   describe 'AR::Base' do

data/spec/models/mongo_mapper_spec.rb

@@ -1,18 +1,21 @@
-require File.expand_path('../spec_helper', File.dirname(__FILE__))
+require 'spec_helper'
 require 'mongo_mapper'
-require File.expand_path('../../lib/kaminari/models/mongo_mapper_extension', File.dirname(__FILE__))
+require 'kaminari/models/mongo_mapper_extension'
 
 describe Kaminari::MongoMapperExtension do
-  before :all do
-    MongoMapper.connection = Mongo::Connection.new('localhost', 27017)
-    MongoMapper.database = "kaminari_test"
-    class Developer
-      include ::MongoMapper::Document
-      key :salary, Integer
-    end
-  end
   before do
-    stub(subject).count { 300 } # in order to avoid DB access...
+    begin
+      MongoMapper.connection = Mongo::Connection.new('localhost', 27017)
+      MongoMapper.database = "kaminari_test"
+      class Developer
+        include ::MongoMapper::Document
+        key :salary, Integer
+      end
+
+      stub(subject).count { 300 } # in order to avoid DB access...
+    rescue Mongo::ConnectionFailure
+      pending 'can not connect to MongoDB'
+    end
   end
 
   describe '#page' do

data/spec/models/mongoid_spec.rb

@@ -1,6 +1,6 @@
-require File.expand_path('../spec_helper', File.dirname(__FILE__))
+require 'spec_helper'
 require 'mongoid'
-require File.expand_path('../../lib/kaminari/models/mongoid_extension', File.dirname(__FILE__))
+require 'kaminari/models/mongoid_extension'
 
 describe Kaminari::MongoidExtension do
   before :all do
@@ -9,11 +9,12 @@ describe Kaminari::MongoidExtension do
       field :salary, :type => Integer
     end
   end
-  before do
-    stub(subject).count { 300 } # in order to avoid DB access...
-  end
 
   describe '#page' do
+    before do
+      stub(subject).count { 300 } # in order to avoid DB access...
+    end
+
     context 'page 1' do
       subject { Developer.page 1 }
       it { should be_a Mongoid::Criteria }
@@ -58,10 +59,13 @@ describe Kaminari::MongoidExtension do
       its(:num_pages) { should == 12 }
       it { should skip 25 }
     end
-
   end
 
   describe '#per' do
+    before do
+      stub(subject).count { 300 } # in order to avoid DB access...
+    end
+
     subject { Developer.page(2).per(10) }
     it { should be_a Mongoid::Criteria }
     its(:current_page) { should == 2 }
@@ -69,4 +73,57 @@ describe Kaminari::MongoidExtension do
     its(:num_pages) { should == 30 }
     it { should skip 10 }
   end
+
+  describe '#page in embedded documents' do
+    before :all do
+      class MongoDeveloper
+        include ::Mongoid::Document
+        field :salary, :type => Integer
+        embeds_many :frameworks
+      end
+
+      class Framework
+        include ::Mongoid::Document
+        field :name, :type => String
+        field :language, :type => String
+        embedded_in :mongo_developer
+      end
+    end
+
+    before :all do
+      @mongo_developer = MongoDeveloper.new
+      @mongo_developer.frameworks.new(:name => "rails", :language => "ruby")
+      @mongo_developer.frameworks.new(:name => "merb", :language => "ruby")
+      @mongo_developer.frameworks.new(:name => "sinatra", :language => "ruby")
+      @mongo_developer.frameworks.new(:name => "cakephp", :language => "php")
+      @mongo_developer.frameworks.new(:name => "tornado", :language => "python")
+    end
+
+    context 'page 1' do
+      subject { @mongo_developer.frameworks.page(1).per(1) }
+      it { should be_a Mongoid::Criteria }
+      its(:total_count) { should == 5 }
+      its(:limit_value) { should == 1 }
+      its(:current_page) { should == 1 }
+      its(:num_pages) { should == 5 }
+    end
+
+    context 'with criteria after' do
+      subject { @mongo_developer.frameworks.page(1).per(2).where(:language => "ruby") }
+      it { should be_a Mongoid::Criteria }
+      its(:total_count) { should == 3 }
+      its(:limit_value) { should == 2 }
+      its(:current_page) { should == 1 }
+      its(:num_pages) { should == 2 }
+    end
+
+    context 'with criteria before' do
+      subject { @mongo_developer.frameworks.where(:language => "ruby").page(1).per(2) }
+      it { should be_a Mongoid::Criteria }
+      its(:total_count) { should == 3 }
+      its(:limit_value) { should == 2 }
+      its(:current_page) { should == 1 }
+      its(:num_pages) { should == 2 }
+    end
+  end
 end

data/spec/models/scopes_spec.rb

@@ -1,149 +1,163 @@
-require File.expand_path('../spec_helper', File.dirname(__FILE__))
+require 'spec_helper'
+
+shared_examples_for 'the first page' do
+  it { should have(25).users }
+  its('first.name') { should == 'user001' }
+end
+
+shared_examples_for 'blank page' do
+  it { should have(0).users }
+end
 
 describe Kaminari::ActiveRecordExtension do
   before :all do
     1.upto(100) {|i| User.create! :name => "user#{'%03d' % i}", :age => (i / 10)}
+    1.upto(100) {|i| GemDefinedModel.create! :name => "user#{'%03d' % i}", :age => (i / 10)}
   end
 
-  describe '#page' do
-    shared_examples_for 'the first page' do
-      it { should have(25).users }
-      its('first.name') { should == 'user001' }
-    end
-
-    shared_examples_for 'blank page' do
-      it { should have(0).users }
-    end
-
-    context 'page 1' do
-      subject { User.page 1 }
-      it_should_behave_like 'the first page'
-    end
-
-    context 'page 2' do
-      subject { User.page 2 }
-      it { should have(25).users }
-      its('first.name') { should == 'user026' }
-    end
-
-    context 'page without an argument' do
-      subject { User.page }
-      it_should_behave_like 'the first page'
-    end
-
-    context 'page < 1' do
-      subject { User.page 0 }
-      it_should_behave_like 'the first page'
-    end
-
-    context 'page > max page' do
-      subject { User.page 5 }
-      it_should_behave_like 'blank page'
-    end
-
-    describe 'ensure #order_values is preserved' do
-      subject { User.order('id').page 1 }
-      its(:order_values) { should == ['id'] }
-    end
-  end
-
-  describe '#per' do
-    context 'page 1 per 5' do
-      subject { User.page(1).per(5) }
-      it { should have(5).users }
-      its('first.name') { should == 'user001' }
-    end
-  end
-
-  describe '#num_pages' do
-    context 'per 25 (default)' do
-      subject { User.page }
-      its(:num_pages) { should == 4 }
-    end
-
-    context 'per 7' do
-      subject { User.page(2).per(7) }
-      its(:num_pages) { should == 15 }
-    end
-
-    context 'per 65536' do
-      subject { User.page(50).per(65536) }
-      its(:num_pages) { should == 1 }
-    end
-
-    context 'per 0 (using default)' do
-      subject { User.page(50).per(0) }
-      its(:num_pages) { should == 4 }
-    end
-
-    context 'per -1 (using default)' do
-      subject { User.page(5).per(-1) }
-      its(:num_pages) { should == 4 }
-    end
-
-    context 'per "String value that can not be converted into Number" (using default)' do
-      subject { User.page(5).per('aho') }
-      its(:num_pages) { should == 4 }
-    end
-  end
-
-  describe '#current_page' do
-    context 'page 1' do
-      subject { User.page }
-      its(:current_page) { should == 1 }
-    end
-
-    context 'page 2' do
-      subject { User.page(2).per 3 }
-      its(:current_page) { should == 2 }
-    end
-  end
-
-  describe '#first_page?' do
-    context 'on first page' do
-      subject { User.page(1).per(10) }
-      its(:first_page?) { should == true }
-    end
-
-    context 'not on first page' do
-      subject { User.page(5).per(10) }
-      its(:first_page?) { should == false }
+  [User, Admin, GemDefinedModel].each do |model_class|
+    context "for #{model_class}" do
+      describe '#page' do
+        context 'page 1' do
+          subject { model_class.page 1 }
+          it_should_behave_like 'the first page'
+        end
+
+        context 'page 2' do
+          subject { model_class.page 2 }
+          it { should have(25).users }
+          its('first.name') { should == 'user026' }
+        end
+
+        context 'page without an argument' do
+          subject { model_class.page }
+          it_should_behave_like 'the first page'
+        end
+
+        context 'page < 1' do
+          subject { model_class.page 0 }
+          it_should_behave_like 'the first page'
+        end
+
+        context 'page > max page' do
+          subject { model_class.page 5 }
+          it_should_behave_like 'blank page'
+        end
+
+        describe 'ensure #order_values is preserved' do
+          subject { model_class.order('id').page 1 }
+          its('order_values.uniq') { should == ['id'] }
+        end
+      end
+
+      describe '#per' do
+        context 'page 1 per 5' do
+          subject { model_class.page(1).per(5) }
+          it { should have(5).users }
+          its('first.name') { should == 'user001' }
+        end
+      end
+
+      describe '#padding' do
+        context 'page 1 per 5 padding 1' do
+          subject { model_class.page(1).per(5).padding(1) }
+          it { should have(5).users }
+          its('first.name') { should == 'user002' }
+        end
+      end
+
+      describe '#num_pages' do
+        context 'per 25 (default)' do
+          subject { model_class.page }
+          its(:num_pages) { should == 4 }
+        end
+
+        context 'per 7' do
+          subject { model_class.page(2).per(7) }
+          its(:num_pages) { should == 15 }
+        end
+
+        context 'per 65536' do
+          subject { model_class.page(50).per(65536) }
+          its(:num_pages) { should == 1 }
+        end
+
+        context 'per 0 (using default)' do
+          subject { model_class.page(50).per(0) }
+          its(:num_pages) { should == 4 }
+        end
+
+        context 'per -1 (using default)' do
+          subject { model_class.page(5).per(-1) }
+          its(:num_pages) { should == 4 }
+        end
+
+        context 'per "String value that can not be converted into Number" (using default)' do
+          subject { model_class.page(5).per('aho') }
+          its(:num_pages) { should == 4 }
+        end
+      end
+
+
+      describe '#current_page' do
+        context 'page 1' do
+          subject { model_class.page }
+          its(:current_page) { should == 1 }
+        end
+
+        context 'page 2' do
+          subject { model_class.page(2).per 3 }
+          its(:current_page) { should == 2 }
+        end
+      end
+
+      describe '#first_page?' do
+        context 'on first page' do
+          subject { model_class.page(1).per(10) }
+          its(:first_page?) { should == true }
+        end
+
+        context 'not on first page' do
+          subject { model_class.page(5).per(10) }
+          its(:first_page?) { should == false }
+        end
+      end
+
+      describe '#last_page?' do
+        context 'on last page' do
+          subject { model_class.page(10).per(10) }
+          its(:last_page?) { should == true }
+        end
+
+        context 'not on last page' do
+          subject { model_class.page(1).per(10) }
+          its(:last_page?) { should == false }
+        end
+      end
+
+      describe '#count' do
+        context 'page 1' do
+          subject { model_class.page }
+          its(:count) { should == 25 }
+        end
+
+        context 'page 2' do
+          subject { model_class.page 2 }
+          its(:count) { should == 25 }
+        end
+      end
+
+      context 'chained with .group' do
+        subject { model_class.group('age').page(2).per 5 }
+        # 0..10
+        its(:total_count) { should == 11 }
+        its(:num_pages) { should == 3 }
+      end
+
+      context 'activerecord descendants' do
+        subject { ActiveRecord::Base.descendants }
+        its(:length) { should_not == 0 }
+      end
     end
   end
-
-  describe '#last_page?' do
-    context 'on last page' do
-      subject { User.page(10).per(10) }
-      its(:last_page?) { should == true }
-    end
-
-    context 'not on last page' do
-      subject { User.page(1).per(10) }
-      its(:last_page?) { should == false }
-    end
-  end
-
-  describe '#count' do
-    context 'page 1' do
-      subject { User.page }
-      its(:count) { should == 25 }
-    end
-
-    context 'page 2' do
-      subject { User.page 2 }
-      its(:count) { should == 25 }
-    end
-  end
-
-  context 'chained with .group' do
-    subject { User.group('age').page(2).per 5 }
-    # 0..10
-    its(:total_count) { should == 11 }
-    its(:num_pages) { should == 3 }
-  end
-
-  context 'activerecord descendants' do
-    subject { ActiveRecord::Base.descendants }
-    its(:length) { should_not == 0 }
-  end
 end