dry-auto_inject 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5c8a5ebc379dd91da33ab5b74b37b8b5233ce496
-  data.tar.gz: 530f763d2a088affcfba27591b4b414b0005b1e5
+  metadata.gz: e19494845c22047aae417893bf936a829356f39d
+  data.tar.gz: 7bc7e22907061ae09efbad0003a161b00c9b6210
 SHA512:
-  metadata.gz: fb55775c7a465d2f796e56421c6098abef986ea5c177b18851e4ba2cce6111bbdf8dad8fd873d52072e674901e7fcaeac23a304cae5cd6198e0581f7c5d32c6a
-  data.tar.gz: 0c9c3d0012f2978420fa74fd536eed7c77cdf0d579d85a28afb05d79399f02d31e770b31718679babe16befc8c1a3de22df0635bda5ef495f2475b12c69e0378
+  metadata.gz: eee0b597419ae7d17edcbfa400f21af26bd8f2c736e055ab121596cc8b59845f0d77bb02f7a542f7df4251a28535e57fb870f87bcfafffba5ff72537b03c66e4
+  data.tar.gz: f50518d06a4fd563824f09523513dc4b19f5fe63900941ce6ac42aff72551054df6bf0daee2556ea3bd3985a4ee863c6891ea7be88d5b4e588732afd0a7f6873

data/.rspec

@@ -1,3 +1,2 @@
---format documentation
 --color
--r ./spec/spec_helper.rb
+--require spec_helper

data/.travis.yml

@@ -5,9 +5,9 @@ bundler_args: --without console
 script:
   - bundle exec rake spec
 rvm:
-  - 2.0
   - 2.1
   - 2.2
+  - 2.3.1
   - rbx
   - ruby-head
   - jruby-head
@@ -19,6 +19,9 @@ matrix:
     - rvm: rbx
     - rvm: ruby-head
     - rvm: jruby-head
+addons:
+  code_climate:
+    repo_token: 029b5aa100b82b6710e285597b7eabe760357b5dd1f626514371356055378d8c
 notifications:
   email: false
   webhooks:

data/CHANGELOG.md

