dependency_injection 0.3.1 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0a4efb8f1e9100648b5f0b63615f783523c84173
-  data.tar.gz: e90d72986360efd613cb027bf3446a247a1ecc58
+  metadata.gz: bee91995df1f5fc2220d223c7387a9e4c384e9fe
+  data.tar.gz: 15be55e7503b817ee7d3d79d1e88eb143c752485
 SHA512:
-  metadata.gz: 58fa9d7f5421a2037c3d8d14a8a6ac0f302befdeaa472141c0b90b7626437fabbb1cc3e4d8a9eda078721fd020e344fd1436bb89fe6545347352793fe4ed2cdf
-  data.tar.gz: 3beddb5b426c1dd99025f0bbf88f5bfea2079e5ca7166d4fd6882aadfe4e3cc1d0991b35d39a64a15dd235b9e1ebd98147482be62f1400d2f501f058b0517635
+  metadata.gz: a519c2ff82347e5ea4718adde01b6b87fc80b29d8409b4e2f4e1567ffbe551923a8ad79351271ffb6d83036360cf868c2342d7cfd8f5817da242df71928d7215
+  data.tar.gz: 1d12edeafb9daa99ffc1919f9f9f834fff87043681cb3572086413345f4b2076a85cce1b5886468e8e63530ddc2bf306cab356ba8460f48dd446c5263078aba9

data/CHANGELOG.md

@@ -1,3 +1,9 @@
+# 0.4.0 (2014-02-23)
+
+* Fix an error when a service use another service
+* Add ERB support in YAML loader
+* Add file_path option
+
 # 0.3.1 (2013-09-17)
 
 * Nothing adding/removed. Just fix the Gemfile.lock

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    dependency_injection (0.3.1)
+    dependency_injection (0.4.0)
       activesupport
 
 GEM
@@ -17,7 +17,7 @@ GEM
       rest-client
       simplecov (>= 0.7)
       thor
-    i18n (0.6.4)
+    i18n (0.6.9)
     metaclass (0.0.1)
     mime-types (1.23)
     minitest (5.0.6)

data/README.md

@@ -172,6 +172,8 @@ And here's some more details about each keyword:
 
 * `calls`: An array containing an array of each instance method and its parameters. Note that you only need to define the methods during your class instantiation.
 
+* `file_path`: A string containing the file path to require to load the class.
+
 * `lazy`: Returns a Proxy Object if true. The _real_ object will only be instantiated at the first method call.
 
 * `alias`: A string containing the target service name.

data/lib/dependency_injection/container.rb

@@ -15,8 +15,12 @@ module DependencyInjection
       @parameters[name] = value
     end
 
+    def find(name)
+      @definitions[name]
+    end
+
     def get(name)
-      if (definition = @definitions[name])
+      if (definition = self.find(name))
         definition.object
       end
     end

data/lib/dependency_injection/definition.rb

@@ -3,11 +3,12 @@ require 'dependency_injection/scope_widening_injection_error'
 
 module DependencyInjection
   class Definition
-    attr_accessor :arguments, :configurator, :klass_name, :method_calls, :scope
+    attr_accessor :arguments, :configurator, :file_path, :klass_name, :method_calls, :scope
 
     def initialize(klass_name, container)
       @container        = container
       self.arguments    = []
+      self.file_path    = nil
       self.klass_name   = klass_name
       self.method_calls = {}
       self.scope        = :container
@@ -50,6 +51,7 @@ module DependencyInjection
     end
 
     def initialize_object
+      require_object
       object = self.klass.new(*resolve(self.arguments))
       self.method_calls.each { |method_name, arguments| object.send(method_name, *resolve(arguments)) }
       if self.configurator
@@ -61,10 +63,26 @@ module DependencyInjection
       object
     end
 
+    def object_already_required?
+      true if Kernel.const_get(self.klass_name)
+    rescue
+      false
+    end
+
     def prototype_scoped_object
       initialize_object
     end
 
+    def require_object
+      return if object_already_required?
+
+      if self.file_path
+        require self.file_path
+      else
+        require self.klass_name.underscore
+      end
+    end
+
     def resolve(arguments)
       resolve_references(resolve_container_parameters(arguments))
     end
@@ -82,8 +100,9 @@ module DependencyInjection
     def resolve_references(arguments)
       arguments.map do |argument|
         if /^@(?<reference_name>.*)/ =~ argument
-          reference = @container.get(reference_name)
-          raise ScopeWideningInjectionError if reference.scope == :prototype && scope == :container
+          reference_definition = @container.find(reference_name)
+          reference            = reference_definition.object
+          raise ScopeWideningInjectionError if reference_definition.scope == :prototype && scope == :container
 
           reference
         else

data/lib/dependency_injection/loaders/yaml.rb

