sinject 1.0.2 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 2ff53f60728f9e82cae343665133a1afb5fd25a4
-  data.tar.gz: 484be6ecc45857889ad818ac5f9cb5d4d5af86cd
+SHA256:
+  metadata.gz: 811bf8beccc5b35e04e735fefc564aebcd551cd8f7514cdb4766b8a57776b7ba
+  data.tar.gz: 2b5c71e629c158779a657a533d247081c340c01cb1b748491ce5194dc8c4e14e
 SHA512:
-  metadata.gz: 13c4fca49e08c1da9e8936b4890c087377bfc08fdf0e234b122fdd5279e5d428451c2763e04ceecbdd9a4bff290e58057fdd3db1e13e4a7a5f45b892b9d290c9
-  data.tar.gz: 670cda2067966140a1f3e1577fc24480e7b38a38aaa74366937a06de8731e997d13558ec93bceb17b45acb9c54395f803d6694bd3eb77a2f7c94917c49b0bcc7
+  metadata.gz: 48e343a9ca5e54224f5b248e47600169c3212587aacd9f1fecadbf2d1f59f4a7343ab51adba893da1200e1349d1d54cad34030f3c7c445554fdf05e94e0acfdb
+  data.tar.gz: 44da113fef31cc4eeea74ae590592871bfecb733ec6ba96edd9b3ea934cd4ec0b9f8dc536ee790596ee1934c6244024c8c3908fad8f364f5464bb05e86905089

data/lib/sinject/container.rb

@@ -7,22 +7,25 @@ module Sinject
     end
 
     def initialize(singleton=true)
-      @store = []
-      if singleton
-        Sinject::Container.instance = self
-      end
+      @store = {}
+      Sinject::Container.instance = self if singleton
     end
 
     # Check if an object has been registered with the container.
     #
     # Example:
-    #   >> Sinject::Container.instance.is_registered? :object_key
+    #   >> Sinject::Container.instance.registered? :object_key
     #   => true
     #
     # Arguments:
     #   key:  (Symbol)
+    def registered?(key)
+      @store.has_key?(key)
+    end
+    # @deprecated: Use registered? method instead
     def is_registered?(key)
-      !@store.select { |i| i.key == key}.empty?
+      puts "[#{self.class}] - #is_registered? method is deprecated please use #registered? instead."
+      registered?(key)
     end
 
     # Register an object with the container.
@@ -38,22 +41,15 @@ module Sinject
     #   class_name: (ClassName)
     #   single_instance:  (Boolean)
     def register(options = {}, &initialize_block)
+      raise Sinject::DependencyRegistrationKeyNotSpecifiedException.new unless options.has_key?(:key)
 
-      if(!options.has_key?(:key))
-        raise Sinject::DependencyRegistrationKeyNotSpecifiedException.new
-      end
-
-      if(!options.has_key?(:class))
-        raise Sinject::DependencyRegistrationClassNotSpecifiedException.new
-      end
+      raise Sinject::DependencyRegistrationClassNotSpecifiedException.new unless options.has_key?(:class)
 
       key = options[:key]
       dependency_class_name = options[:class]
 
-      #check if a dependency has already been registered for this key.
-      if is_registered?(key)
-        raise Sinject::DependencyRegistrationException.new(key)
-      end
+      # check if a dependency has already been registered for this key.
+      raise Sinject::DependencyRegistrationException.new(key) if registered?(key)
 
       single_instance = false
       contract_class_name = nil
@@ -66,11 +62,8 @@ module Sinject
         contract_class_name = options[:contract]
       end
 
-      #check if a contract has been specified
-      if contract_class_name != nil
-        #validate the dependency class against the contract
-        validate_contract(dependency_class_name, contract_class_name)
-      end
+      # Validate the dependency class against the contract if a contract has been specified
+      validate_contract(dependency_class_name, contract_class_name) unless contract_class_name.nil?
 
       item = Sinject::ContainerItem.new
       item.key = key
@@ -78,7 +71,8 @@ module Sinject
       item.class_name = dependency_class_name
       item.initialize_block = initialize_block
 
-      @store.push(item)
+      @store[item.key] = item
+      true
     end
 
     # Get an object from the container.
@@ -90,25 +84,24 @@ module Sinject
     # Arguments:
     #   key:  (Symbol)
     def get(key)
-      #get the dependency from the container store for the specified key
-      items = @store.select { |i| i.key == key}
-      if !items.empty?
-        item = items.first
-
-        #check if the item has been registered as a single instance item.
+      # get the dependency from the container store for the specified key
+      item = @store[key]
+      if !item.nil?
+        # check if the item has been registered as a single instance item.
         if item.single_instance == true
-          #check if the instance needs to be created
-          if item.instance == nil
-            item.instance = create_instance(item)
-          end
+          # check if the instance needs to be created
+          item.instance = create_instance(item) if item.instance.nil?
+
           return item.instance
         else
           return create_instance(item)
         end
       else
