[TEST] Added an integration test for combining pagination with scopes

See: http://guides.rubyonrails.org/active_record_querying.html#scopes

Related: elastic#16

Author: karmi

elasticsearch-model/test/integration/active_record_pagination_test.rb

@@ -4,9 +4,11 @@ module Elasticsearch
   module Model
     class ActiveRecordPaginationTest < Elasticsearch::Test::IntegrationTestCase
 
-      class ::Article < ActiveRecord::Base
+      class ::ArticleForPagination < ActiveRecord::Base
         include Elasticsearch::Model
 
+        scope :published, -> { where(published: true) }
+
         settings index: { number_of_shards: 1, number_of_replicas: 0 } do
           mapping do
             indexes :title,      type: 'string', analyzer: 'snowball'
@@ -20,23 +22,26 @@ class ::Article < ActiveRecord::Base
       context "ActiveRecord pagination" do
         setup do
           ActiveRecord::Schema.define(:version => 1) do
-            create_table :articles do |t|
+            create_table ::ArticleForPagination.table_name do |t|
               t.string   :title
               t.datetime :created_at, :default => 'NOW()'
+              t.boolean  :published
             end
           end
 
-          Article.delete_all
-          Article.__elasticsearch__.create_index! force: true
+          ArticleForPagination.delete_all
+          ArticleForPagination.__elasticsearch__.create_index! force: true
 
-          68.times do |i| ::Article.create! title: "Test #{i}" end
+          68.times do |i|
+            ::ArticleForPagination.create! title: "Test #{i}", published: (i % 2 == 0)
+          end
 
-          Article.import
-          Article.__elasticsearch__.refresh_index!
+          ArticleForPagination.import
+          ArticleForPagination.__elasticsearch__.refresh_index!
         end
 
         should "be on the first page by default" do
-          records = Article.search('title:test').page(1).records
+          records = ArticleForPagination.search('title:test').page(1).records
 
           assert_equal 25, records.size
           assert_equal 1, records.current_page
@@ -50,7 +55,7 @@ class ::Article < ActiveRecord::Base
         end
 
         should "load next page" do
-          records = Article.search('title:test').page(2).records
+          records = ArticleForPagination.search('title:test').page(2).records
 
           assert_equal 25, records.size
           assert_equal 2, records.current_page
@@ -64,7 +69,7 @@ class ::Article < ActiveRecord::Base
         end
 
         should "load last page" do
-          records = Article.search('title:test').page(3).records
+          records = ArticleForPagination.search('title:test').page(3).records
 
           assert_equal 18, records.size
           assert_equal 3, records.current_page
@@ -78,7 +83,7 @@ class ::Article < ActiveRecord::Base
         end
 
         should "not load invalid page" do
-          records = Article.search('title:test').page(6).records
+          records = ArticleForPagination.search('title:test').page(6).records
 
           assert_equal 0, records.size
           assert_equal 6, records.current_page
@@ -91,16 +96,22 @@ class ::Article < ActiveRecord::Base
           assert   records.out_of_range?, "Should be out of range"
         end
 
+        should "be combined with scopes" do
+          records = ArticleForPagination.search('title:test').page(2).records.published
+          assert records.all? { |r| r.published? }
+          assert_equal 12, records.size
+        end
+
         context "with specific model settings" do
           teardown do
-            Article.instance_variable_set(:@_default_per_page, nil)
+            ArticleForPagination.instance_variable_set(:@_default_per_page, nil)
           end
         end
 
         should "respect paginates_per" do
-          Article.paginates_per 50
+          ArticleForPagination.paginates_per 50
 
-          assert_equal 50, Article.search('*').page(1).records.size
+          assert_equal 50, ArticleForPagination.search('*').page(1).records.size
         end
       end