@@ -1,3 +1,4 @@
+require 'erb'
 require 'yaml'
 
 module DependencyInjection
@@ -8,7 +9,7 @@ module DependencyInjection
       end
 
       def load(filename)
-        file = YAML::load_file(filename)
+        file = load_file(filename)
         add_parameters(file['parameters']) if file['parameters']
         add_services(file['services']) if file['services']
       end
@@ -38,7 +39,8 @@ module DependencyInjection
       def add_standard_service(name, parameters)
         lazy_load  = parameters['lazy'] || false
         definition = @container.register(name, parameters['class'], lazy_load)
-        definition.scope = parameters['scope'] if parameters['scope']
+        definition.scope     = parameters['scope'] if parameters['scope']
+        definition.file_path = parameters['file_path'] if parameters['file_path']
         definition.add_arguments(*parameters['arguments']) if parameters['arguments']
         if (configurator = parameters['configurator'])
           definition.add_configurator(configurator[0], configurator[1])
@@ -47,6 +49,10 @@ module DependencyInjection
           parameters['calls'].each { |method_name, arguments| definition.add_method_call(method_name, *arguments) }
         end
       end
+
+      def load_file(filename)
+        YAML::load(ERB.new(IO.read(filename)).result)
+      end
     end
   end
 end

data/lib/dependency_injection/version.rb

@@ -1,3 +1,3 @@
 module DependencyInjection
-  VERSION = '0.3.1'
+  VERSION = '0.4.0'
 end

data/test/dependency_injection/loaders/test_yaml.rb

@@ -8,28 +8,28 @@ class TestYaml < Minitest::Test
   end
 
   def test_loading_file_without_parameters
-    YAML.stubs(:load_file).with('services.yml').returns({ 'services' => [] })
+    @yaml_loader.stubs(:load_file).with('services.yml').returns({ 'services' => [] })
     @yaml_loader.expects(:add_parameters).never
 
     @yaml_loader.load('services.yml')
   end
 
   def test_loading_file_with_parameters
-    YAML.stubs(:load_file).with('services.yml').returns({'parameters' => { 'key_1' => 'value_1' }, 'services' => [] })
+    @yaml_loader.stubs(:load_file).with('services.yml').returns({'parameters' => { 'key_1' => 'value_1' }, 'services' => [] })
     @yaml_loader.expects(:add_parameters).with({ 'key_1' => 'value_1' })
 
     @yaml_loader.load('services.yml')
   end
 
   def test_loading_file_without_services
-    YAML.stubs(:load_file).with('services.yml').returns({ 'parameters' => [] })
+    @yaml_loader.stubs(:load_file).with('services.yml').returns({ 'parameters' => [] })
     @yaml_loader.expects(:add_services).never
 
     @yaml_loader.load('services.yml')
   end
 
   def test_loading_file_with_services
-    YAML.stubs(:load_file).with('services.yml').returns({ 'parameters' => {}, 'services' => { 'service_1' => { 'class' => 'MyKlass' }}})
+    @yaml_loader.stubs(:load_file).with('services.yml').returns({ 'parameters' => {}, 'services' => { 'service_1' => { 'class' => 'MyKlass' }}})
     @yaml_loader.expects(:add_services).with({ 'service_1' => { 'class' => 'MyKlass' }})
 
     @yaml_loader.load('services.yml')
@@ -152,4 +152,12 @@ class TestYaml < Minitest::Test
 
     @yaml_loader.send(:add_standard_service, 'key_1', { 'class' => 'MyKlass', 'configurator' => ['ConfiguratorKlass', 'method_name'] })
   end
+
+  def test_adding_standard_service_with_file_path
+    definition = mock
+    @container.stubs(:register).with('key_1', 'MyKlass', false).returns(definition)
+    definition.expects(:file_path=).with('path/to/file')
+
+    @yaml_loader.send(:add_standard_service, 'key_1', { 'class' => 'MyKlass', 'file_path' => 'path/to/file' })
+  end
 end

data/test/dependency_injection/test_container.rb

@@ -29,6 +29,15 @@ class TestContainer < Minitest::Test
     assert_equal({ 'my.parameter' => 'other value' }, @container.parameters)
   end
 
+  def test_find_a_registered_definition_returns_a_definition
+    @container.register('my_definition', 'MyDefinition')
+    assert_equal(@definition, @container.find('my_definition'))
+  end
+
+  def test_find_a_not_registered_definition_returns_nil
+    assert_equal(nil, @container.find('my_definition'))
+  end
+
   def test_getting_a_registered_definition_returns_an_object
     @container.register('my_definition', 'MyDefinition')
     assert_equal(@final_object, @container.get('my_definition'))

data/test/dependency_injection/test_definition.rb

@@ -71,6 +71,7 @@ class TestDefinition < Minitest::Test
   end
 
   def test_getting_object_with_arguments
