draper 3.0.1 → 4.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: cc23ed0f81227aa958510ecb67676453d97b40cc
-  data.tar.gz: ed8a159772bafb7df87cc5c767757e8b697f55b3
+SHA256:
+  metadata.gz: a90d57e1df50336b735a6f16cc119a558ee8472b70bd9748c59f25b70afc319a
+  data.tar.gz: a4510f40a8afac7976e7afea4659d8e1c3eb981c005bc4046660b4d110874df7
 SHA512:
-  metadata.gz: 466c609e53461c1e1f260f9e2bf6a0eb522238e5bdae8a2f37a88c1cd22d22665bf18293ba497f9f64d27faa0e12f0dbb9e3d906bae569f324c6124c02ad6b7e
-  data.tar.gz: db229fe0cf5daf0acc2e52834c599a21e9ba85782458a8fd3a0520574e77f1857ec30c57c81dceb75561016170c941f8da3593d3a58fd395137c01ce57322cf0
+  metadata.gz: bee71715525fdc2de5336c399b1dfe4a774d9fd68071bb01468c8f31712504cf6af8beecf1f4facf4ea25ef5bcad7a14b2e2f12bec12f7b6f1e55c04318c4f7a
+  data.tar.gz: 8f013f4b36efa767ada401a979e5bf10e28f068997d24fd8035e4025254300709eb7c2e80c5f0498cc49c176b7561fbff48e2cc03f39b418f368c3266bf30359

data/.travis.yml

@@ -1,16 +1,28 @@
+env:
+  global:
+    - CC_TEST_REPORTER_ID=b7ba588af2a540fa96c267b3655a2afe31ea29976dc25905a668dd28d5e88915
+
 language: ruby
-sudo: false
 cache: bundler
 
 services:
   - mongodb
 
 rvm:
-  - 2.2.7
-  - 2.3.4
-  - 2.4.1
+  - 2.4.9
+  - 2.5.7
+  - 2.6.5
+  - 2.7.0
   - ruby-head
 
 matrix:
   allow_failures:
     - rvm: ruby-head
+
+before_script:
+  - curl -L https://codeclimate.com/downloads/test-reporter/test-reporter-latest-linux-amd64 > ./cc-test-reporter
+  - chmod +x ./cc-test-reporter
+  - ./cc-test-reporter before-build
+
+after_script:
+  - ./cc-test-reporter after-build --exit-code $TRAVIS_TEST_RESULT

data/CHANGELOG.md

@@ -1,5 +1,35 @@
 # Draper Changelog
 
