diff options
author | Joshua Justice <jjustice6@bloomberg.net> | 2019-02-06 17:47:37 -0500 |
---|---|---|
committer | Joshua Justice <jjustice6@bloomberg.net> | 2019-02-06 17:52:25 -0500 |
commit | 212c465199adb2895638685e1816f6d8fc3c6205 (patch) | |
tree | 75680b4ff830b321899eab27682b19ec1d08a0ef | |
parent | 71144fa01afceb7847847d55fd8db3118cf816ea (diff) | |
download | chef-212c465199adb2895638685e1816f6d8fc3c6205.tar.gz |
Add a test for this situation so we don't make the same mistake twice.
Signed-off-by: Joshua Justice <jjustice6@bloomberg.net>
-rw-r--r-- | spec/unit/provider/group/solaris_spec.rb | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/spec/unit/provider/group/solaris_spec.rb b/spec/unit/provider/group/solaris_spec.rb index e6000e6012..0e0f189c52 100644 --- a/spec/unit/provider/group/solaris_spec.rb +++ b/spec/unit/provider/group/solaris_spec.rb @@ -62,6 +62,15 @@ describe Chef::Provider::Group::Solaris do expect { @provider.run_action(@provider.process_resource_requirements) }.to raise_error(Chef::Exceptions::Group, "setting group members directly is not supported by #{@provider}, must set append true in group") end + it "should groupmod the whole batch when append is false" do + current_resource = @new_resource.dup + @provider.current_resource = current_resource + @node.automatic_attrs[:platform] = "solaris2" + @new_resource.append(false) + expect(@provider).to receive(:shell_out_compacted!).with("groupmod", "-U", "all,your,base", "wheel") + @provider.modify_group_members + end + platforms.each do |platform, flags| it "should usermod +/- each user when the append option is set on #{platform}" do current_resource = @new_resource.dup |