From c06e88fee3ab99b728bda3f9018a12e1dfde7289 Mon Sep 17 00:00:00 2001 From: Jay Mundrawala Date: Mon, 17 Aug 2015 11:19:43 -0500 Subject: Don't modify members of new_resource in pw group implmentation This was causing the ``` Chef::Resource::Group group modify action when there is a group behaves like correct group management when the users exist when append is not set when group already contains some users should remove all existing users and only add the new users to the group ``` spec to fail. The test passes an array which gets modified which breaks the test. --- lib/chef/provider/group/pw.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/chef/provider/group/pw.rb b/lib/chef/provider/group/pw.rb index f877ed2424..5b5c8136f1 100644 --- a/lib/chef/provider/group/pw.rb +++ b/lib/chef/provider/group/pw.rb @@ -109,7 +109,7 @@ class Chef else # Append is not set so we're resetting the membership of # the group to the given members. - members_to_be_added = @new_resource.members + members_to_be_added = @new_resource.members.dup @current_resource.members.each do |member| # No need to re-add a member if it's present in the new # list of members -- cgit v1.2.1