@@ -1,4 +1,35 @@
-# v.0.3.0, 2016-06-02
+# 0.4.0 / 2016-07-26
+
+### Added
+
+- Support for strategy chaining, which is helpful in opting for alternatives to an application's normal strategy (timriley in [#25](https://github.com/dry-rb/dry-auto_inject/pull/25))
+
+    ```ruby
+    # Define the application's injector with a non-default
+    MyInject = Dry::AutoInject(MyContainer).hash
+
+    # Opt for a different strategy in a particular class
+    class MyClass
+      include MyInject.args["foo"]
+    end
+
+    # You can chain as long as you want (silly example to demonstrate the flexibility)
+    class OtherClass
+      include MyInject.args.hash.kwargs.args["foo"]
+    end
+    ```
+
+### Changed
+
+- Use a `BasicObject`-based environment for the injector builder API instead of the previous `define_singleton_method`-based approach, which had negative performance characteristics (timriley in [#26](https://github.com/dry-rb/dry-auto_inject/pull/26))
+
+### Fixed
+
+- Fixed issue with kwargs injectors used at multiple points in a class inheritance heirarchy (flash-gordon in [#27](https://github.com/dry-rb/dry-auto_inject/pull/27))
+
+[Compare v0.3.0...v0.4.0](https://github.com/dryrb/dry-auto_inject/compare/v0.3.0...v0.4.0)
+
+# 0.3.0, 2016-06-02
 
 ### Added
 
@@ -7,7 +38,7 @@
     These strategies can be accessed via methods on the main builder object:
 
     ```ruby
-    MyInject = Dry::AutoInect(my_container)
+    MyInject = Dry::AutoInject(my_container)
 
     class MyClass
       include MyInject.hash["my_dep"]
@@ -59,7 +90,7 @@
     These aliases enable you to specify your own name for dependencies, both for their local readers and their keys in the kwargs- and hash-based initializers. Specify aliases by passing a hash of names:
 
     ```ruby
-    MyInject = Dry::AutoInect(my_container)
+    MyInject = Dry::AutoInject(my_container)
 
     class MyClass
       include MyInject[my_dep: "some_other.dep"]
@@ -84,6 +115,8 @@
 * `kwargs` is the new default injection strategy
 * Rubinius support is not available for the `kwargs` strategy (see [#18](https://github.com/dry-rb/dry-auto_inject/issues/18))
 
+[Compare v0.2.0...v0.3.0](https://github.com/dryrb/dry-auto_inject/compare/v0.2.0...v0.3.0)
+
 # v0.2.0 2016-02-09
 
 ### Added

data/Gemfile

@@ -3,6 +3,10 @@ source 'https://rubygems.org'
 # Specify your gem's dependencies in dry-container.gemspec
 gemspec
 
+group :test do
+  gem "codeclimate-test-reporter", require: nil
+end
+
 group :tools do
   gem 'byebug', platforms: :mri
   gem 'pry'

data/LICENSE

@@ -0,0 +1,20 @@
+Copyright (c) 2015-2016 Piotr Solnica
+
+Permission is hereby granted, free of charge, to any person obtaining
+a copy of this software and associated documentation files (the
+"Software"), to deal in the Software without restriction, including
+without limitation the rights to use, copy, modify, merge, publish,
+distribute, sublicense, and/or sell copies of the Software, and to
+permit persons to whom the Software is furnished to do so, subject to
+the following conditions:
+
+The above copyright notice and this permission notice shall be
+included in all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

data/dry-auto_inject.gemspec

@@ -8,6 +8,7 @@ Gem::Specification.new do |spec|
   spec.version       = Dry::AutoInject::VERSION.dup
   spec.authors       = ['Piotr Solnica']
   spec.email         = ['piotr.solnica@gmail.com']
+  spec.license       = 'MIT'
 
   spec.summary       = 'Container-agnostic automatic constructor injection'
   spec.homepage      = 'https://github.com/dryrb/dry-auto_inject'
@@ -17,7 +18,7 @@ Gem::Specification.new do |spec|
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ['lib']
 
-  spec.required_ruby_version = '>= 2.0.0'
+  spec.required_ruby_version = '>= 2.1.0'
 
   spec.add_runtime_dependency 'dry-container', '~> 0.3.4'
 

data/lib/dry/auto_inject/builder.rb

@@ -3,7 +3,7 @@ require 'dry/auto_inject/injector'
 
 module Dry
   module AutoInject
-    class Builder
+    class Builder < BasicObject
       # @api private
       attr_reader :container
 
@@ -13,19 +13,26 @@ module Dry
       def initialize(container, options = {})
         @container = container
         @strategies = options.fetch(:strategies) { Strategies }
-
-        strategies.keys.each do |strategy_name|
-          define_singleton_method(strategy_name) do
-            strategy = strategies[strategy_name]
-            Injector.new(container, strategy)
-          end
-        end
       end
 
       # @api public
       def [](*dependency_names)
         default[*dependency_names]
       end
+
+      def respond_to?(name, include_private = false)
+        Builder.public_instance_methods.include?(name) || strategies.key?(name)
+      end
+
+      private
+
+      def method_missing(name, *args, &block)
+        if strategies.key?(name)
+          Injector.new(container, strategies[name], builder: self)
+        else
+          super
+        end
+      end
     end
   end
 end

data/lib/dry/auto_inject/injector.rb

@@ -2,7 +2,7 @@ require 'dry/auto_inject/strategies'
 
 module Dry
   module AutoInject
-    class Injector
+    class Injector < BasicObject
       # @api private
       attr_reader :container
 
@@ -10,14 +10,28 @@ module Dry
       attr_reader :strategy
 
       # @api private
-      def initialize(container, strategy)
+      attr_reader :builder
+
+      # @api private
+      def initialize(container, strategy, builder:)
         @container = container
         @strategy = strategy
+        @builder = builder
       end
 
       def [](*dependency_names)
         strategy.new(container, *dependency_names)
       end
+
+      def respond_to?(name, include_private = false)
+        Injector.instance_methods.include?(name) || builder.respond_to?(name)
+      end
+
+      private
+
+      def method_missing(name, *args, &block)
+        builder.__send__(name)
+      end
     end
   end
 end

data/lib/dry/auto_inject/strategies/kwargs.rb

@@ -49,7 +49,11 @@ module Dry
 
           # Pass through any non-dependency args if the super method accepts `**args`
           if super_method.parameters.any? { |type, _| type == :keyrest }
-            super_params += ', **args'
+            if super_params.empty?
+              super_params = '**args'
+            else
+              super_params += ', **args'
+            end
           end
 
           instance_mod.class_eval <<-RUBY, __FILE__, __LINE__ + 1

data/lib/dry/auto_inject/version.rb

@@ -1,5 +1,5 @@
 module Dry
   module AutoInject
-    VERSION = '0.3.0'.freeze
+    VERSION = '0.4.0'.freeze
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: dry-auto_inject
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - Piotr Solnica
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2016-06-01 00:00:00.000000000 Z
+date: 2016-07-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: dry-container
@@ -80,6 +80,7 @@ files:
 - ".travis.yml"
 - CHANGELOG.md
 - Gemfile
+- LICENSE
 - README.md
 - Rakefile
 - bin/console
@@ -98,7 +99,8 @@ files:
 - lib/dry/auto_inject/version.rb
 - rakelib/rubocop.rake
 homepage: https://github.com/dryrb/dry-auto_inject
-licenses: []
+licenses:
+- MIT
 metadata: {}
 post_install_message: 
 rdoc_options: []
@@ -108,7 +110,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.0.0
+      version: 2.1.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="