+## 4.0.1 - 2020-03-25
+
+### Fixes
+* Check only object's private methods when preventing delegation [#875](https://github.com/drapergem/draper/pull/875)
+
+### Other Changes
+* Use `alias` over `alias_method` [#877](https://github.com/drapergem/draper/pull/877)
+
+## 4.0.0 - 2020-02-05
+
+### Breaking Changes
+* Drop support for Ruby < 2.4 [#852](https://github.com/drapergem/draper/pull/852), [#872](https://github.com/drapergem/draper/pull/872)
+* Don't delegate public methods overridden by a private method in the decorator [#849](https://github.com/drapergem/draper/pull/849)
+
+### Fixes
+* Add preservation of decorator options on QueryMethods [#868](https://github.com/drapergem/draper/pull/868)
+* Add `#respond_to_missing?` to `CollectionDecorator` so it correctly responds to ORM methods [#850](https://github.com/drapergem/draper/pull/850)
+* Fix deprecation warning with the new Rails 6 `ActionView::Base` constructor [#866](https://github.com/drapergem/draper/pull/866)
+* Fix deprecation warning with Ruby 2.7 [#870](https://github.com/drapergem/draper/pull/870)
+
+### Other Changes
+* Add SimpleCov for code coverage analysis [#851](https://github.com/drapergem/draper/pull/851)
+* Update RSpec syntax in the README [#855](https://github.com/drapergem/draper/pull/855)
+* Update continuous integration configuration [#861](https://github.com/drapergem/draper/pull/861), [#862](https://github.com/drapergem/draper/pull/862), [#863](https://github.com/drapergem/draper/pull/863), [#872](https://github.com/drapergem/draper/pull/872)
+
+## 3.1.0
+* Rails 6 support [#841](https://github.com/drapergem/draper/pull/841)
+* Include ORM query methods in `CollectionDecorator` (e.g. `includes`) [#845](https://github.com/drapergem/draper/pull/845)
+* Document the fix for view context leaking in specs [#847](https://github.com/drapergem/draper/pull/847)
+
 ## 3.0.1
 * Let `decorator_class` infer anonymous class decorator from superclass [#820](https://github.com/drapergem/draper/pull/820)
 * When inferring decorator fails, show original class instead of `ActiveRecord::Base` [#821](https://github.com/drapergem/draper/pull/821)

data/Gemfile

@@ -3,7 +3,11 @@ source "https://rubygems.org"
 gemspec
 
 platforms :ruby do
-  gem "sqlite3"
+  if RUBY_VERSION >= "2.5.0"
+    gem 'sqlite3'
+  else
+    gem 'sqlite3', '~> 1.3.6'
+  end
 end
 
 platforms :jruby do
@@ -11,5 +15,9 @@ platforms :jruby do
   gem "activerecord-jdbcsqlite3-adapter"
 end
 
-gem "rails", "~> 5.0"
+if RUBY_VERSION >= "2.5.0"
+  gem "rails", "~> 6.0"
+else
+  gem "rails", "~> 5.0"
+end
 gem "mongoid", github: "mongodb/mongoid"

data/README.md

@@ -2,6 +2,7 @@
 
 [![TravisCI Build Status](https://travis-ci.org/drapergem/draper.svg?branch=master)](http://travis-ci.org/drapergem/draper)
 [![Code Climate](https://codeclimate.com/github/drapergem/draper.svg)](https://codeclimate.com/github/drapergem/draper)
+[![Test Coverage](https://api.codeclimate.com/v1/badges/0d40c43951d516bf6985/test_coverage)](https://codeclimate.com/github/drapergem/draper/test_coverage)
 [![Inline docs](http://inch-ci.org/github/drapergem/draper.svg?branch=master)](http://inch-ci.org/github/drapergem/draper)
 
 Draper adds an object-oriented layer of presentation logic to your Rails
@@ -107,7 +108,7 @@ Decorators are the ideal place to:
 
 ## Installation
 
-As of version 3.0.0, Draper is only compatible with Rails 5 / Ruby 2.2 and later. Add Draper to your Gemfile.
+As of version 4.0.0, Draper only officially supports Rails 5.2 / Ruby 2.4 and later. Add Draper to your Gemfile.
 
 ```ruby
   gem 'draper'
@@ -326,6 +327,17 @@ your `ArticleDecorator` and they'll return decorated objects:
 @article = ArticleDecorator.find(params[:id])
 ```
 
+### Decorated Query Methods
+By default, Draper will decorate all [QueryMethods](https://api.rubyonrails.org/classes/ActiveRecord/QueryMethods.html)
+of ActiveRecord.
+If you're using another ORM, in order to support it, you can tell Draper to use a custom strategy:
+
+```ruby
+Draper.configure do |config|
+  config.default_query_methods_strategy = :mongoid
+end
+```
+
 ### When to Decorate Objects
 
 Decorators are supposed to behave very much like the models they decorate, and
@@ -368,7 +380,7 @@ can continue to use the `@article` instance variable to manipulate the model -
 for example, `@article.comments.build` to add a new blank comment for a form.
 
 ## Configuration
-Draper works out the box well, but also provides a hook for you to configure its 
+Draper works out the box well, but also provides a hook for you to configure its
 default functionality. For example, Draper assumes you have a base `ApplicationController`.
 If your base controller is named something different (e.g. `BaseController`),
 you can tell Draper to use it by adding the following to an initializer:
@@ -457,12 +469,29 @@ end
 ```
 
 Then you can stub the specific route helper functions you need using your
-preferred stubbing technique (this example uses RSpec's `stub` method):
+preferred stubbing technique. This examples uses Rspec currently recommended API
+available in RSpec 3.6+
 
 ```ruby
-helpers.stub(users_path: '/users')
+without_partial_double_verification do
+  allow(helpers).to receive(:users_path).and_return('/users')
+end
+```
+
+### View context leakage
+As mentioned before, Draper needs to build a view context to access helper methods. In MiniTest, the view context is
+cleared during `before_setup` preventing any view context leakage. In RSpec, the view context is cleared before each
+`decorator`, `controller`, and `mailer` spec. However, if you use decorators in other types of specs
+(e.g. `job`), you may still experience the view context leaking from the previous spec. To solve this, add the
+following to your `spec_helper` for each type of spec you are experiencing the leakage:
+
+```ruby
+config.before(:each, type: :type) { Draper::ViewContext.clear! }
 ```
 
+_Note_: The `:type` above is just a placeholder. Replace `:type` with the type of spec you are experiencing
+the leakage from.
+
 ## Advanced usage
 
 ### Shared Decorator Methods
@@ -620,7 +649,7 @@ you can include this module manually.
 
 ### Active Job Integration
 
-[Active Job](http://edgeguides.rubyonrails.org/active_job_basics.html) allows you to pass ActiveRecord 
+[Active Job](http://edgeguides.rubyonrails.org/active_job_basics.html) allows you to pass ActiveRecord
 objects to background tasks directly and performs the necessary serialization and deserialization. In
 order to do this, arguments to a background job must implement [Global ID](https://github.com/rails/globalid).
 Decorated objects implement Global ID by delegating to the object they are decorating. This means

data/draper.gemspec

@@ -17,17 +17,18 @@ Gem::Specification.new do |s|
 
   s.required_ruby_version = '>= 2.2.2'
 
-  s.add_dependency 'activesupport', '~> 5.0'
-  s.add_dependency 'actionpack', '~> 5.0'
-  s.add_dependency 'request_store', '~> 1.0'
-  s.add_dependency 'activemodel', '~> 5.0'
-  s.add_dependency 'activemodel-serializers-xml', '~> 1.0'
+  s.add_dependency 'activesupport', '>= 5.0'
+  s.add_dependency 'actionpack', '>= 5.0'
+  s.add_dependency 'request_store', '>= 1.0'
+  s.add_dependency 'activemodel', '>= 5.0'
+  s.add_dependency 'activemodel-serializers-xml', '>= 1.0'
 
   s.add_development_dependency 'ammeter'
   s.add_development_dependency 'rake'
   s.add_development_dependency 'rspec-rails'
   s.add_development_dependency 'minitest-rails'
   s.add_development_dependency 'capybara'
-  s.add_development_dependency 'active_model_serializers', '~> 0.10'
+  s.add_development_dependency 'active_model_serializers', '>= 0.10'
   s.add_development_dependency 'rubocop'
+  s.add_development_dependency 'simplecov'
 end

data/lib/draper.rb

@@ -23,6 +23,7 @@ require 'draper/factory'
 require 'draper/decorated_association'
 require 'draper/helper_support'
 require 'draper/view_context'
+require 'draper/query_methods'
 require 'draper/collection_decorator'
 require 'draper/undecorate'
 require 'draper/decorates_assigned'

data/lib/draper/automatic_delegation.rb

@@ -20,6 +20,8 @@ module Draper
 
     # @private
     def delegatable?(method)
+      return if private_methods(false).include?(method)
+
       object.respond_to?(method)
     end
 

data/lib/draper/collection_decorator.rb

@@ -2,6 +2,7 @@ module Draper
   class CollectionDecorator
     include Enumerable
     include Draper::ViewHelpers
+    include Draper::QueryMethods
     extend Draper::Delegation
 
     # @return the collection being decorated.
@@ -34,7 +35,7 @@ module Draper
     end
 
     class << self
-      alias_method :decorate, :new
+      alias :decorate :new
     end
 
     # @return [Array] the decorated items.
@@ -58,11 +59,12 @@ module Draper
       true
     end
 
-    alias_method :decorated_with?, :instance_of?
+    alias :decorated_with? :instance_of?
 
     def kind_of?(klass)
       decorated_collection.kind_of?(klass) || super
     end
+
     alias_method :is_a?, :kind_of?
 
     def replace(other)

data/lib/draper/configuration.rb

@@ -11,5 +11,13 @@ module Draper
     def default_controller=(controller)
       @@default_controller = controller
     end
+
+    def default_query_methods_strategy
+      @@default_query_methods_strategy ||= :active_record
+    end
+
+    def default_query_methods_strategy=(strategy)
+      @@default_query_methods_strategy = strategy
+    end
   end
 end

data/lib/draper/decoratable.rb

@@ -48,7 +48,6 @@ module Draper
     end
 
     module ClassMethods
-
       # Decorates a collection of objects. Used at the end of a scope chain.
       #
       # @example

data/lib/draper/decorated_association.rb

@@ -1,7 +1,6 @@
 module Draper
   # @private
   class DecoratedAssociation
-
     def initialize(owner, association, options)
       options.assert_valid_keys(:with, :scope, :context)
 
@@ -30,6 +29,5 @@ module Draper
 
       @decorated = factory.decorate(associated, context_args: owner.context)
     end
-
   end
 end

data/lib/draper/decorator.rb

@@ -12,7 +12,8 @@ module Draper
 
     # @return the object being decorated.
     attr_reader :object
-    alias_method :model, :object
+
+    alias :model :object
 
     # @return [Hash] extra data to be used in user-defined methods.
     attr_accessor :context
@@ -36,7 +37,7 @@ module Draper
     end
 
     class << self
-      alias_method :decorate, :new
+      alias :decorate :new
     end
 
     # Automatically delegates instance methods to the source object. Class
@@ -190,7 +191,8 @@ module Draper
     def kind_of?(klass)
       super || object.kind_of?(klass)
     end
-    alias_method :is_a?, :kind_of?
+
+    alias :is_a? :kind_of?
 
     # Checks if `self.instance_of?(klass)` or `object.instance_of?(klass)`
     #
@@ -225,7 +227,7 @@ module Draper
     # @return [Class] the class created by {decorate_collection}.
     def self.collection_decorator_class
       name = collection_decorator_name
-      name_constant = name && name.safe_constantize
+      name_constant = name&.safe_constantize
 
       name_constant || Draper::CollectionDecorator
     end
@@ -248,7 +250,7 @@ module Draper
 
     def self.inferred_object_class
       name = object_class_name
-      name_constant = name && name.safe_constantize
+      name_constant = name&.safe_constantize
       return name_constant unless name_constant.nil?
 
       raise Draper::UninferrableObjectError.new(self)
@@ -256,7 +258,7 @@ module Draper
 
     def self.collection_decorator_name
       singular = object_class_name
-      plural = singular && singular.pluralize
+      plural = singular&.pluralize
 
       "#{plural}Decorator" unless plural == singular
     end

data/lib/draper/delegation.rb

@@ -7,7 +7,7 @@ module Draper
     #   @return [void]
     def delegate(*methods)
       options = methods.extract_options!
-      super *methods, options.reverse_merge(to: :object)
+      super(*methods, **options.reverse_merge(to: :object))
     end
   end
 end

data/lib/draper/finders.rb

@@ -3,7 +3,6 @@ module Draper
   # do not have to extend this module directly; it is extended by
   # {Decorator.decorates_finders}.
   module Finders
-
     def find(id, options = {})
       decorate(object_class.find(id), options)
     end

data/lib/draper/helper_proxy.rb

@@ -2,7 +2,6 @@ module Draper
   # Provides access to helper methods - both Rails built-in helpers, and those
   # defined in your application.
   class HelperProxy
-
     # @overload initialize(view_context)
     def initialize(view_context)
       @view_context = view_context

data/lib/draper/lazy_helpers.rb

@@ -3,13 +3,11 @@ module Draper
   # so that you can stop typing `h.` everywhere, at the cost of mixing in a
   # bazillion methods.
   module LazyHelpers
-
     # Sends missing methods to the {HelperProxy}.
     def method_missing(method, *args, &block)
       helpers.send(method, *args, &block)
     rescue NoMethodError
       super
     end
-
   end
 end

data/lib/draper/query_methods.rb

@@ -0,0 +1,23 @@
+require_relative 'query_methods/load_strategy'
+
+module Draper
+  module QueryMethods
+    # Proxies missing query methods to the source class if the strategy allows.
+    def method_missing(method, *args, &block)
+      return super unless strategy.allowed? method
+
+      object.send(method, *args, &block).decorate(with: decorator_class, context: context)
+    end
+
+    def respond_to_missing?(method, include_private = false)
+      strategy.allowed?(method) || super
+    end
+
+    private
+
+    # Configures the strategy used to proxy the query methods, which defaults to `:active_record`.
+    def strategy
+      @strategy ||= LoadStrategy.new(Draper.default_query_methods_strategy)
+    end
+  end
+end

data/lib/draper/query_methods/load_strategy.rb

@@ -0,0 +1,21 @@
+module Draper
+  module QueryMethods
+    module LoadStrategy
+      def self.new(name)
+        const_get(name.to_s.camelize).new
+      end
+
+      class ActiveRecord
+        def allowed?(method)
+          ::ActiveRecord::Relation::VALUE_METHODS.include? method
+        end
+      end
+
+      class Mongoid
+        def allowed?(method)
+          raise NotImplementedError
+        end
+      end
+    end
+  end
+end

data/lib/draper/test_case.rb

@@ -3,9 +3,9 @@ module Draper
 
   class TestCase < ::ActiveSupport::TestCase
     module ViewContextTeardown
-      def teardown
-        super
+      def before_setup
         Draper::ViewContext.clear!
+        super
       end
     end
 
@@ -21,7 +21,7 @@ module Draper
       end
 
       include Draper::ViewHelpers::ClassMethods
-      alias_method :helper, :helpers
+      alias :helper :helpers
     end
 
     include Behavior

data/lib/draper/undecorate.rb

@@ -6,7 +6,7 @@ module Draper
       object
     end
   end
-  
+
   def self.undecorate_chain(object)
     if object.respond_to?(:decorated?) && object.decorated?
       undecorate_chain(object.object)

data/lib/draper/version.rb

@@ -1,3 +1,3 @@
 module Draper
-  VERSION = '3.0.1'
+  VERSION = '4.0.1'
 end

data/lib/draper/view_context/build_strategy.rb

@@ -2,7 +2,6 @@ module Draper
   module ViewContext
     # @private
     module BuildStrategy
-
       def self.new(name, &block)
         const_get(name.to_s.camelize).new(&block)
       end
@@ -13,7 +12,7 @@ module Draper
         end
 
         def call
-          view_context_class.new
+          view_context_class.respond_to?(:empty) ? view_context_class.empty : view_context_class.new
         end
 
         private
@@ -51,7 +50,6 @@ module Draper
           ActionController::TestRequest.method(:create).parameters.first == [:req, :controller_class]
         end
       end
-
     end
   end
 end

data/lib/draper/view_helpers.rb

@@ -5,7 +5,6 @@ module Draper
     extend ActiveSupport::Concern
 
     module ClassMethods
-
       # Access the helpers proxy to call built-in and user-defined
       # Rails helpers from a class context.
       #
@@ -13,8 +12,8 @@ module Draper
       def helpers
         Draper::ViewContext.current
       end
-      alias_method :h, :helpers
 
+      alias :h :helpers
     end
 
     # Access the helpers proxy to call built-in and user-defined
@@ -24,14 +23,15 @@ module Draper
     def helpers
       Draper::ViewContext.current
     end
-    alias_method :h, :helpers
+
+    alias :h :helpers
 
     # Alias for `helpers.localize`, since localize is something that's used
     # quite often. Further aliased to `l` for convenience.
     def localize(*args)
       helpers.localize(*args)
     end
-    alias_method :l, :localize
 
+    alias :l :localize
   end
 end

data/spec/draper/collection_decorator_spec.rb

@@ -7,7 +7,6 @@ module Draper
 
     describe "#initialize" do
       describe "options validation" do
-
         it "does not raise error on valid options" do
           valid_options = {with: Decorator, context: {}}
           expect{CollectionDecorator.new([], valid_options)}.not_to raise_error
@@ -287,6 +286,5 @@ module Draper
         expect(decorator.replace([:foo, :bar])).to be decorator
       end
     end
-
   end
 end

data/spec/draper/configuration_spec.rb

@@ -6,20 +6,44 @@ module Draper
       Draper.configure { |config| expect(config).to be Draper }
     end
 
-    it 'defaults default_controller to ApplicationController' do
-      expect(Draper.default_controller).to be ApplicationController
+    describe '#default_controller' do
+      it 'defaults default_controller to ApplicationController' do
+        expect(Draper.default_controller).to be ApplicationController
+      end
+
+      it 'allows customizing default_controller through configure' do
+        default = Draper.default_controller
+
+        Draper.configure do |config|
+          config.default_controller = CustomController
+        end
+
+        expect(Draper.default_controller).to be CustomController
+
+        Draper.default_controller = default
+      end
     end
 
-    it 'allows customizing default_controller through configure' do
-      default = Draper.default_controller
+    describe '#default_query_methods_strategy' do
+      let!(:default) { Draper.default_query_methods_strategy }
+
+      subject { Draper.default_query_methods_strategy }
 
-      Draper.configure do |config|
-        config.default_controller = CustomController
+      context 'when there is no custom strategy' do
+        it { is_expected.to eq(:active_record) }
       end
 
-      expect(Draper.default_controller).to be CustomController
+      context 'when using a custom strategy' do
+        before do
+          Draper.configure do |config|
+            config.default_query_methods_strategy = :mongoid
+          end
+        end
 
-      Draper.default_controller = default
+        after { Draper.default_query_methods_strategy = default }
+
+        it { is_expected.to eq(:mongoid) }
+      end
     end
   end
 end

data/spec/draper/decoratable_spec.rb

@@ -3,7 +3,6 @@ require 'support/shared_examples/decoratable_equality'
 
 module Draper
   describe Decoratable do
-
     describe "#decorate" do
       it "returns a decorator for self" do
         product = Product.new
@@ -11,7 +10,7 @@ module Draper
 
         expect(decorator).to be_a ProductDecorator
         expect(decorator.object).to be product
-     end
+      end
 
       it "accepts context" do
         context = {some: "context"}
@@ -232,6 +231,5 @@ module Draper
         end
       end
     end
-
   end
 end

data/spec/draper/decorated_association_spec.rb

@@ -2,7 +2,6 @@ require 'spec_helper'
 
 module Draper
   describe DecoratedAssociation do
-
     describe "#initialize" do
       it "accepts valid options" do
         valid_options = {with: Decorator, scope: :foo, context: {}}
@@ -79,6 +78,5 @@ module Draper
         end
       end
     end
-
   end
 end

data/spec/draper/decorator_spec.rb

@@ -664,6 +664,38 @@ module Draper
           expect{decorator.hello_world}.to raise_error NoMethodError
           expect(decorator.methods).not_to include :hello_world
         end
+
+        context 'when decorator overrides a public method defined on the object with a private' do
+          let(:decorator_class) do
+            Class.new(Decorator) do
+              private
+
+              def hello_world
+                'hello world'
+              end
+            end
+          end
+
+          let(:object) { Class.new { def hello_world; end }.new }
+
+          it 'does not delegate the public method defined on the object' do
+            decorator = decorator_class.new(object)
+
+            expect{ decorator.hello_world }.to raise_error NoMethodError
+          end
+        end
+
+        context 'when delegated method has the same name as private method defined on another object' do
+          let(:decorator_class) { Class.new(Decorator) }
+          let(:object) { Class.new { def print; end }.new }
+
+          it 'delegates the public method defined on the object' do
+            decorator = decorator_class.new(object)
+
+            # `print` private method is defined on `Object`
+            expect{ decorator.print }.not_to raise_error NoMethodError
+          end
+        end
       end
 
       context ".method_missing" do

data/spec/draper/draper_spec.rb

@@ -1,5 +1,6 @@
 require 'spec_helper'
 require 'support/shared_examples/view_helpers'
+SimpleCov.command_name 'test:unit'
 
 module Draper
   describe Draper do

data/spec/draper/factory_spec.rb

@@ -2,7 +2,6 @@ require 'spec_helper'
 
 module Draper
   describe Factory do
-
     describe "#initialize" do
       it "accepts valid options" do
         valid_options = {with: Decorator, context: {foo: "bar"}}
@@ -88,11 +87,9 @@ module Draper
         end
       end
     end
-
   end
 
   describe Factory::Worker do
-
     describe "#call" do
       it "calls the decorator method" do
         object = double
@@ -246,6 +243,5 @@ module Draper
         end
       end
     end
-
   end
 end

data/spec/draper/query_methods/load_strategy_spec.rb

@@ -0,0 +1,26 @@
+require 'spec_helper'
+require 'active_record'
+
+module Draper
+  module QueryMethods
+    describe LoadStrategy do
+      describe '#new' do
+        subject { described_class.new(:active_record) }
+
+        it { is_expected.to be_an_instance_of(LoadStrategy::ActiveRecord) }
+      end
+    end
+
+    describe LoadStrategy::ActiveRecord do
+      describe '#allowed?' do
+        it 'checks whether or not ActiveRecord::Relation::VALUE_METHODS has the given method' do
+          allow(::ActiveRecord::Relation::VALUE_METHODS).to receive(:include?)
+
+          described_class.new.allowed? :foo
+
+          expect(::ActiveRecord::Relation::VALUE_METHODS).to have_received(:include?).with(:foo)
+        end
+      end
+    end
+  end
+end

data/spec/draper/query_methods_spec.rb

@@ -0,0 +1,70 @@
+require 'spec_helper'
+require_relative '../dummy/app/decorators/post_decorator'
+
+Post = Struct.new(:id) { }
+
+module Draper
+  describe QueryMethods do
+    let(:fake_strategy) { instance_double(QueryMethods::LoadStrategy::ActiveRecord) }
+
+    before { allow(QueryMethods::LoadStrategy).to receive(:new).and_return(fake_strategy) }
+
+    describe '#method_missing' do
+      let(:collection) { [ Post.new, Post.new ] }
+      let(:collection_context) { { user: 'foo' } }
+      let(:collection_decorator) { PostDecorator.decorate_collection(collection, context: collection_context) }
+
+      context 'when strategy allows collection to call the method' do
+        let(:results) { spy(:results) }
+
+        before do
+          allow(fake_strategy).to receive(:allowed?).with(:some_query_method).and_return(true)
+          allow(collection).to receive(:send).with(:some_query_method).and_return(results)
+        end
+
+        it 'calls the method on the collection and decorate it results' do
+          collection_decorator.some_query_method
+
+          expect(results).to have_received(:decorate)
+        end
+
+        it 'calls the method on the collection and keeps the decoration options' do
+          collection_decorator.some_query_method
+
+          expect(results).to have_received(:decorate).with({ context: collection_context, with: PostDecorator })
+        end
+      end
+
+      context 'when strategy does not allow collection to call the method' do
+        before { allow(fake_strategy).to receive(:allowed?).with(:some_query_method).and_return(false) }
+
+        it 'raises NoMethodError' do
+          expect { collection_decorator.some_query_method }.to raise_exception(NoMethodError)
+        end
+      end
+    end
+
+    describe "#respond_to?" do
+      let(:collection) { [ Post.new, Post.new ] }
+      let(:collection_decorator) { PostDecorator.decorate_collection(collection) }
+
+      subject { collection_decorator.respond_to?(:some_query_method) }
+
+      context 'when strategy allows collection to call the method' do
+        before do
+          allow(fake_strategy).to receive(:allowed?).with(:some_query_method).and_return(true)
+        end
+
+        it { is_expected.to eq(true) }
+      end
+
+      context 'when strategy does not allow collection to call the method' do
+        before do
+          allow(fake_strategy).to receive(:allowed?).with(:some_query_method).and_return(false)
+        end
+
+        it { is_expected.to eq(false) }
+      end
+    end
+  end
+end

data/spec/dummy/app/assets/config/manifest.js

@@ -0,0 +1,3 @@
+//= link_tree ../images
+//= link_directory ../javascripts .js
+//= link_directory ../stylesheets .css

data/spec/dummy/app/controllers/application_controller.rb

@@ -0,0 +1,2 @@
+class ApplicationController < ActionController::Base
+end

data/spec/generators/controller/controller_generator_spec.rb

@@ -3,6 +3,7 @@ require 'dummy/config/environment'
 require 'ammeter/init'
 require 'generators/controller_override'
 require 'generators/rails/decorator_generator'
+SimpleCov.command_name 'test:generator'
 
 describe Rails::Generators::ControllerGenerator do
   destination File.expand_path("../tmp", __FILE__)

data/spec/integration/integration_spec.rb

@@ -1,6 +1,7 @@
 require 'spec_helper'
 require 'support/dummy_app'
 require 'support/matchers/have_text'
+SimpleCov.command_name 'test:integration'
 
 app = DummyApp.new(ENV["RAILS_ENV"])
 spec_types = {

data/spec/spec_helper.rb

@@ -1,3 +1,10 @@
+require 'simplecov'
+SimpleCov.start do
+  add_filter 'spec'
+  add_group 'Draper', 'lib/draper'
+  add_group 'Generators', 'lib/generators'
+end
+
 require 'bundler/setup'
 require 'draper'
 require 'action_controller'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: draper
 version: !ruby/object:Gem::Version
-  version: 3.0.1
+  version: 4.0.1
 platform: ruby
 authors:
 - Jeff Casimir
@@ -9,76 +9,76 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-10-12 00:00:00.000000000 Z
+date: 2020-03-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '5.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '5.0'
 - !ruby/object:Gem::Dependency
   name: actionpack
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '5.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '5.0'
 - !ruby/object:Gem::Dependency
   name: request_store
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '1.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '1.0'
 - !ruby/object:Gem::Dependency
   name: activemodel
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '5.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '5.0'
 - !ruby/object:Gem::Dependency
   name: activemodel-serializers-xml
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '1.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '1.0'
 - !ruby/object:Gem::Dependency
@@ -155,14 +155,14 @@ dependencies:
   name: active_model_serializers
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0.10'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0.10'
 - !ruby/object:Gem::Dependency
@@ -179,6 +179,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Draper adds an object-oriented layer of presentation logic to your Rails
   apps.
 email:
@@ -220,6 +234,8 @@ files:
 - lib/draper/helper_proxy.rb
 - lib/draper/helper_support.rb
 - lib/draper/lazy_helpers.rb
+- lib/draper/query_methods.rb
+- lib/draper/query_methods/load_strategy.rb
 - lib/draper/railtie.rb
 - lib/draper/tasks/test.rake
 - lib/draper/test/devise_helper.rb
@@ -255,6 +271,8 @@ files:
 - spec/draper/finders_spec.rb
 - spec/draper/helper_proxy_spec.rb
 - spec/draper/lazy_helpers_spec.rb
+- spec/draper/query_methods/load_strategy_spec.rb
+- spec/draper/query_methods_spec.rb
 - spec/draper/undecorate_chain_spec.rb
 - spec/draper/undecorate_spec.rb
 - spec/draper/view_context/build_strategy_spec.rb
@@ -262,6 +280,8 @@ files:
 - spec/draper/view_helpers_spec.rb
 - spec/dummy/.rspec
 - spec/dummy/Rakefile
+- spec/dummy/app/assets/config/manifest.js
+- spec/dummy/app/controllers/application_controller.rb
 - spec/dummy/app/controllers/base_controller.rb
 - spec/dummy/app/controllers/localized_urls.rb
 - spec/dummy/app/controllers/posts_controller.rb
@@ -364,8 +384,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.11
+rubygems_version: 3.0.8
 signing_key: 
 specification_version: 4
 summary: View Models for Rails
@@ -382,6 +401,8 @@ test_files:
 - spec/draper/finders_spec.rb
 - spec/draper/helper_proxy_spec.rb
 - spec/draper/lazy_helpers_spec.rb
+- spec/draper/query_methods/load_strategy_spec.rb
+- spec/draper/query_methods_spec.rb
 - spec/draper/undecorate_chain_spec.rb
 - spec/draper/undecorate_spec.rb
 - spec/draper/view_context/build_strategy_spec.rb
@@ -389,6 +410,8 @@ test_files:
 - spec/draper/view_helpers_spec.rb
 - spec/dummy/.rspec
 - spec/dummy/Rakefile
+- spec/dummy/app/assets/config/manifest.js
+- spec/dummy/app/controllers/application_controller.rb
 - spec/dummy/app/controllers/base_controller.rb
 - spec/dummy/app/controllers/localized_urls.rb
 - spec/dummy/app/controllers/posts_controller.rb