Add a test for this situation so we don't make the same mistake twice.

Signed-off-by: Joshua Justice <jjustice6@bloomberg.net>

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