kaminari 0.13.0 → 0.14.0

data/spec/models/active_record/max_per_page_spec.rb

@@ -0,0 +1,32 @@
+require 'spec_helper'
+
+if defined? ActiveRecord
+
+  describe 'max per_page' do
+    describe 'AR::Base' do
+      subject { ActiveRecord::Base }
+      it { should_not respond_to :max_paginates_per }
+    end
+
+    subject { User.page(0).per(100) }
+
+    context 'by default' do
+      its(:limit_value) { should == 100 }
+    end
+
+    context 'when explicitly set via max_paginates_per' do
+      before { User.max_paginates_per 10 }
+      its(:limit_value) { should == 10 }
+      after { User.max_paginates_per nil }
+    end
+
+    describe "max per_page value's independency per model" do
+      context "when User's max per_page was changed" do
+        before { User.max_paginates_per 10 }
+        subject { Book.page(0).per(100) }
+        its(:limit_value) { should == 100 }
+        after { User.max_paginates_per nil }
+      end
+    end
+  end
+end

data/spec/models/active_record/scopes_spec.rb

@@ -0,0 +1,166 @@
+require 'spec_helper'
+
+if defined? ActiveRecord
+
+  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 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
+
+    [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 '#total_pages' do
+          context 'per 25 (default)' do
+            subject { model_class.page }
+            its(:total_pages) { should == 4 }
+          end
+
+          context 'per 7' do
+            subject { model_class.page(2).per(7) }
+            its(:total_pages) { should == 15 }
+          end
+
+          context 'per 65536' do
+            subject { model_class.page(50).per(65536) }
+            its(:total_pages) { should == 1 }
+          end
+
+          context 'per 0 (using default)' do
+            subject { model_class.page(50).per(0) }
+            its(:total_pages) { should == 4 }
+          end
+
+          context 'per -1 (using default)' do
+            subject { model_class.page(5).per(-1) }
+            its(:total_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(:total_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(:total_pages) { should == 3 }
+        end
+
+        context 'activerecord descendants' do
+          subject { ActiveRecord::Base.descendants }
+          its(:length) { should_not == 0 }
+        end
+      end
+    end
+  end
+end

data/spec/models/array_spec.rb

@@ -56,35 +56,35 @@ describe Kaminari::PaginatableArray do
     end
   end
 
-  describe '#num_pages' do
+  describe '#total_pages' do
     context 'per 25 (default)' do
       subject { array.page }
-      its(:num_pages) { should == 4 }
+      its(:total_pages) { should == 4 }
     end
 
     context 'per 7' do
       subject { array.page(2).per(7) }
-      its(:num_pages) { should == 15 }
+      its(:total_pages) { should == 15 }
     end
 
     context 'per 65536' do
       subject { array.page(50).per(65536) }
-      its(:num_pages) { should == 1 }
+      its(:total_pages) { should == 1 }
     end
 
     context 'per 0 (using default)' do
       subject { array.page(50).per(0) }
-      its(:num_pages) { should == 4 }
+      its(:total_pages) { should == 4 }
     end
 
     context 'per -1 (using default)' do
       subject { array.page(5).per(-1) }
-      its(:num_pages) { should == 4 }
+      its(:total_pages) { should == 4 }
     end
 
     context 'per "String value that can not be converted into Number" (using default)' do
       subject { array.page(5).per('aho') }
-      its(:num_pages) { should == 4 }
+      its(:total_pages) { should == 4 }
     end
   end
 

data/spec/models/data_mapper/data_mapper_spec.rb

@@ -0,0 +1,155 @@
+require 'spec_helper'
+
+if defined? DataMapper
+  describe Kaminari::DataMapperExtension do
+    before do
+      100.times do |i|
+        User.create(:name => "User#{i}", :age => i)
+      end
+
+      worker0 = User[0]
+      50.times do |i|
+        worker0.projects << Project.create(:name => "Project#{i}")
+      end
+      worker0.projects.save
+    end
+
+    describe 'Collection' do
+      subject{ User.all }
+      it { should respond_to(:page) }
+      it { should_not respond_to(:per) }
+    end
+
+    describe 'Model' do
+      subject{ User }
+      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 { User.all(:age.gte => 60).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 == User.count(:age.gte => 60) }
+        its(:total_pages) { should == 2 }
+      end
+
+      context 'page 1' do
+        subject { User.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 == 100 }
+        its(:total_pages) { should == 4 }
+      end
+
+      context 'page 2' do
+        subject { User.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 == 100 }
+        its(:total_pages) { should == 4 }
+      end
+
+      context 'page "foobar"' do
+        subject { User.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 == 100 }
+        its(:total_pages) { should == 4 }
+      end
+
+      context 'with criteria before' do
+        subject { User.all(:age.gt => 60).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 == User.count(:age.gt => 60) }
+        its(:total_pages) { should == 2 }
+      end
+
+      context 'with criteria after' do
+        subject { User.page(2).all(:age.gt => 60) }
+        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 == User.count(:age.gt => 60) }
+        its(:total_pages) { should == 2 }
+      end
+    end
+
+    describe '#per' do
+      context 'on simple query' do
+        subject { User.page(2).per(20) }
+        it { should be_a DataMapper::Collection }
+        its(:current_page) { should == 2 }
+        its('query.limit') { should == 20 }
+        its(:limit_value) { should == 20 }
+        its('query.offset') { should == 20 }
+        its(:total_count) { should == 100 }
+        its(:total_pages) { should == 5 }
+      end
+
+      context 'on query with condition' do
+        subject { User.page(5).all(:age.lte => 80).per(13) }
+        its(:current_page) { should == 5 }
+        its('query.limit') { should == 13 }
+        its('query.offset') { should == 52 }
+        its(:total_count) { should == 81 }
+        its(:total_pages) { should == 7 }
+      end
+
+      context 'on query with order' do
+        subject { User.page(5).all(:age.lte => 80, :order => [:age.asc]).per(13) }
+        it('includes user with age 52') { should include(User.first(:age => 52)) }
+        it('does not include user with age 51') { should_not include(User.first(:age => 51)) }
+        it('includes user with age 52') { should include(User.first(:age => 64)) }
+        it('does not include user with age 51') { should_not include(User.first(:age => 65)) }
+        its(:current_page) { should == 5 }
+        its('query.limit') { should == 13 }
+        its('query.offset') { should == 52 }
+        its(:total_count) { should == 81 }
+        its(:total_pages) { should == 7 }
+      end
+
+      context 'on chained queries' do
+        subject { User.all(:age.gte => 50).page(3).all(:age.lte => 80).per(13) }
+        its(:current_page) { should == 3 }
+        its('query.limit') { should == 13 }
+        its('query.offset') { should == 26 }
+        its(:total_count) { should == 31 }
+        its(:total_pages) { should == 3 }
+      end
+
+      context 'on query on association' do
+        subject { User[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(:total_pages) { should == 10 }
+      end
+
+      context 'on query with association conditions' do
+        subject { User.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(:total_pages) { should == 10 }
+      end
+    end
+  end
+end

data/spec/models/mongo_mapper/mongo_mapper_spec.rb

@@ -0,0 +1,72 @@
+require 'spec_helper'
+
+if defined? MongoMapper
+  describe Kaminari::MongoMapperExtension do
+    before(:each) do
+      User.destroy_all
+      41.times { User.create!({:salary => 1}) }
+    end
+
+    describe '#page' do
+      context 'page 1' do
+        subject { User.page(1) }
+        it { should be_a Plucky::Query }
+        its(:current_page) { should == 1 }
+        its(:limit_value) { should == 25 }
+        its(:total_pages) { should == 2 }
+        it { should skip(0) }
+      end
+
+      context 'page 2' do
+        subject { User.page 2 }
+        it { should be_a Plucky::Query }
+        its(:current_page) { should == 2 }
+        its(:limit_value) { should == 25 }
+        its(:total_pages) { should == 2 }
+        it { should skip 25 }
+      end
+
+      context 'page "foobar"' do
+        subject { User.page 'foobar' }
+        it { should be_a Plucky::Query }
+        its(:current_page) { should == 1 }
+        its(:limit_value) { should == 25 }
+        its(:total_pages) { should == 2 }
+        it { should skip 0 }
+      end
+
+      context 'with criteria before' do
+        it "should have the proper criteria source" do
+          User.where(:salary => 1).page(2).criteria.source.should == {:salary => 1}
+        end
+
+        subject { User.where(:salary => 1).page 2 }
+        its(:current_page) { should == 2 }
+        its(:limit_value) { should == 25 }
+        its(:total_pages) { should == 2 }
+        it { should skip 25 }
+      end
+
+      context 'with criteria after' do
+        it "should have the proper criteria source" do
+          User.where(:salary => 1).page(2).criteria.source.should == {:salary => 1}
+        end
+
+        subject { User.page(2).where(:salary => 1) }
+        its(:current_page) { should == 2 }
+        its(:limit_value) { should == 25 }
+        its(:total_pages) { should == 2 }
+        it { should skip 25 }
+      end
+    end
+
+    describe '#per' do
+      subject { User.page(2).per(10) }
+      it { should be_a Plucky::Query }
+      its(:current_page) { should == 2 }
+      its(:limit_value) { should == 10 }
+      its(:total_pages) { should == 5 }
+      it { should skip 10 }
+    end
+  end
+end