Fixing #293, I think. The problem was that the groups state was not correctly passing strings in all cases, which caused some very strange problems internally.

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

1 files changed, 9 insertions, 1 deletions

diff --git a/test/types/user.rb b/test/types/user.rb
index 958cf9441..703004f57 100755
--- a/test/types/user.rb
+++ b/test/types/user.rb
@@ -283,6 +283,8 @@ class TestUser < Test::Unit::TestCase
             user.retrieve
         }
 
+        assert_instance_of(String, user.state(:groups).should)
+
         # Some tests to verify that groups work correctly startig from nothing
         # Remove our user
         user[:ensure] = :absent
@@ -312,7 +314,7 @@ class TestUser < Test::Unit::TestCase
             user[:groups] = main
         }
 
-        assert_equal((main + extra).sort, user.state(:groups).should.sort)
+        assert_equal((main + extra).sort, user.state(:groups).should.split(",").sort)
 
         assert_nothing_raised {
             user.retrieve
@@ -348,6 +350,12 @@ class TestUser < Test::Unit::TestCase
         list = user.state(:groups).is
         assert_equal(main.sort, list.sort, "Group list is not equal")
 
+        # Set the values a bit differently.
+        user.state(:groups).should = list.sort { |a,b| b <=> a }
+        user.state(:groups).is = list.sort
+
+        assert(user.state(:groups).insync?, "Groups state did not sort groups")
+
         user.delete(:groups)
     end