wulffeld_kaminari 0.12.4.1

data/spec/helpers/tags_spec.rb

@@ -0,0 +1,140 @@
+require File.expand_path('../spec_helper', File.dirname(__FILE__))
+include Kaminari::Helpers
+
+describe 'Kaminari::Helpers' do
+  describe 'Paginator' do
+    describe 'Paginator::PageProxy' do
+      describe '#current?' do
+        context 'current_page == page' do
+          subject { Paginator::PageProxy.new({:current_page => 26}, 26, nil) }
+          its(:current?) { should be_true }
+        end
+        context 'current_page != page' do
+          subject { Paginator::PageProxy.new({:current_page => 13}, 26, nil) }
+          its(:current?) { should_not be_true }
+        end
+      end
+
+      describe '#first?' do
+        context 'page == 1' do
+          subject { Paginator::PageProxy.new({:current_page => 26}, 1, nil) }
+          its(:first?) { should be_true }
+        end
+        context 'page != 1' do
+          subject { Paginator::PageProxy.new({:current_page => 13}, 2, nil) }
+          its(:first?) { should_not be_true }
+        end
+      end
+
+      describe '#last?' do
+        context 'current_page == page' do
+          subject { Paginator::PageProxy.new({:num_pages => 39}, 39, nil) }
+          its(:last?) { should be_true }
+        end
+        context 'current_page != page' do
+          subject { Paginator::PageProxy.new({:num_pages => 39}, 38, nil) }
+          its(:last?) { should_not be_true }
+        end
+      end
+
+      describe '#next?' do
+        context 'page == current_page + 1' do
+          subject { Paginator::PageProxy.new({:current_page => 52}, 53, nil) }
+          its(:next?) { should be_true }
+        end
+        context 'page != current_page + 1' do
+          subject { Paginator::PageProxy.new({:current_page => 52}, 77, nil) }
+          its(:next?) { should_not be_true }
+        end
+      end
+
+      describe '#prev?' do
+        context 'page == current_page - 1' do
+          subject { Paginator::PageProxy.new({:current_page => 77}, 76, nil) }
+          its(:prev?) { should be_true }
+        end
+        context 'page != current_page + 1' do
+          subject { Paginator::PageProxy.new({:current_page => 77}, 80, nil) }
+          its(:prev?) { should_not be_true }
+        end
+      end
+
+      describe '#left_outer?' do
+        context 'current_page == left' do
+          subject { Paginator::PageProxy.new({:left => 3}, 3, nil) }
+          its(:left_outer?) { should be_true }
+        end
+        context 'current_page == left + 1' do
+          subject { Paginator::PageProxy.new({:left => 3}, 4, nil) }
+          its(:left_outer?) { should_not be_true }
+        end
+        context 'current_page == left + 2' do
+          subject { Paginator::PageProxy.new({:left => 3}, 5, nil) }
+          its(:left_outer?) { should_not be_true }
+        end
+      end
+
+      describe '#right_outer?' do
+        context 'num_pages - page > right' do
+          subject { Paginator::PageProxy.new({:num_pages => 10, :right => 3}, 6, nil) }
+          its(:right_outer?) { should_not be_true }
+        end
+        context 'num_pages - page == right' do
+          subject { Paginator::PageProxy.new({:num_pages => 10, :right => 3}, 7, nil) }
+          its(:right_outer?) { should_not be_true }
+        end
+        context 'num_pages - page < right' do
+          subject { Paginator::PageProxy.new({:num_pages => 10, :right => 3}, 8, nil) }
+          its(:right_outer?) { should be_true }
+        end
+      end
+
+      describe '#inside_window?' do
+        context 'page > current_page' do
+          context 'page - current_page > window' do
+            subject { Paginator::PageProxy.new({:current_page => 4, :window => 5}, 10, nil) }
+            its(:inside_window?) { should_not be_true }
+          end
+          context 'page - current_page == window' do
+            subject { Paginator::PageProxy.new({:current_page => 4, :window => 6}, 10, nil) }
+            its(:inside_window?) { should be_true }
+          end
+          context 'page - current_page < window' do
+            subject { Paginator::PageProxy.new({:current_page => 4, :window => 7}, 10, nil) }
+            its(:inside_window?) { should be_true }
+          end
+        end
+        context 'current_page > page' do
+          context 'current_page - page > window' do
+            subject { Paginator::PageProxy.new({:current_page => 15, :window => 4}, 10, nil) }
+            its(:inside_window?) { should_not be_true }
+          end
+          context 'current_page - page == window' do
+            subject { Paginator::PageProxy.new({:current_page => 15, :window => 5}, 10, nil) }
+            its(:inside_window?) { should be_true }
+          end
+          context 'current_page - page < window' do
+            subject { Paginator::PageProxy.new({:current_page => 15, :window => 6}, 10, nil) }
+            its(:inside_window?) { should be_true }
+          end
+        end
+      end
+      describe '#was_truncated?' do
+        before do
+          stub(@template = Object.new) do
+            options { {} }
+            params { {} }
+          end
+        end
+        context 'last.is_a? Gap' do
+          subject { Paginator::PageProxy.new({}, 10, Gap.new(@template)) }
+          its(:was_truncated?) { should be_true }
+        end
+        context 'last.is not a Gap' do
+          subject { Paginator::PageProxy.new({}, 10, Page.new(@template)) }
+          its(:was_truncated?) { should_not be_true }
+        end
+      end
+    end
+  end
+end

