Merge pull request #27 from mattfawcett/master

Coercion does not work with MergeInitializer
hashie · Feb 12, 2013 · c2815fa · c2815fa
2 parents 60dacaa + 68ed0a3
commit c2815fa
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 2 deletions.
diff --git a/lib/hashie/extensions/merge_initializer.rb b/lib/hashie/extensions/merge_initializer.rb
@@ -17,7 +17,9 @@ module Extensions
     module MergeInitializer
       def initialize(hash = {}, default = nil, &block)
         default ? super(default) : super(&block)
-        update(hash)
+        hash.each do |key, value|
+          self[key] = value
+        end
       end
     end
   end

diff --git a/spec/hashie/extensions/coercion_spec.rb b/spec/hashie/extensions/coercion_spec.rb
@@ -17,7 +17,10 @@ def self.coerce(obj)
   end
 
   before(:each) do
-    class ExampleCoercableHash < Hash; include Hashie::Extensions::Coercion end
+    class ExampleCoercableHash < Hash
+      include Hashie::Extensions::Coercion
+      include Hashie::Extensions::MergeInitializer
+    end
   end
   subject { ExampleCoercableHash }
   let(:instance){ subject.new }
@@ -39,6 +42,13 @@ class ExampleCoercableHash < Hash; include Hashie::Extensions::Coercion end
       instance[:foo].value.should == "String"
       instance[:foo].should_not be_coerced
     end
+
+    it "should coerce when the merge initializer is used" do
+      subject.coerce_key :foo, Coercable
+      instance = subject.new(:foo => "bar")
+
+      instance[:foo].should be_coerced
+    end
   end
 
   describe '.coerce_value' do