switching groups back to having namevar be a parameter instead of a state

git-svn-id: https://reductivelabs.com/svn/puppet/trunk@661 980ebf18-57e1-0310-9a29-db15c13687c0

1 files changed, 10 insertions, 14 deletions

diff --git a/test/types/tc_group.rb b/test/types/tc_group.rb
index 2fbdeb4d8..c2288fc3d 100755
--- a/test/types/tc_group.rb
+++ b/test/types/tc_group.rb
@@ -24,13 +24,7 @@ class TestGroup < TestPuppet
         end
         @@tmpgroups.each { |group|
             unless missing?(group)
-                # if we've gotten this far, the group exists, so remove it
-                Puppet.info "Cleaning %s" % group
-                group = Puppet::Type::Group.new(
-                    :name => group
-                )
-                group[:name] = :notfound
-                group.state(:name).sync
+                remove(group)
             end
         }
         super
@@ -60,6 +54,10 @@ class TestGroup < TestPuppet
 
             return nil
         end
+
+        def remove(group)
+            system("niutil -destroy / /groups/%s" % group)
+        end
     else
         def missing?(group)
             begin
@@ -78,6 +76,10 @@ class TestGroup < TestPuppet
 
             return nil
         end
+
+        def remove(group)
+            system("groupdel %s" % group)
+        end
     end
 
     def groupnames
@@ -238,13 +240,7 @@ class TestGroup < TestPuppet
                 end
             }
 
-            es = nil
-            if os == "Darwin"
-                es = [:group_modified, :group_deleted]
-            else
-                es = [:group_deleted]
-            end
-            assert_rollback_events(trans, es, "group")
+            assert_rollback_events(trans, [:group_deleted], "group")
 
             assert(missing?(name), "Group %s is still present" % name)
         end