summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTim Smith <tsmith@chef.io>2018-02-23 10:14:59 -0800
committerTim Smith <tsmith@chef.io>2018-03-09 10:41:50 -0800
commitb044f08c549bea14b654647f291f140cd3a7c21a (patch)
treec51f3dc763e8a3e6380aae766f258386f06570d1
parent94ebd5fa1191a7643606d6e1c73af21be9092cfc (diff)
downloadchef-group.tar.gz
Don't modify frozen arrays in the specsgroup
Signed-off-by: Tim Smith <tsmith@chef.io>
-rw-r--r--spec/unit/provider/group_spec.rb8
1 files changed, 4 insertions, 4 deletions
diff --git a/spec/unit/provider/group_spec.rb b/spec/unit/provider/group_spec.rb
index c3e0d4b345..5073aa20d1 100644
--- a/spec/unit/provider/group_spec.rb
+++ b/spec/unit/provider/group_spec.rb
@@ -108,20 +108,20 @@ describe Chef::Provider::User do
end
it "should return true if append is true and the group member(s) do not already exist" do
- @new_resource.members << "extra_user"
+ @new_resource.members ["extra_user"]
@new_resource.append(true)
expect(@provider.compare_group).to be_truthy
end
it "should return false if append is true and excluded_members include a non existing member" do
- @new_resource.excluded_members << "extra_user"
+ @new_resource.excluded_members ["extra_user"]
@new_resource.append(true)
expect(@provider.compare_group).to be_falsey
end
it "should return true if the append is true and excluded_members include an existing user" do
- @new_resource.members.each { |m| @new_resource.excluded_members << m }
- @new_resource.members.clear
+ @current_resource.members "some_user_that_will_never_exist"
+ @new_resource.excluded_members "some_user_that_will_never_exist"
@new_resource.append(true)
expect(@provider.compare_group).to be_truthy
end