-        #no dependency has been registered for the specified key, attempt to convert the key into a class name and initialize it.
+        # no dependency has been registered for the specified key,
+        # attempt to convert the key into a class name and initialize it.
         class_name = "#{key}".split('_').collect(&:capitalize).join
-        puts "[Sinject] - WARNING: No registered dependency could be found for key: #{key}. Attempting to load class: #{class_name}."
+        puts "[#{self.class}] - WARNING: No registered dependency could be found for key: #{key}. " \
+"Attempting to load class: #{class_name}."
         Object.const_get(class_name).new
       end
     end
@@ -116,7 +109,7 @@ module Sinject
     def load_groups
       Sinject::DependencyGroup.descendants.each do |g|
         group = g.new
-        if group.is_valid?
+        if (group.respond_to?(:valid?) && group.valid?) || (group.respond_to?(:is_valid?) && group.is_valid?)
           group.register(self)
         end
       end
@@ -125,46 +118,69 @@ module Sinject
     private
 
     def validate_contract(dependency_class, contract_class)
-
-      #get the methods defined for the contract
+      # get the methods defined for the contract
       contract_methods = (contract_class.instance_methods - Object.instance_methods)
-      #get the methods defined for the dependency
+      # get the methods defined for the dependency
       dependency_methods = (dependency_class.instance_methods - Object.instance_methods)
-      #calculate any methods specified in the contract that are not specified in the dependency
+      # calculate any methods specified in the contract that are not specified in the dependency
       missing_methods = contract_methods - dependency_methods
 
       if !missing_methods.empty?
         raise Sinject::DependencyContractMissingMethodsException.new(missing_methods)
       end
 
-      #loop through each contract method
+      # loop through each contract method
       contract_methods.each do |method|
+        # get the contract method parameters
+        contract_params = contract_class.instance_method(method).parameters.map{ |p| { type: p[0], name: p[1] } }
 
-        #get the contract method parameters
-        cmp = contract_class.instance_method(method).parameters
-        #get teh dependency method parameters
-        dmp = dependency_class.instance_method(method).parameters
+        # get the dependency method parameters
+        dependency_params = dependency_class.instance_method(method).parameters.map{ |p| { type: p[0], name: p[1] } }
 
-        #check if the parameters match for both methods
-        if cmp != dmp
-          raise Sinject::DependencyContractInvalidParametersException.new(method, cmp)
+        errors = []
+
+        contract_params.each do |cp|
+          dp = dependency_params.detect { |p| p[:name] == cp[:name] }
+          if dp.nil? || !match?(cp, dp)
+            errors << cp[:name]
+          end
+        end
+
+        dependency_params.each do |dp|
+          cp = contract_params.detect { |p| p[:name] == dp[:name] }
+          if cp.nil?
+            errors << dp[:name]
+          end
         end
 
+        # check if any parameter errors
+        if errors.length > 0
+          raise Sinject::DependencyContractInvalidParametersException.new(method, errors)
+        end
       end
+    end
 
+    def match?(contract, dependency)
+      return true if contract[:type] == dependency[:type]
+      return true if contract[:type] == :req && dependency[:type] == :opt
+      return true if contract[:type] == :keyreq && dependency[:type] == :key
+      return false
     end
 
-    def create_instance(item)
+    # this method is called to get a standard param type for comparison purposes
+    def param_type(type)
+      return :arg if type == :opt || type == :req
+      return :key if type == :keyreq || type == :key
+    end
 
-      #check if a custom initializer block has been specified
+    def create_instance(item)
+      # check if a custom initializer block has been specified
       if item.initialize_block != nil
-        #call the block to create the dependency instance
+        # call the block to create the dependency instance
         instance = item.initialize_block.call
 
-        #verify the block created the expected dependency type
-        if !instance.is_a?(item.class_name)
-          raise Sinject::DependencyInitializeException.new(item.class_name)
-        end
+        # verify the block created the expected dependency type
+        raise Sinject::DependencyInitializeException.new(item.class_name) unless instance.is_a?(item.class_name)
       else
         instance = item.class_name.new
       end
@@ -172,4 +188,4 @@ module Sinject
       instance
     end
   end
-end
+end

data/lib/sinject/dependency_group.rb

@@ -1,17 +1,10 @@
 module Sinject
   class DependencyGroup
-
     def register(container)
-
-    end
-
-    def is_valid?
-      return true
     end
 
     def self.descendants
       ObjectSpace.each_object(Class).select { |klass| klass < self }
     end
-
   end
-end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sinject
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 1.1.0
 platform: ruby
 authors:
 - Sage One
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2016-11-30 00:00:00.000000000 Z
+date: 2018-03-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -87,7 +87,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.5
+rubygems_version: 2.7.6
 signing_key: 
 specification_version: 4
 summary: Simple Dependency Injection.