diff options
author | Tim Smith <tsmith@chef.io> | 2018-02-23 10:14:59 -0800 |
---|---|---|
committer | Tim Smith <tsmith@chef.io> | 2018-03-09 10:41:50 -0800 |
commit | b044f08c549bea14b654647f291f140cd3a7c21a (patch) | |
tree | c51f3dc763e8a3e6380aae766f258386f06570d1 | |
parent | 94ebd5fa1191a7643606d6e1c73af21be9092cfc (diff) | |
download | chef-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.rb | 8 |
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 |