Merge pull request #182 from lmello/fix_bug_20681_delete_values

delete_values() fix bug #20681.

Author: adrienthebo

lib/puppet/parser/functions/delete_values.rb

@@ -21,6 +21,6 @@ module Puppet::Parser::Functions
       raise(TypeError, "delete_values(): First argument must be a Hash. " + \
                        "Given an argument of class #{hash.class}.") 
     end
-    hash.delete_if { |key, val| item == val }
+    hash.dup.delete_if { |key, val| item == val }
   end
 end

spec/unit/puppet/parser/functions/delete_values_spec.rb

@@ -27,4 +27,10 @@
     result.should(eq({ 'a'=>'A', 'B'=>'C' }))
   end
 
+  it "should not change origin hash passed as argument" do 
+    origin_hash = { 'a' => 1, 'b' => 2, 'c' => 3 } 
+    result = scope.function_delete_values([origin_hash, 2])
+    origin_hash.should(eq({ 'a' => 1, 'b' => 2, 'c' => 3 }))
+  end 
+
 end