Change paginator configuration

Instead of boolean predicate methods, just use one `.paginator` method
that returns the paginator name symbolized.

Signed-off-by: David Celis <[email protected]>

Author: davidcelis

README.md

@@ -84,6 +84,13 @@ Link: <http://localhost:3000/movies?page=1>; rel="first">,
 # ...
 ```
 
+## Testing
+
+```bash
+PAGINATOR=kaminari bundle exec rspec
+PAGINATOR=will_paginate bundle exec rspec
+```
+
 ## Contributing
 
 1. Fork it

lib/api-pagination.rb

@@ -3,19 +3,16 @@
 
 module ApiPagination
   class << self
-    attr_writer :kaminari
-    attr_writer :will_paginate
-
-    def kaminari?() !!@kaminari end
-    def will_paginate?() !!@will_paginate end
+    attr_reader :paginator
 
     def paginate(collection, options = {}, &block)
       options[:page]     ||= 1
       options[:per_page] ||= 10
 
-      if ApiPagination.kaminari?
+      case ApiPagination.paginator
+      when :kaminari
         collection.page(options[:page]).per(options[:per_page]).tap(&block)
-      elsif ApiPagination.will_paginate?
+      when :will_paginate
         collection.paginate(:page => options[:page], :per_page => options[:per_page]).tap(&block)
       end
     end

lib/api-pagination/hooks.rb

@@ -26,12 +26,12 @@ def first_page?() !previous_page end
           def last_page?() !next_page end
         end
 
-        ApiPagination.will_paginate = true
+        ApiPagination.instance_variable_set(:@paginator, :will_paginate)
       end
 
       begin; require 'kaminari'; rescue LoadError; end
       if defined?(Kaminari)
-        ApiPagination.kaminari = true
+        ApiPagination.instance_variable_set(:@paginator, :kaminari)
       end
 
       STDERR.puts <<-EOC unless defined?(Kaminari) || defined?(WillPaginate)

spec/spec_helper.rb

@@ -26,8 +26,12 @@ def paginate(options = {})
   end
 end
 
-ApiPagination.kaminari      = ENV['PAGINATOR'] == 'kaminari'
-ApiPagination.will_paginate = ENV['PAGINATOR'] == 'will_paginate'
+if ENV['PAGINATOR']
+  ApiPagination.instance_variable_set(:@paginator, ENV['PAGINATOR'].to_sym)
+else
+  warn 'No PAGINATOR set. Defaulting to kaminari. To test against will_paginate, run `PAGINATOR=will_paginate bundle exec rspec`'
+  ApiPagination.instance_variable_set(:@paginator, :kaminari)
+end
 
 RSpec.configure do |config|
   config.include Rack::Test::Methods