+    @definition.stubs(:require_object).returns(mock)
     final_object = mock
     final_class  = mock
     @definition.stubs(:klass).returns(final_class)
@@ -81,6 +82,7 @@ class TestDefinition < Minitest::Test
   end
 
   def test_getting_object_without_arguments
+    @definition.stubs(:require_object).returns(mock)
     final_object = mock
     final_class  = mock
     @definition.stubs(:klass).returns(final_class)
@@ -90,6 +92,7 @@ class TestDefinition < Minitest::Test
   end
 
   def test_getting_object_with_method_calls
+    @definition.stubs(:require_object).returns(mock)
     final_object = mock
     final_class  = mock
     @definition.stubs(:klass).returns(final_class)
@@ -104,6 +107,7 @@ class TestDefinition < Minitest::Test
   end
 
   def test_getting_object_without_configurator
+    @definition.stubs(:require_object).returns(mock)
     configurator_object = mock
     final_object        = mock
     final_class         = mock
@@ -117,6 +121,7 @@ class TestDefinition < Minitest::Test
   end
 
   def test_getting_object_with_configurator
+    @definition.stubs(:require_object).returns(mock)
     configurator_object = mock
     final_object        = mock
     final_class         = mock
@@ -189,38 +194,58 @@ class TestDefinition < Minitest::Test
   end
 
   def test_resolving_references_with_defintion_and_referenced_object_in_container_scope
-    referenced_object = mock
-    referenced_object.stubs(:scope).returns(:container)
+    referenced_definition = mock
+    referenced_object     = mock
+    referenced_definition.stubs(:object).returns(referenced_object)
+    referenced_definition.stubs(:scope).returns(:container)
     @definition.scope= :container
-    @container.stubs(:get).with('reference.name').returns(referenced_object)
+    @container.stubs(:find).with('reference.name').returns(referenced_definition)
 
     assert_equal(['first', referenced_object], @definition.send(:resolve_references, %w(first @reference.name)))
   end
 
   def test_resolving_references_with_defintion_and_referenced_object_in_prototype_scope
-    referenced_object = mock
-    referenced_object.stubs(:scope).returns(:prototype)
+    referenced_definition = mock
+    referenced_object     = mock
+    referenced_definition.stubs(:object).returns(referenced_object)
+    referenced_definition.stubs(:scope).returns(:prototype)
     @definition.scope= :prototype
-    @container.stubs(:get).with('reference.name').returns(referenced_object)
+    @container.stubs(:find).with('reference.name').returns(referenced_definition)
 
     assert_equal(['first', referenced_object], @definition.send(:resolve_references, %w(first @reference.name)))
   end
 
   def test_resolving_references_with_defintion_in_prototype_scope_and_referenced_object_in_container_scope
-    referenced_object = mock
-    referenced_object.stubs(:scope).returns(:container)
+    referenced_definition = mock
+    referenced_object     = mock
+    referenced_definition.stubs(:object).returns(referenced_object)
+    referenced_definition.stubs(:scope).returns(:container)
     @definition.scope= :prototype
-    @container.stubs(:get).with('reference.name').returns(referenced_object)
+    @container.stubs(:find).with('reference.name').returns(referenced_definition)
 
     assert_equal(['first', referenced_object], @definition.send(:resolve_references, %w(first @reference.name)))
   end
 
   def test_resolving_references_with_defintion_in_container_scope_and_referenced_object_in_prototype_scope
-    referenced_object = mock
-    referenced_object.stubs(:scope).returns(:prototype)
+    referenced_definition = mock
+    referenced_object     = mock
+    referenced_definition.stubs(:object).returns(referenced_object)
+    referenced_definition.stubs(:scope).returns(:prototype)
     @definition.scope= :container
-    @container.stubs(:get).with('reference.name').returns(referenced_object)
+    @container.stubs(:find).with('reference.name').returns(referenced_definition)
 
     assert_raises(ScopeWideningInjectionError) { @definition.send(:resolve_references, %w(first @reference.name)) }
   end
+
+  def test_require_object_with_file_path
+    @definition.expects(:require).with('/file/to/path').once
+    @definition.file_path = '/file/to/path'
+    @definition.send(:require_object)
+  end
+
+  def test_require_object_automatically
+    @definition.expects(:require).with('my_module/my_class').once
+    @definition.klass_name = 'MyModule::MyClass'
+    @definition.send(:require_object)
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: dependency_injection
 version: !ruby/object:Gem::Version
-  version: 0.3.1
+  version: 0.4.0
 platform: ruby
 authors:
 - Kevin Disneur
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-09-17 00:00:00.000000000 Z
+date: 2014-02-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -88,7 +88,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.6
+rubygems_version: 2.0.14
 signing_key: 
 specification_version: 4
 summary: Dependency Injection system for Ruby