@@ -45,6 +45,7 @@ def load_current_resource
         end
 
         if group_info
+          @new_resource.gid(group_info.gid)
           @current_resource.gid(group_info.gid)
           @current_resource.members(group_info.mem)
         end
@@ -58,9 +59,18 @@ def load_current_resource
       # <true>:: If a change is required
       # <false>:: If a change is not required
       def compare_group
-        !![ :gid, :members ].find do |group_attrib|
-          @new_resource.send(group_attrib) != @current_resource.send(group_attrib)
+        return true if @new_resource.gid != @current_resource.gid
+
+        if(@new_resource.append)
+          @new_resource.members.each do |member|
+            next if @current_resource.members.include?(member)
+            return true
+          end
+        else
+          return true if @new_resource.members != @current_resource.members
         end
+
+        return false
       end
 
       def action_create
 
@@ -109,7 +109,8 @@
       :null_object => true,
       :group_name => "aj",
       :gid => 50,
-      :members => [ "root", "aj"]
+      :members => [ "root", "aj"],
+      :append => false
     )
     @current_resource = mock("Chef::Resource::Group",
       :null_object => true,
@@ -128,9 +129,22 @@
     end
   end
 
-  it "should return false if no change is required" do
+  it "should return false if gid and members are equal" do
+    @provider.compare_group.should be_false
+  end
+
+  it "should return false if append is true and the group member(s) already exists" do
+    @current_resource.members << "extra_user"
+    @new_resource.stub!(:append).and_return(true)
     @provider.compare_group.should be_false
   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.stub!(:append).and_return(true)
+    @provider.compare_group.should be_true
+  end
+
 end
 
 describe Chef::Provider::Group, "action_create" do