Merge pull request #14 from iknow/fix-merge-mutation

Don't alias subspecs when merge!ing

lib/deep_preloader/polymorphic_spec.rb

@@ -35,7 +35,7 @@ def merge!(other)
         if specs_by_type[k]
           specs_by_type[k].merge!(v)
         else
-          specs_by_type[k] = v
+          specs_by_type[k] = v.deep_dup
         end
       end
     else
@@ -54,6 +54,10 @@ def ==(other)
 
   alias eql? ==
 
+  def deep_dup
+    self.class.new(specs_by_type.deep_dup)
+  end
+
   def inspect
     "PolySpec#{specs_by_type.inspect}"
   end

lib/deep_preloader/spec.rb

@@ -38,7 +38,7 @@ def merge!(other)
         if association_specs[k]
           association_specs[k].merge!(v)
         else
-          association_specs[k] = v
+          association_specs[k] = v.deep_dup
         end
       end
     else
@@ -57,6 +57,10 @@ def ==(other)
 
   alias eql? ==
 
+  def deep_dup
+    self.class.new(association_specs.deep_dup)
+  end
+
   def inspect
     "Spec#{association_specs.inspect}"
   end

spec/unit/deep_preloader_spec.rb

@@ -34,6 +34,18 @@
     expect(DeepPreloader::Spec.parse(spec)).to eq(expected_spec)
   end
 
+  it 'can merge specs' do
+    a = DeepPreloader::Spec.parse(a: :b)
+    b = DeepPreloader::Spec.parse(a: :c)
+    result = DeepPreloader::Spec.new.merge!(a).merge!(b)
+
+    # The result should contain each of the merged specs and the merged specs
+    # themselves should not be altered
+    expect(result).to eq(DeepPreloader::Spec.parse(a: [:b, :c]))
+    expect(a).to eq(DeepPreloader::Spec.parse(a: :b))
+    expect(b).to eq(DeepPreloader::Spec.parse(a: :c))
+  end
+
   context 'with a test model' do
     with_temporary_table(:model)