data/spec/models/active_record_relation_methods_spec.rb

@@ -0,0 +1,28 @@
+require File.expand_path('../spec_helper', File.dirname(__FILE__))
+
+describe Kaminari::ActiveRecordRelationMethods do
+  describe '#total_count' do
+    before do
+      @author = User.create! :name => 'author'
+      @author2 = User.create! :name => 'author2'
+      @author3 = User.create! :name => 'author3'
+      @books = 2.times.map {|i| @author.books_authored.create!(:title => "title%03d" % i) }
+      @books2 = 3.times.map {|i| @author2.books_authored.create!(:title => "title%03d" % i) }
+      @books3 = 4.times.map {|i| @author3.books_authored.create!(:title => "subject%03d" % i) }
+      @readers = 4.times.map { User.create! :name => 'reader' }
+      @books.each {|book| book.readers << @readers }
+    end
+
+    context "when the scope includes an order which references a generated column" do
+      it "should successfully count the results" do
+        @author.readers.by_read_count.page(1).total_count.should == @readers.size
+      end
+    end
+    context "when the scope use conditions on includes" do
+      it "should keep includes and successfully count the results" do
+        # Only @author and @author2 have books titled with the title00x partern
+        User.includes(:books_authored).where("books.title LIKE 'title00%'").page(1).total_count.should == 2
+      end
+    end
+  end
+end

data/spec/models/array_spec.rb

@@ -0,0 +1,105 @@
+require File.expand_path('../spec_helper', File.dirname(__FILE__))
+
+describe Kaminari::PaginatableArray do
+  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(:first) { should == 1 }
+    end
+
+    shared_examples_for 'blank array page' do
+      it { should have(0).items }
+    end
+
+    context 'page 1' do
+      subject { array.page 1 }
+      it_should_behave_like 'the first page of array'
+    end
+
+    context 'page 2' do
+      subject { array.page 2 }
+      it { should have(25).users }
+      its(:first) { should == 26 }
+    end
+
+    context 'page without an argument' do
+      subject { array.page }
+      it_should_behave_like 'the first page of array'
+    end
+
+    context 'page < 1' do
+      subject { array.page 0 }
+      it_should_behave_like 'the first page of array'
+    end
+
+    context 'page > max page' do
+      subject { array.page 5 }
+      it_should_behave_like 'blank array page'
+    end
+  end
+
+  describe '#per' do
+    context 'page 1 per 5' do
+      subject { array.page(1).per(5) }
+      it { should have(5).users }
+      its(:first) { should == 1 }
+    end
+  end
+
+  describe '#num_pages' do
+    context 'per 25 (default)' do
+      subject { array.page }
+      its(:num_pages) { should == 4 }
+    end
+
+    context 'per 7' do
+      subject { array.page(2).per(7) }
+      its(:num_pages) { should == 15 }
+    end
+
+    context 'per 65536' do
+      subject { array.page(50).per(65536) }
+      its(:num_pages) { should == 1 }
+    end
+
+    context 'per 0 (using default)' do
+      subject { array.page(50).per(0) }
+      its(:num_pages) { should == 4 }
+    end
+
+    context 'per -1 (using default)' do
+      subject { array.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 { array.page(5).per('aho') }
+      its(:num_pages) { should == 4 }
+    end
+  end
+
+  describe '#current_page' do
+    context 'page 1' do
+      subject { array.page }
+      its(:current_page) { should == 1 }
+    end
+
+    context 'page 2' do
+      subject { array.page(2).per 3 }
+      its(:current_page) { should == 2 }
+    end
+  end
+
+  describe '#count' do
+    context 'page 1' do
+      subject { array.page }
+      its(:count) { should == 25 }
+    end
+
+    context 'page 2' do
+      subject { array.page 2 }
+      its(:count) { should == 25 }
+    end
+  end
+end

data/spec/models/default_per_page_spec.rb

@@ -0,0 +1,29 @@
+require File.expand_path('../spec_helper', File.dirname(__FILE__))
+
+describe 'default per_page' do
+  describe 'AR::Base' do
+    subject { ActiveRecord::Base }
+    it { should_not respond_to :paginates_per }
+  end
+
+  subject { User.page 0 }
+
+  context 'by default' do
+    its(:limit_value) { should == 25 }
+  end
+
+  context 'when explicitly set via paginates_per' do
+    before { User.paginates_per 1326 }
+    its(:limit_value) { should == 1326 }
+    after { User.paginates_per nil }
+  end
+
+  describe "default per_page value's independency per model" do
+    context "when User's default per_page was changed" do
+      before { User.paginates_per 1326 }
+      subject { Book.page 0 }
+      its(:limit_value) { should == 25 }
+      after { User.paginates_per nil }
+    end
+  end
+end

data/spec/models/mongo_mapper_spec.rb

@@ -0,0 +1,79 @@
+require File.expand_path('../spec_helper', File.dirname(__FILE__))
+require 'mongo_mapper'
+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...
+  end
+
+  describe '#page' do
+    context 'page 1' do
+      subject { Developer.page(1) }
+      it { should be_a Plucky::Query }
+      its(:current_page) { should == 1 }
+      its(:limit_value) { should == 25 }
+      its(:num_pages) { should == 12 }
+      it { should skip(0) }
+    end
+
+    context 'page 2' do
+      subject { Developer.page 2 }
+      it { should be_a Plucky::Query }
+      its(:current_page) { should == 2 }
+      its(:limit_value) { should == 25 }
+      its(:num_pages) { should == 12 }
+      it { should skip 25 }
+    end
+
+    context 'page "foobar"' do
+      subject { Developer.page 'foobar' }
+      it { should be_a Plucky::Query }
+      its(:current_page) { should == 1 }
+      its(:limit_value) { should == 25 }
+      its(:num_pages) { should == 12 }
+      it { should skip 0 }
+    end
+
+    context 'with criteria before' do
+      it "should have the proper criteria source" do
+        Developer.where(:salary => 1).page(2).criteria.source.should == {:salary => 1}
+      end
+
+      subject { Developer.where(:salary => 1).page 2 }
+      its(:current_page) { should == 2 }
+      its(:limit_value) { should == 25 }
+      its(:num_pages) { should == 12 }
+      it { should skip 25 }
+    end
+
+    context 'with criteria after' do
+      it "should have the proper criteria source" do
+        Developer.where(:salary => 1).page(2).criteria.source.should == {:salary => 1}
+      end
+
+      subject { Developer.page(2).where(:salary => 1) }
+      its(:current_page) { should == 2 }
+      its(:limit_value) { should == 25 }
+      its(:num_pages) { should == 12 }
+      it { should skip 25 }
+    end
+  end
+
+  describe '#per' do
+    subject { Developer.page(2).per(10) }
+    it { should be_a Plucky::Query }
+    its(:current_page) { should == 2 }
+    its(:limit_value) { should == 10 }
+    its(:num_pages) { should == 30 }
+    it { should skip 10 }
+  end
+end

data/spec/models/mongoid_spec.rb

@@ -0,0 +1,72 @@
+require File.expand_path('../spec_helper', File.dirname(__FILE__))
+require 'mongoid'
+require 'kaminari/models/mongoid_extension'
+
+describe Kaminari::MongoidExtension do
+  before :all do
+    class Developer
+      include ::Mongoid::Document
+      field :salary, :type => Integer
+    end
+  end
+  before do
+    stub(subject).count { 300 } # in order to avoid DB access...
+  end
+
+  describe '#page' do
+    context 'page 1' do
+      subject { Developer.page 1 }
+      it { should be_a Mongoid::Criteria }
+      its(:current_page) { should == 1 }
+      its(:limit_value) { should == 25 }
+      its(:num_pages) { should == 12 }
+      it { should skip(0) }
+    end
+
+    context 'page 2' do
+      subject { Developer.page 2 }
+      it { should be_a Mongoid::Criteria }
+      its(:current_page) { should == 2 }
+      its(:limit_value) { should == 25 }
+      its(:num_pages) { should == 12 }
+      it { should skip 25 }
+    end
+
+    context 'page "foobar"' do
+      subject { Developer.page 'foobar' }
+      it { should be_a Mongoid::Criteria }
+      its(:current_page) { should == 1 }
+      its(:limit_value) { should == 25 }
+      its(:num_pages) { should == 12 }
+      it { should skip 0 }
+    end
+
+    context 'with criteria before' do
+      subject { Developer.where(:salary => 1).page 2 }
+      its(:selector) { should == {:salary => 1} }
+      its(:current_page) { should == 2 }
+      its(:limit_value) { should == 25 }
+      its(:num_pages) { should == 12 }
+      it { should skip 25 }
+    end
+
+    context 'with criteria after' do
+      subject { Developer.page(2).where(:salary => 1) }
+      its(:selector) { should == {:salary => 1} }
+      its(:current_page) { should == 2 }
+      its(:limit_value) { should == 25 }
+      its(:num_pages) { should == 12 }
+      it { should skip 25 }
+    end
+
+  end
+
+  describe '#per' do
+    subject { Developer.page(2).per(10) }
+    it { should be_a Mongoid::Criteria }
+    its(:current_page) { should == 2 }
+    its(:limit_value) { should == 10 }
+    its(:num_pages) { should == 30 }
+    it { should skip 10 }
+  end
+end

data/spec/models/scopes_spec.rb

@@ -0,0 +1,149 @@
+require File.expand_path('../spec_helper', File.dirname(__FILE__))
+
+describe Kaminari::ActiveRecordExtension do
+  before :all do
+    1.upto(100) {|i| User.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 }
+    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

data/spec/spec_helper.rb

@@ -0,0 +1,28 @@
+$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
+$LOAD_PATH.unshift(File.dirname(__FILE__))
+require 'rails'
+require 'mongoid'
+require 'kaminari'
+require 'database_cleaner'
+# Ensure we use 'syck' instead of 'psych' in 1.9.2
+# RubyGems >= 1.5.0 uses 'psych' on 1.9.2, but
+# Psych does not yet support YAML 1.1 merge keys.
+# Merge keys is often used in mongoid.yml
+# See: http://redmine.ruby-lang.org/issues/show/4300
+if RUBY_VERSION >= '1.9.2'
+  YAML::ENGINE.yamler = 'syck'
+end
+require File.join(File.dirname(__FILE__), 'fake_app')
+
+require 'rspec/rails'
+# Requires supporting files with custom matchers and macros, etc,
+# in ./support/ and its subdirectories.
+Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each {|f| require f}
+
+RSpec.configure do |config|
+  config.mock_with :rr
+  config.before :all do
+#     ActiveRecord::Base.connection.execute 'CREATE TABLE "users" ("id" INTEGER PRIMARY KEY AUTOINCREMENT NOT NULL, "name" varchar(255))' unless ActiveRecord::Base.connection.table_exists? 'users'
+    CreateAllTables.up unless ActiveRecord::Base.connection.table_exists? 'users'
+  end
+end

data/spec/support/database_cleaner.rb

@@ -0,0 +1,13 @@
+DatabaseCleaner.strategy = :transaction
+
+RSpec.configure do |config|
+  config.before :suite do
+    DatabaseCleaner.clean_with :truncation
+  end
+  config.before :each do
+    DatabaseCleaner.start
+  end
+  config.after :each do
+    DatabaseCleaner.clean
+  end
+end

data/spec/support/matchers.rb

@@ -0,0 +1,46 @@
+RSpec::Matchers.define :contain_tag do |klass|
+  match do |collection|
+    if @num.blank?
+      collection.any? {|tag| tag.is_a? klass}
+    else
+      (@count = collection.count {|tag| tag.is_a? klass}) == @num
+    end
+  end
+
+  def count(num)
+    @num = num
+    self
+  end
+
+  description do
+    "contain #{@num || 'any'} instance(s) of #{klass.name}"
+  end
+  failure_message_for_should do |collection|
+    "expected #{@num || 'any'} instance(s) of #{klass.name} but was #{@count}"
+  end
+end
+
+RSpec::Matchers.define :contain_tag_old do |count|
+  match do |collection|
+    (@count = collection.count {|tag| tag.is_a? @klass}) == count
+  end
+
+  def instance_of(klass)
+    @klass = klass
+    self
+  end
+  alias :instances_of :instance_of
+
+  description do
+    "contain #{count || 'any'} instance(s) of #{@klass.name}"
+  end
+  failure_message_for_should do |collection|
+    "expected #{count || 'any'} instance(s) of #{@klass.name} but was #{@count}"
+  end
+end
+
+RSpec::Matchers.define :skip do |num|
+  match do |criteria|
+    criteria.instance_variable_get('@options')[:skip] == num
+  end
+end